Ampelschaltung mit if-Befehl

[Viktor A. Kaiser]

Guten Abend zusammen,

wie man dem Titel schon entnehmen kann, geht es um das bekannte Thema der Ampelschaltung.

Ich habe zwar schon so einiges hier aus dem Forum mir diesbezüglich angeschaut, doch kam ich mit den jeweiligen Kommentaren in der Struktur meines Programmes nicht weiter.
Mein Problem ist, dass alle Ampelphasen angezeigt werden bis auf eine. Dabei hängt das davon ab, in welcher Reihenfolge die if-Befehle stehen. Je nach dem fehlt entweder einfach rot oder wie hier im dargestellten Abschnitt gelb.
Das hängt damit zusammen, dass das Programm zwar am Anfang richtig schaltet, aber später wieder anders, weil sich die Zustände halt verändert haben.

Dabei habe ich mich unter anderem auch schon an if-Verschachtelung probiert und an diesem Prinzip ( https://www.java-forum.org/thema/ampelschaltung-mit-if-anweisung.170408/ ) orientiert, doch nach dieser Ordnung ging noch vieles mehr nicht.

Ich hoffe, dass man mir zeitnah weiterhelfen kann und bedanke mich im Voraus.

Mit freundlichen Grüßen

Viktor A. Kaiser

if (panelGelb.getBackground() == Color.BLACK&&panelGruen.getBackground() == Color.GREEN&&panelRot.getBackground() == Color.BLACK) {
    panelGruen.setBackground(Color.black);
    panelRot.setBackground(Color.black);
    panelGelb.setBackground(Color.yellow);
    }
        
if(panelRot.getBackground() == Color.RED&&panelGelb.getBackground() == Color.YELLOW){     
    panelGruen.setBackground(Color.green);
    panelRot.setBackground(Color.black);
    panelGelb.setBackground(Color.black);
    }
          
if (panelRot.getBackground() == Color.RED&&panelGelb.getBackground() == Color.BLACK) {
    panelRot.setBackground(Color.red);
    panelGruen.setBackground(Color.black);
    panelGelb.setBackground(Color.yellow);
    }
          
if (panelGelb.getBackground() == Color.YELLOW&&panelGruen.getBackground() == Color.BLACK&&panelRot.getBackground() == Color.BLACK) {
    panelGruen.setBackground(Color.black);
    panelRot.setBackground(Color.red);
    panelGelb.setBackground(Color.black);
    }         

if(panelGelb.getBackground() == Color.BLACK&&panelRot.getBackground() == Color.BLACK&&panelGruen.getBackground() == Color.BLACK){    //&& = und
    panelRot.setBackground(Color.red);
    }

 

[yfons123]

hattest du klassen schon ?

 

[Viktor A. Kaiser]

Nein, noch nicht.
An sich soll die Aufgabe auch nach so einem ähnlichen Prinzip erstellt werden. Als Tipp nannte man mir zwar die Verkettung von if-Abfragen,
doch kann man meiner Meinung nach das doch auch nach so einer Vorgehensweise funktionieren, oder? Es ist ja zumindest nur eine einzige Ampel-Phase, die nicht funktioniert...

 

[yfons123]

ich würde dir raten system.out.printlns einzufügen was im moment passiert
ob du das if überhuapt betrittst
und ob du das if sofort wieder überschreibst

 

[KonradN]

Dein Problem hier ist vermutlich schlicht und einfach, dass Du alle ifs durchläufst. Damit wird in einem Zyklus mehrfach geschaltet.

Es gibt zwei Möglichkeiten, die hier in Frage kommen:
1. Du baust eine if ... else if .... else if .... else if .... Konstruktion, d.h. Du baust else mit ein.
2. Wenn es in einer Klasse ist, dann kannst Du nach dem Umschalten mit return zurück gehen. Ans Ende jedes Blockes kommt dann also ein return.

Das sind einfache Lösungen. Generell bietet es sich an, das auch alles etwas umzuschreiben. Das ist aber abhängig davon, was Du so schon alles hast. Hier könnte man z.B. jede Phase als Enum abbilden und dann hast Du einfache Übergänge (z.B. mit einem switch Statement oder als Status innerhalb des Enums). Dazu muss man aber Enums und Klassen etwas besser kennen, denn die Enums hätten dann auch jeweils weitere Werte intern gespeichert.

 

[Viktor A. Kaiser]

Also, was, wie passiert, das weiß ich bereits.
Wenn es grün ist, wird nicht auf gelb, sondern auf rot direkt umgeschaltet. Grund dafür ist, dass zwar der folgende Befehl zunächst richtigerweise auf gelb umschaltet,

if (panelGelb.getBackground() == Color.BLACK&&panelGruen.getBackground() == Color.GREEN&&panelRot.getBackground() == Color.BLACK) {
    panelGruen.setBackground(Color.black);
    panelRot.setBackground(Color.black);
    panelGelb.setBackground(Color.yellow);
    }

aber die danach kommende if-Abfrage auf rot umschaltet, weil inzwischen im Programm mittlerweile ja auf gelb umgestellt worden ist.

 if (panelGelb.getBackground() == Color.YELLOW&&panelGruen.getBackground() == Color.BLACK&&panelRot.getBackground() == Color.BLACK) {
    panelGruen.setBackground(Color.black);
    panelRot.setBackground(Color.red);
    panelGelb.setBackground(Color.black);
    }

Wenn ich nun allerdings das anders sortiere, dann ist es im nächsten Schritt eine andere Ampelphase nicht schaltet möchte, z.B. rot-gelb.

Ergänzung zu KonradN:

Klassen sind noch nicht Teil meiner Kenntnisse, das schließt Enum ein. Darüber habe ich in Folge meiner Recherchen tatsächlich auch schon einiges gelesen, doch sollte das Programm besser auf den bereits vermittelten Bildungsstand basieren. Dementsprechend wären if....else if, etc. schon besser.
Eine solche ähnliche Variante habe ich tatsächlich auch schon ausprobiert, doch ging dann nur noch rot und rot-gelb. Ich suche gerade mal dieses Programm nochmal raus und stelle es gleich hier rein.

Zweite Ergänzung mit dem Programm:

if (panelRot.getBackground() == Color.RED) {
    panelGruen.setBackground(Color.black);
    panelRot.setBackground(Color.red);
    }
        
    else if (panelRot.getBackground() == Color.RED&&panelGelb.getBackground() == Color.YELLOW) {
        panelGruen.setBackground(Color.black);
        }
          
    else if (panelGruen.getBackground() == Color.GREEN) {
        panelGruen.setBackground(Color.black);
        panelRot.setBackground(Color.black);
        panelGelb.setBackground(Color.yellow);
        }
            
    else {
        panelRot.setBackground(Color.red);
        }

 

[yfons123]

naja deswegen wurde dir bei der aufgaben schaltung zu verkettung geraten

if(){
   
}else if() {
   
}
// vs
if(){
   
}
if(){
   
}

wo ist denn hier der unterschied

 

[Viktor A. Kaiser]

Also beim ersten heißt es doch, dass, wenn die eine Bedingung zutrifft, dann mache das folgende, wenn noch das zutrifft - denke ich....
Beim zweiten ist es klassisch, wenn das eine zutrifft, dann mache das und wenn das andere zutrifft, dann mache das.

 

[KonradN]

Wenn Du einfach bei dem Code, den Du oben gezeigt hast, vor jedes if (außer dem ersten natürlich!) ein else packst, dann sollte es schon funktionieren.

Generell solltest Du den Code aber leserlicher schreiben. Bei den && solltest Du mindestens ein Leerzeichen davor und danach einfügen. Aber es empfiehlt sich bei so langen Zeilen, das auch einfach aufzuteilen auf mehrere Zeilen. Dann sieht es z.B. so aus:

        if (panelGelb.getBackground() == Color.BLACK
                && panelGruen.getBackground() == Color.GREEN
                && panelRot.getBackground() == Color.BLACK) {
            panelGruen.setBackground(Color.black);
            panelRot.setBackground(Color.black);
            panelGelb.setBackground(Color.yellow);
        }

 

[Viktor A. Kaiser]

Danke, das hat tatsächlich geholfen! Die Ampel funktioniert nun einwandfrei. So nah und doch so fern, wie ich immer so schön sage.
Aber könnten wir diesbezüglich kurz nochmal auf den Hintergrund eingehen, inwiefern diese Verkettung nun funktioniert?
Läuft das nach dem Prinzip ab, das ich in der Nachricht zuvor erwähnte?

 

[KonradN]

Evtl. meinst Du das Richtige, aber Du hast es aus meiner Sicht nicht gut ausgedrückt.

Das mit dem if ist ja ein Wenn ... Dann ...

Mit dem Else wird das zu einem if / else und wenn man es ausdrücken würde, dann wäre es ein Wenn ... Dann ... sonst ...

Das if else if ist nichts eigenständiges.

Du hast das:

if (Bedingung) {
    // ...
} else {
    // ...
}

und nun kommt in den else block ein neues if / else rein:

if (Bedingung) {
    // ...
} else {
    if (Bedingung2) {
        // ...
    } else {
        // ...
    }
}

Nun kommt ein weiterer Punkt, der wichtig ist: Statt so Blöcken kann man auch genau einen Befehl schreiben, also sowas wie:

if (Bedingung) doSomething();
else doSomethingElse();

Und damit schauen wir uns das Konstrukt mit dem if else if else noch einmal an: In der ersten else ist nur ein Befehl - dieses if else - also kann man das ohne den Block schreiben:

if (Bedingung) {
    // Hier haben wir den Block / die {} gelassen
} else if (Bedingung2) {
    // ...
} else {
    // ...
}

Durch das Weglassen der geschweiften Klammern haben wir nicht die immer stärkere Verschachtelung und die damit immer tiefere Einrückung.

Hat dies das Verständnis etwas verbessert und ist das deutlich geworden?

 

[Viktor A. Kaiser]

Vielen Dank, für diese ausführliche Erklärung.
Und ja, das Prinzip mit dem else if ist nun wirklich deutlicher geworden. Danke vielmals dafür.