login fehler

[breckal]

Hey Leute!

Bin Umsteiger (habe mal viel PHP programmiert) und bin nun bei meinem ersten Login-Fenster. Habe bereits die Suchfunktion probiert, jedoch hab ich nix gefunden was mir hier weiterhilft.

Grundsätzlich hab ich ein Fenster mit zwei JTextLabel - Wenn man bei beiden TextLabels die gleichen Strings übergibt soll ein Dialog kommen mit "Die eingegebenen Daten sind in Ordnung!" - wenn man falsche daten übergibt, dann soll "Die eingegebenen Daten sind NICHT in Ordnung!" kommen.

Aber irgendwie funktioniert die if funktion nicht richtig und es wird immer nur die else bedingung ausgegeben ....

import java.awt.FlowLayout;
import javax.swing.JFrame;
import javax.swing.JTextField;
import javax.swing.JOptionPane;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import javax.swing.JButton;


public class login extends JFrame {
		
	private JTextField text1;
	private JTextField text2;
	private JButton button1;
	private JButton button2;
	
	
	public login(){
		super("Titlebar");
		setLayout(new FlowLayout());
		
		text1 = new JTextField("hallo");
		add(text1);
		text2 = new JTextField("hallo");
		add(text2);
		button1 = new JButton("OK");
		add(button1);
		button2 = new JButton("Abbrechen");
		add(button2);
		
		thehandler handler = new thehandler();
		button2.addActionListener(handler);
		button1.addActionListener(handler);
		
		
	}
	
	private class thehandler implements ActionListener{
		
		private String user = new String(text1.getText());
		private String pw = new String(text2.getText());
		private String text = "";
		
		public void actionPerformed(ActionEvent event){
			
			if(event.getSource() == button1){
				if(user == pw)
					text = String.format("Die eingegebenen Daten sind in Ordnung! %s, %s", user, pw);
				else
					text = String.format("Die eingegebenen Daten sind NICHT in Ordnung! %s, %s", user, pw);
				
				
			}
			else {
				text = String.format("Du hast die Aktion abgebrochen!");
			}
			
			JOptionPane.showMessageDialog(null, text);
		}
			
	}
	
	
}

Könnt ihr mir sagen wo hier der Fehler liegt???

Danke gleich im Voraus

lg breckal

 

[njans]

Abgesehen von der Semantik:
(user == pw)
Das geht wohl immer schief, da du da auf Referenzgleichheit prüfst.
Probiere mal (user.equals(pw))!

P.S. Du solltest versuchen dich an die Java Notation zu gewöhnen, Klassen immer groß schreiben.
Zusammengesetze Namen werden im CamelCase geschrieben (z.B. MeinKlasse oder bei Methoden doSomething)

 

[breckal]

super, danke für die schnelle Antwort!

jetzt wird immer der if-zweig ausgeführt - auch wenn ich unterschiedliche werte in die felder eingeb!


Ja, java funktioniert doch ziemlich anders als php - da muss ich mich noch dran gewöhnen!

 

[njans]

Dann mach doch mal:
System.out.println(pw + " : " + user);

Dann kannst du dir mal angucken, ob die gleich sind

 

[breckal]

ja, hab ich schon probiert - seitdem ich mit (user.equals(pw)) vergleiche haben weder user noch pw einen wert ...... somit is es auch klar dass die if-bedinung ausgeführt wird

 

[njans]

Ah was du nicht machen solltest:
Dir die Werte als Instanzvariablen zu holen

 private String user = new String(text1.getText());
 private String pw = new String(text2.getText());

Da bezieht der die Werte der Felder, wenn du den Handler erstellst.
Hol dir die werte lieber in der actionPerformed Methode, dann geht das auch

Nebenbei:
String user = new String(text1.getText());
ist umständlicher als:
String user = text1.getText();

 

[breckal]

habs gerade rausgefunden .....

schon klar wenn man die variablen auf nen wert setzt obwohl es noch keinen wert gibt .... tztztz

möchte nochmal danke sagen für deine hilfe!!

lg breckal

 

[breckal]

hatte deine antwort noch nicht gelesen gehabt ......

wenn mans nochmal genau durchsieht kommt man ja oft eh von selber drauf .....