Projektbeurteilung Ampelsteuerung

petaod schrieb:

ErfinderDesRades schrieb:

Das verkompliziert aber die Ampeln.

Aber es vereinfacht die Kreuzung.

Ja, muss man sich überlegen, ob dieser Gewinn es wert ist.
Gewinn: Es vereinfacht die SteuerungsLogik der Kreuzung ein bischen. Kompliziert bleibt die Kreuzung dennoch.
Kosten: Ampel (die zuvor ganz "dumm" war) benötigt nun auch SteuerungsLogik. Kreuzung muss die neue Ampelsteuerung antriggern.

quasi eine grosse State-Machine die eine kleine ansteuert.
Aber für OOP ganz dolle: Vielleicht kann man das zum Anlass nehmen, eine State-Machine-Base zu schaffen, die die Gemeinsamkeiten der beiden implementiert (ja, warn Witz )

Gonger96 schrieb:

...StateMachine...

Joah - vom inhaltlichen her sagte ich das bereits in Post#33

Entweder ist deine C# Version oder VS Version zu klein. Warum benutzt du 2022 noch VS 2013? Das ist auch eine Express Version. Ich selbst hab 2017 Commiunity benutzt. Läuft sogar auf meinem alten Tablet

Aufgrund Deines zuletzt geposteten Codes bekommt die Variable AktuelleAmpelphase einen neuen Wert. Aber es ist nirgends ersichtlich, dass die Ampel-Instanzen angewiesen werden, diese Phase zu übernehmen oder selbständig ihre eigene Phase zu ändern. Oder hast Du Code (verschwiegen), der die Ampeln zum Phasenwechsel bringt?

Dann wieder in kleineren Schritten. Was hast Du bisher an Code und was ist Dein nächstes Vorhaben?

Stop. Du bist bereits seit einiger Zeit woanders abgebogen. Da werd ich Dich jetzt abholen.

VaporiZed schrieb:

Die Kreuzung sollte den Ampeln nur die Anweisung geben, dass sie in ihre nächste Phase schalten sollen. Die Kreuzung sollte diese Phasen nicht selber festlegen. Wozu auch? Die Reihenfolge ist ja durch das Enum quasi festgelegt.

petaod schrieb:

Wie [die Ampeln] vom einen Zustand in den anderen kommen, müssen sie selbst wissen. Wenn sie grün sind und den Stop-Befehl bekommen, müssen sie 3s gelb anzeigen und danach auf rot.

Dementsprechend ist Deine SchalteAmpelphase genau so, wie es nicht sein sollte, denn sie zählt selber und legt Phasen fest. Aber die Ampel-Instanzen werden nicht geändert. Die aktuellen Phasen der Ampel bleiben wie sie sind. Jede Ampel hat eine aktuelle Phase. Aber die wird durch Deine Sub nicht geändert. Die Ampeln merken von der Umstellung also gar nix.

In der Sub sollte eigentlich nur stehen:
Die Logik dahinter sollte aufgrund des hier gezeigten Codes klar sein: Die Kreuzung weist die Ampel, die gerade dran ist, an, sich selbständig in die nächste Phase zu schalten. Also z.B.: eine rote Ampel soll rotgelb werden, oder eine gelbe Ampel auf rot umschalten. In welcher Phase die aktuell zu schaltende Ampel ist, kann der Kreuzung erstmal egal sein. Danach fragt die Kreuzung in Ampel, ob sie "fertig" ist und wenn sie es ist, kümmert sich die Kreuzung darum, dass beim nächsten Aufruf von SchalteAmpelphase die nächste Ampel ihre Phase ändern, während die bisherige in der Phase bleibt, wo sie ist.

Damit der Code läuft, musst Du Dich natürlich um die Details kümmern. Dazu musst Du innerhalb der Kreuzung eine Private Variable AktuellUmzuschaltendeAmpel anlegen. Welchen Datentyp hat diese Variable?
Dann brauchst Du eine Methode namens GetNächsteAmpel. Ist das eine Sub oder eine Function? Wenn es eine Function ist, welchen Datentyp liefert sie zurück?
Die Ampelklasse braucht 2 Methoden: SchalteInNächstePhase und IstFertig. Sind das beides Subs oder Functions oder eines das und eines das und wenn es Functions gibt, welchen Rückgabedatentyp haben diese?
Du musst noch keinen Code innerhalb der Methoden schreiben, bitte erstmal nur die Methodenrümpfe, also z.B. Friend Sub … und End Sub, keinen Code dazwischen. Denn wenn der Anfang nicht passt, ergibt der Code in der Methode keinen Sinn.