Projektbeurteilung Ampelsteuerung

Ja, wenn es funktioniert, ist das Möglichkeit. Logisch. ABER: Das hilft Dir nur bedingt bei Deinem Projektaufbau weiter. Wenn es um die Übung geht, wie man Klassenmethoden aufruft, ok. Aber Du solltest Dein Ziel nicht aus den Augen verlieren. OOP hat auch etwas mit Datenmodellierung zu tun. Ja, ein Form braucht man wohl bei einem WinForms-Projekt. Aber das Form ist m.E. v.a. für folgende Sachen gedacht:

• Benutzerdateneingabe
  
• Anzeige und Darstellung der Modelle (Modelle = eigene Klassen(instanzen))
  
• Weiterleitung der Daten und Benutzereingaben an andere Klassen.
  

Du hast momentan eine LED/einen Lichtkasten, welche(r) direkt vom Form Daten erhält. Wenn Dir bewusst ist, dass da später noch einige Klassen dazwischengeschaltet werden müssen, damit es die Realität sinnvoll(er) abbildet, ist ok. Wenn nicht, dann verrennst Du Dich.

Was ist Ampelphase 0, 1, …? Was haben die Phasen zu bedeuten und wieviele gibt es?

btw:
Case Is = 0 -> Case 0
clLichtkaesten: Der cl-zusatz soll Klasse bedeuten? Sowas sollte nicht verwendet werden. Und wieviele Lichtkästen modelliert eine clLichtkaesten-Instanz? Wenn eine Instanz einen Lichtkasten modelliert, dann ist der Name falsch, da nicht im Singular.

Das Daten-Modellieren ist das Anlegen und (mit Code) Befüllen von eigenen (Modell)Klassen.
Was Du brauchst: Zum einen das umfassendste Objekt, welches Du aus der Realität nachbauen willst. Ich tippe auf eine Kreuzung. Also brauchst Du eine Klasse Kreuzung. Dann überlegst Du Dir, was eine Kreuzung alles für Objekte in Deinem Programm braucht. Ich vermute eine Ansammlung von Ampeln. Also brauchst Du z.B. eine List(Of Ampel). Und dafür natürlich eine Ampel-Klasse. Was brauchst Du noch für die Kreuzung? Einen Taktgeber? Dann also eine Taktgeber-Klasse. Noch was? Weiß ich nicht, ist Dein Programm. Was braucht der Taktgeber? Keine Ahnung, ob der irgendwelche "Bestandteile" (also andere Objekte) hat. Dann als nächstes die Ampel-Klasse: Die hat mehrere Lichtkästen, also auch eine List(Of Lichtkasten). Was hat ein Lichtkasten für Objekte? Keine Ahnung, sag Du es.
Soviel zum Thema erste Schritte bei der Datenmodellierung. Danach kommt das Verhalten der einzelnen Klassen rein, also nötige Klassenvariablen, Properties und Methoden einbauen.

Also grob zum Thema Datenmopdellierung aus meiner Sicht: Das größte Objekt hernehmen und eine Klasse bilden. Schauen, welche Bestandteile existieren und Klassen dafür bilden. Usw.

privates Beispiel: Ich bastel gerade an einem TowerDefense-Spiel. Das größte Objekt ist das Spiel, also hab ich eine Spiel-Klasse. Diese Klasse hat als "Bestandteile": Das Spielfeld, eine Liste von Türmen, eine Liste von Gegnern und eine Gruppe von Schüssen. Also brauch ich als Klassen neben Spiel: Spielfeld, Turm, Gegner (Singular!), Schuss.

Ah, ziemlich konkrete Vorstellungen. Gut! Je präziser man das formulieren kann, desto besser weiß man, was man programmieren muss.

