Java-Anwendung if/case-Empfehlung

Question

Ich schreibe eine Anwendung für einen Java-Kurs. Ich bin ein kompletter Anfänger und gehe gerade Material aus, das ich aus dem Kurs und aus dem Internet gelernt habe. Die Anwendung zeigt etwas Verhalten und ich bin mir nicht sicher, was es verursacht. Die Anwendung ist GUI-basiert und führt Berechnungen für Benutzereingaben durch. Für den Aktion Hörer Abschnitt habe ich eine Reihe von If-Anweisungen wie:

alle in einer Reihe wie die „wenn diese Schaltfläche, um diese tun, wenn Sie diese Taste, um dies tun“. Es sieht so aus, als würde die Anwendung ALLE if-Anweisungen ausführen, statt die zu starten, die der gedrückten Schaltfläche entspricht.

Wäre es besser, eine Case/Switch-Struktur für solche Dinge zu verwenden?

Ich kann meinen Code wenn nötig, ich bin neu um diese Website und bin mir nicht sicher, ob das Ding akzeptabel ist.

Answer 1

Sie sollten wahrscheinlich keinen einzigen Aktionslistener haben - ein separater für jedes Steuerelement zu erstellen hilft dabei, den Code lesbarer zu halten.

Wenn Ihre Steuerelemente jedoch viel Code freigeben (das Zeug vor oder nach den if-Anweisungen), dann könnte es sinnvoll sein, dies auf diese Weise zu tun. In diesem Fall sollte es if/elseif sein.

Auch Ihre Listener vollwertige Klassen (anstatt anonyme innere Klassen) können helfen, Code wieder zu verwenden (das Zeug in jeder if-Anweisung geht in jede Unterklasse). Dies kann über das hinausgehen, was Sie bisher gelernt haben.

Edit: (mehr direkte Antwort auf Ihre eigentliche Frage)

Soweit Ihr Problem, das wenn nicht alles ausführen soll, es sei denn Antwort entweder drs9222 ist, daß die Sie am Ende Ihrer wenn ein Semikolon wurden unter Verwendung von ist richtig, oder Ihre if-Anweisung testet die falsche Sache.

Sie könnten nur das, was Sie testen, posten, aber Sie müssen Ihr bekanntes "Button" -Objekt mit event.getSource() vergleichen, indem Sie == oder .equals verwenden. Da jedes Button-Objekt, mit dem Sie vergleichen, anders ist, sollte nur eines ausgeführt werden.

Answer 2

Ohne den Code zu sehen, kann ich nicht sicher sagen, aber etwas, was ich immer gemacht habe, war einfach if und nicht else if - wenn mehrere meiner Bedingungen erfüllt waren, dann wurde der gesamte zugehörige Code ausgeführt. Wenn nur die erste Übereinstimmung ausgeführt werden soll, verwenden Sie if-else, um sicherzustellen, dass keine der folgenden Anweisungen ausgeführt wird.

Answer 3

Bis ich den Code zu sehen Ich werde erraten müssen aber Ihre zugelassen Neuartigkeit gegeben Sie können schreiben, wenn Aussagen wie diese

if (condition); 
{ 
    ... 
} 

statt wie diese

if (condition) 
{ 
    ... 
}