if else Abfrage

[Ci]

Hallo,

ich hab ne Oberfläche, in der ich sieben JToggleButtons implementiert hab.
je nachdem welcher Button angeklickt wurde (kann max. ein Button angeklickt werden)
soll die jeweilige Zahl auf der Konsole ausgegeben werden.

ist ein klasischer if-else-if Abfrage dacht ich, nur funktioniert das nicht so wie ich es gerne haben wollte:

if (toggleButton1.isVisible()){
					System.out.println("1");
				}else{
					if (toggleButton2.isVisible()){
					System.out.println("2");
				}else{
					if (toggleButton3.isVisible()){
					System.out.println("3");
				}else{
					if (toggleButton4.isVisible()){
					System.out.println("4");
				}else{
					if (toggleButton5.isVisible()){
					System.out.println("5");
				}else{
					if (toggleButton6.isVisible()){
					System.out.println("6");
				}else{
					if (toggleButton7.isVisible()){
					System.out.println("7");
					}	
									}
								}
							}
						}
					}
				}

...wenn ich z.b. auf den Button 5 drücke, wird die 1 auf der Konsole ausgegeben.
Es wird jedesmal die 1 ausgegeben!!!

was mach ich da falsch?
oder ist das doch keine if-else-if Abfrage?

gruß

 

[truesoul]

Hallo...

if (toggleButton1.isVisible()){
     System.out.println("1");
}else if (toggleButton2.isVisible()){
     System.out.println("2");
}else if (toggleButton3.isVisible()){
     System.out.println("2");
} 

...

Wäre schöner zu lesen und für den Fall angebrachter.

Warum immer 1 ausgegeben wird wird wohl daran liegen das toggleButton1.isVisible() true ist, immer
Also immer "Sichtbar"

EDIT : :lol:

ausgeführt aber niemals die anderen.

Wer hat das denn geschrieben, tztz ...

Jep sind sie

 

[SlaterB]

an diesem Code liegt es nicht, obwohl man ihn kürzer schreiben könnte:
if () {

} else if () {

} else ..

muss an restlichen Code liegen, vielleicht ist toggleButton1 immer visible?
fange mit nur zwei Buttons an und schreibe vorher:
System.out.println("die Situation der Welt ist folgende: button1 visible?: "+ ..., button2 .. );


edit
@truesoul
> ausgeführt aber niemals die anderen.

sind nicht alle anderen ifs und else genau im else-Fall des vorherigen ifs?

edit: ok, inzwischen geändert

 

[Volvagia]

Schau mal, auf was du überhaupt prüfst.
Das hat nämlich recht wenig mit "angeklickt" zu tun.

btw. schreib nicht

else{
	if (...){
		...
}

sondern

else if (...) {
	 	...
}

Und du solltest auf Arrays umsteigen.

 

[Gast2]

Schau mal, auf was du überhaupt prüfst.
Das hat nämlich recht wenig mit "angeklickt" zu tun.

Stichwort ActionListener (siehe: Java ist auch eine Insel – 15.7 Eine Schaltfläche (JButton))

 

[Ci]

ok, hat sich geklärt.
Lag tatsächlich nicht am if-else-if, sondern an der Selektierung meines Buttons!!!
statt isVisible musste natürlich is Selected rein!!!

die anderen Verbesserungsvorshläge habe ich mit reinimplementiert.

Vielen Dank an alle

 

[truesoul]

Ja aber was ist wenn du 2 Button selektiert hast?
Dann würde dein jetztiger If else If nicht mehr funktionieren

 

[Gast2]

ok, hat sich geklärt.
Lag tatsächlich nicht am if-else-if, sondern an der Selektierung meines Buttons!!!
statt isVisible musste natürlich is Selected rein!!!

die anderen Verbesserungsvorshläge habe ich mit reinimplementiert.

Vielen Dank an alle

Die Selektierung zu prüfen anstatt dem Klick ist trotzdem falsch. Was ist zum Beispiel wenn der Button nach einer Minute immer noch selektiert ist? Oder wenn er ohne Klick selektiert wurde?

Schau dir die ActionListener unbedingt an. An denen kommst du ohnehin nicht vorbei.

 

[SlaterB]

ob das falsch ist oder nicht kann man von außen doch nicht sagen,
vielleicht ist es ein Formular mit 5 Textfeldern, den Toggeln und sonstigen Einstellungen,
unten ein Absenden-Button,

dann will man genau zum Arbeitszeitpunkt die Selektion wissen, nicht bei der Auswahl

Edit:
ok, evtl. wegen:
> je nachdem welcher Button angeklickt wurde

 

[Ci]

Ja aber was ist wenn du 2 Button selektiert hast?

ich kann nur max. 1 Button anklicken! Hab alle meine Buttons einem ButtonGroup zugewiesen!

Was ist zum Beispiel wenn der Button nach einer Minute immer noch selektiert ist? Oder wenn er ohne Klick selektiert wurde?

Schau dir die ActionListener unbedingt an. An denen kommst du ohnehin nicht vorbei.

der Button wird nur durch den Anwender der Oberläche angeklickt! Die Zeit spielt da keine Rolle!
Das ganze befindet sich auch im ActionListener!

gruß

 

[Gast2]

ich kann nur max. 1 Button anklicken! Hab alle meine Buttons einem ButtonGroup zugewiesen!



der Button wird nur durch den Anwender der Oberläche angeklickt! Die Zeit spielt da keine Rolle!
Das ganze befindet sich auch im ActionListener!

gruß

Alles klar. Das nächste mal poste einfach etwas mehr Sourcecode, dann hätten wir den ActionListener auch gesehn. Ich versteh nur immer noch nicht so ganz was du machst aber da es nun funktioniert isses ja auch OK.

 

[Ci]

hehe, sorry

der restliche Teil hat ja funktioniert, deshalb wollt ich hier die Seite nicht vollposten.

Ich versteh nur immer noch nicht so ganz was du machst

meine Oberfläche hat sieben Buttons, auf jedem Button befindet sich eine Obstsorte als icon, der Anwender muss täglich eine Obstsorte zu sich nehmen (wer mehr davon zu sich nimmt, spielt in dieser Abfrage keine Rolle)
und bei der Abfrage dann eben die entsprechende Obstsorte auswählen.
Natürlich werden noch weitere Daten erfasst.
Anschließend wird auf ein Prüfung abschließen Button geklickt, dass dann die ganzen Daten in eine Datenbank ablegt. Und so hab ich die ganze Daten von einem, der eine Diät über eine länger Zeit hinweg macht!