eine Kreuzung -> Du brauchst eine Klasse Kreuzung. Besser wäre aber wohl, wenn die Klasse Kreuzungssteuerung heißt. Denn das Straßenmaterial, das Baujahr der Kreuzung etc. interessiert wahrscheinlich nicht. Es geht ja um die Elektronik. Hauptsrasse und Nebenstraße -> erstmal eine (!) Klasse Straße; sollen sich vom Verhalten Haupt- und Nebenstraße unterscheiden? Wenn ja, was macht den Unterschied aus?
Jede Straße braucht mindestens 2 Ampeln zu je mindestens 2 Lichterkästen -> eine Klasse Ampel und eine Klasse Lichtkasten. Wenn Du schreibst "mindestens" zwei Ampeln, dann braucht jede Straße eine List(Of Ampel), damit dann später "im Betrieb" (z.B. zu Programmstart) das flexibel festgelegt werden kann. Auch braucht die Ampelklasse eine List(Of Lichtkasten), damit Du das ebenfalls flexibel steuern kannst.
Taktgeber und Schalter sollten Teil der Kreuzungssteuerung sein. Ob es dafür je eine Klasse benötigt, kommt auf den Funktionsumfang an. Die Schalter werden wohl durch je eine einfache Sub darstellbar sein. Beim Taktgeber sollte sowas auch ggf. möglich sein. Aber es hindert Dich nicht, eine eigene Klasse dafür zu erstellen.
SteuerForm ist Dein HauptForm. Da werden auch die Anzeigen erscheinen. Da ein Extra-Form wird Dir wahscheinlich keinen Vorteil bringen, aber vielleicht kennst Du einen.

Die Ampelphasen sind ein Fall für ein Enum. Dann kannst Du Deine Bezeichnungen verwenden und musst nicht mit Zahlen hantieren:

Da verteilt der Compiler automatisch ganze Zahlen von 0 an.
Dann kannst Du im Code z.B. schreiben:

Amelie schrieb:

soll die Grünphase der Hauptstr. länger andauern als die der NS.

Wenn das erstmal der einzige Unterschied ist, kann man das über ein Feld innerhalb der Ampelklasse erledigen. Die könntest Du DauerDerGrünphaseInSekunden nennen. Die kann dann über einen Parameter im Konstruktor eingestellt werden, z.B.

Was sind die genauen Aufgaben des Taktgebers? Was soll der machen?

Die Forms sind schmuckes Beiwerk. Du willst Dein Programm ja mit Leben/Logik füllen.

Dann ist der Taktgeber m.E. am besten doch nur ein Timer, den Du auf's Form ziehst. Seine aktuelle Aufgabe und sein Timing bekommt er dann am besten über das Form mitgeteilt, welches den Zustand von der Kreuzungssteuerung holt. Wenn also die Kreuzung wodurch auch immer den Gesamtzustand ändert, z.B. auf Nachtbetrieb oder Störung oder Normalbetrieb, dann kann es das dem Form mitteilen und das Form ändert entsprechend die Taktgeber/Timer-Einstellungen. Später könnte man auch eine Stufe mehr OOP gehen und eine weitere Klasse, für diese Aufgabe erstellen. Aber eines nach dem anderen.

Nicht vergessen: Wenn Du einen Beitrag nachbearbeitest, bekommt das keiner mit. Es sei denn, Du "sprichst ihn an", z.B. so: @Amelie
Aber zum Glück hab ich danach Ausschau gehalten.

Die Enums sind schon ein guter Anfang, gehören aber besser nicht in die Klasse. Ich hab noch ein Enum ergänzt: Betriebsmodus. Denn die Ampelsteuerung kann ja immer nur einen haben. Daher sind die Booleans IstNachtbetrieb und Ist24Stdbetrieb aus meiner Sicht nicht richtig. Da ja nur eines von beiden zutreffen kann, oder?
As Object geht schon mal in die falsche Richtung.
Schau nochmal Post#26 an:

VaporiZed schrieb:

Du brauchst eine Klasse […] Kreuzungssteuerung […] eine (!) Klasse Straße […] eine Klasse Ampel [und] eine Klasse Lichtkasten

Um mehr geht es zu Beginn nicht.
Im Form gibt es nur ein Private ReadOnly Ampelsteuerung As New Ampelsteuerung. Wie die Ampeln laufen und in welchem Betrieb die Ampelsteuerung ist usw. soll nicht DIREKT über das Form festgelegt werden. Da soll sich mal schön die Ampelsteuerungsklasse drum kümmern. Das Form hat erstmal nur die Aufgabe, Befehle (oder doch eher Wünsche?) an die Ampelsteuerung zu schicken. Ob dann was innerhalb der Ampelsteuerung passiert oder was oder gar wie, das entscheidet sich nur in der Ampelsteuerungsklasse, nicht im Form!
Der nächste Schritt wäre m.E. die Erstellung einer einfachen Kreuzung. Das könnte z.B. so aussehen, dass Du eine Friend Sub ErstelleEinfacheKreuzung innerhalb der Ampelsteuerungsklasse anlegst. Und in dieser Sub legst Du dann fest, was für Dich eine einfache Kreuzung ist. Also: der Straßenliste 2 Straßen hinzufügen. Jede Straße fügt dann ihrer Ampelliste 2 Ampeln hinzu und auch eine Ampel mit New angelegt wird, soll diese Ampelinstanz sich selbst 3 Lichtkästen hinzufügen. Frage ausschließlich an Dich*: Wie sähe das alles als Code aus?

