Button Dis/Enablen - Was mache ich falsch?

[Topas]

Hi,


ich habe einen Button definiert, der nur per Passwortcheck aktiv wird.

Dieser Check wird erst ausgeführt, wenn ein weiterer Button gedrückt wird.

Dazu baue ich eine Verbindung mit dem Passwort zu meinem Server auf.

Wenn das Passwort richtig ist, gibt das PHP Script "OK!" aus. Dieses "OK!" lasse ich in einem JLabel anzeigen.

Wenn das Label " OK!" zeigt, wird der Button enabled. Das klappt alle wunderbar, nur leider geht das ganze auch, wenn ich ein falsches Passwort angebe. Der Server gibt dann zwar "Falsches Passwort" zurück, der Button wird trotzdem enabled.

        final JLabel checkFieldLabel = new JLabel("     Falsches Passwort!");
        jbutton3.addActionListener(new ActionListener() {

            public void actionPerformed(ActionEvent actionevent)
            {
                try {
					checkFieldLabel.setText((new StringBuilder()).append("     ").append(getProtect(checkField.getText())).toString());
				} catch (IOException e) {
					e.printStackTrace();
				}
				try {
					if(getProtect(checkField.getText()).equals("     OK!"));
					{
						jbutton.setEnabled( true );	
					}
				} catch (IOException e) {
					e.printStackTrace();
				}
            }   
        });

[...]

		jbutton.setEnabled( false );

Wo liegt hier der Fehler? :lol:


lg

 

[Wildcard]

Wenn das was anderes als ein Spaßprojekt ist, dann ist die Sicherung so lächerlich das du sie ganz weg lassen kannst.
Zu deinem Fehler: mach mal den ; hinter dem if weg.

 

[Topas]

Argh!

Der erste Code ist FALSCH

Ich hatte versehentlich einen älteren Versuch kopiert >.<

Richtig schaut es im Moment so aus:

        final JLabel checkFieldLabel = new JLabel("     Falsches Passwort!");
        jbutton3.addActionListener(new ActionListener() {

            public void actionPerformed(ActionEvent actionevent)
            {
                try {
					checkFieldLabel.setText((new StringBuilder()).append("     ").append(getProtect(checkField.getText())).toString());
				} catch (IOException e) {
					e.printStackTrace();
				}
					if(checkFieldLabel.equals("     OK!"));
					{
						jbutton.setEnabled( true );	
					}
            }   
        });

 

[Topas]

Ja, das ist ein Spaßprojekt.

Vielen Dank, es hat geklappt!