If und Boolean

[Errelion]

Ich versuche in einem Programm einen "Always on Top" Knopf zu erstellen, und habe dazu ein wenig geschrieben. Ich bin nun soweit, das ich eine Boolean habe, deren Wert beim Drücken des Knopfes geändert wird. Jetzt will aber die if-Abfrage nicht mitspielen und setzt das Fenster entweder dauerhaft auf AlwaysOnTop oder aber nie. Ich versuche jetzt schon seit mehreren Stunden den Fehler zu beheben und habe auch schon ein paar Foren durchgeschaut, bin aber zu noch keinem Ergebniss gekommen, und hoffe ihr könnt mir weiterhelfen. Hier ein paar Ausschnitte aus dem Quellcode auf den es ankommt:

public class klicker extends JFrame implements ActionListener {

boolean top;

     public klicker() {

     if (top == true){
     setAlwaysOnTop(true);
     }
}

public void actionPerformed(ActionEvent event) {

		String cmd = event.getActionCommand();
		if (cmd.equals("Top") & top == (true)){
			top = false;
			System.out.println(top);
		}
		else{
			if (cmd.equals("Top")){
				top = true;
				System.out.println(top);
			}
}


public static void main(String[] args) {
		new klicker();
	}
}

Jedes mal wenn ich jetzt auf den "Top"-Button klicke, schreibt er mir in die Konsole abwechselnd true und false, also denke ich nicht das der Fehler hier liegt. Aber ich habe mit mehreren Schreibweisen in der ersten If-Abfrage noch kein Ergebniss erzielt, außer wie ich schon sagte ein dauerhaftes an/aus von AlwaysOnTop. Danke für die Hilfe schonmal im Vorraus

 

[bone2]

naja dir fehlt ein setAlwaysOnTop(false); um es wieder abzuschalten

und top == true ist unötig
if (top) {} reicht, bei equals && top) gilt das gleiche

edit besser: if weglassen und einfach setAlwaysOnTop(top);

 

[Beni]

Ein Program macht genau das was du ihm sagst, und nicht mehr. In diesem Fall bewirkt ein Klicken auf den Knopf dass "top" umgeschalten wird und der neue Wert ausgegeben, und nicht mehr. Du musst "setAlwaysOnTop" jedesmal aufrufen, wenn du dein "top" veraenderst.

 

[SlaterB]

allein zum boolean:

if (cmd.equals("Top")) {
  top = !top;
}

 

[Errelion]

Danke für die schnellen Antworten, allerdings haben die ersten beiden Vorschläge leider nicht funktioniert, auch wenn mir der Punkt mit
setAlwaysOnTop(top);
sehr gefallen hätte, dennoch geht es bei mir aus für mich unbekannten Gründen nicht.


Edit: Ich habe gerade eine kleine Änderung durchgeführt, weil Beni sagte, ich solle jedesmal setAlwaysOnTop aufrufen. Mit

if (cmd.equals("Top")){
			top = !top;
			setAlwaysOnTop(top);
			System.out.println(top);
		
			}

funktioniert alles ohne Probleme! Danke euch