* nicht, dass jemand Deinen OOP-Lernenschritt durch Vorsagen eintrübt

VaporiZed schrieb:

Um mehr geht es zu Beginn nicht.

Mir fehlt an diesem Datenmodell die ganze Zeit die Tatsache, dass ein Ampel-System ein Programm durchläuft, üblicherweise zyklisch.
Also es gibt schoma ein TagesProgramm und ein NachtProgramm.
Und jedes Programm hat viele Schritte.
Und in jedem der Schritte werden einige Ampeln beauftragt, umzuschalten.

Amelie schrieb:

Ist bestimmt völlig falsch.

Nana, nicht so selbstüberkritisch.
ErstelleKreuzung ist insofern richtig, dass die Straßen erstellt werden. Allerdings sind die Straßeninstanzen so nur lokal, da sie ja mit Dim innerhalb der Sub erstellt werden. Auch gehst Du die leere Straßenliste durch (Zeile#24 und Zeile#29). Da passiert dann natürlich nix.
Von daher: füge 2 Straßen dieser Straßenliste hinzu. Zum anderen: Die Ampeln sollen nicht in der Kreuzung-Klasse erstellt werden, sondern in der Straßen-Klasse. Und dort sollen sie nach Erstellung der Straßenklassen-Ampelliste hinzugefügt werden. Wenn Du drauf kommst, dann ist gut, ansonsten gibt's einen Vorschlag.

Amelie schrieb:

@ErfinderDesRades
In meiner Nicht OOP-Version hatte ich das von dir erwähnte ja schon drin. Oder verstehe ich dich falsch?

Du hattest gar kein Datenmodell, und folglich gab es da auch keine Klasse, die ein Programm dargestellt hätte oder eine, die einen Programmschritt dargestellt hätte.

Mein Post #33 richtete sich eiglich eher an Vaporized - er versucht ja, ein Datenmodell einzuführen - und da fehlt imo dieser Gesichtspunkt.

Straßenerstellung:


Ampeln:

Mehr erstmal nicht. Ein Problem nach dem anderen.

Das dachte ich mir, dass Du gerade etwas feststeckst. Deswegen habe ich ja auch vorgehabt, es langsam angehen zu lassen und Dich anhand Deines Projektes bei OOP weiterzubringen. Abstrakte Beispiele, die nichts mit Deinem Problem zu tun haben, lassen sich nicht für jeden übertragen. Und manchmal bringen sie einem auch gar nix.
Du hast recht, jetzt sind da jetzt zwei Straßen. Aber was ist Hauptstraße, was ist Nebenstraße? Es wird ja (noch) nicht unterschieden. Das meinte ich auch mit

VaporiZed schrieb:

Ein Problem nach dem anderen.

Gut. Du hast also jetzt 2 gleichberechtigte Straße, willst diese aber doch irgendwie unterscheiden können. (Viel!) später ist das mit Vererbung möglich, jetzt gehen wir erstmal einen leichteren Weg. Du legst eine Property in der Straßenklasse an, die diese Unterscheidung aufnimmt und in der jeweiligen Straßenklasseninstanz hinterlegt/abspeichert. Wir belassen es erstmal bei einem einfachen Fall: Es gibt nur Hauptstraßen und Nebenstraßen. Dann kannst Du diese Property entsprechend benennen: IstHauptstraße. Der Wert kann für eine Straße entweder zutreffen oder nicht. Wenn es nicht zutrifft, ist es erstmal automatisch eine Nebenstraße.
Erste Frage (Grundlagenwissen): Von welchem Datentyp wird also diese neue Property sein, die Du in der Straßenklasse anlegst?
Zweite Frage: Mit welchem Code legst Du somit bei der Erstellung einer Straßenklasseninstanz (also innerhalb der Sub ErstelleKreuzung) für die neue Straße mithilfe der neuen Property fest, ob diese neue Straße eine Hauptstraße wird oder nicht?