Projektbeurteilung Ampelsteuerung

Genau. Deine Sub New in der Klasse Straße hat keine Parameter. Damit weißt Du, dass die Sub New der Klasse (der sogenannte Konstruktor) eine mehr oder weniger normale Methode einer Klasse ist, die ggf. Parameter aufnehmen kann. Das ermöglicht Dir auf eine weitere Weise festzulegen, welchen Straßentyp eine neue Straße hat. Indem Du die Information über den Konstruktor mitgibst. Du kannst auch mehrere Konstruktoren schreiben. Das nennt sich Überladung. Das Prinzip kann man immer wieder nutzen. Der Name einer Methode darf gleich bleiben, nur die Parameter müssen anders sein. In Datentyp oder Anzahl:

Und jetzt wird folgender Code in der Kreuzungsklasse auch funktionieren:


Weil ein Parameter bei der 2. Straße mitgegeben wird, weiß der Compiler automatisch, dass er den 2. Konstruktor aufrufen muss. Ein anderer Parametertyp, also z.B. wenn Du schreibst Straßen.Add(New Straße("Berliner Allee")) würde zum Problem führen, da der Compiler keinen passenden Konstruktor in der Klasse finden kann.

Damit weißt Du also, wie man eine Property einer Klasseninstanz auf 4 Arten festlegen kann:

1. von außen über den Zugriff auf die Property: NeueStraße.TatsächlicherStraßentyp = Straßentyp.Hauptstraße
   
2. über das Festlegen bei der Instanziierung: New Straße With {.TatsächlicherStraßentyp = Straßentyp.Hauptstraße}
   
3. über eine normale Klassenmethode: SetzeStraßentyp(), siehe Post#48
   
4. über einen geeigneten Konstruktor
   

Punkt 1 und 2 haben die Besonderheit, dass die Ziel-Property von außen erreichbar sein muss, die Property muss also mit Public oder Friend deklariert sein.
Punkt#3 hat die Besonderheit, dass der Straßentyp jederzeit von außen geändert werden kann. Obwohl das hier bei einem Straßentyp nicht sinnvoll ist.
Punkt 4 legt die Property von Anfang an fest.
Behalte alle Möglichkeiten in Erinnerung, Du wirst die verschiedenen Möglichkeiten immer wieder brauchen.

Falls Du bis dahin zu den bisherigen Grundlagen keine Fragen hast, bauen wir die Kreuzungsklasse etwas aus. Der User soll über einen Button auf dem Form den Arbeitsmodus der Kreuzungsanlage festlegen können. Das Form ruft dazu eine (noch nicht von Dir erstellte) Sub der Klasse Kreuzung auf. Erstelle und benenne mal jene Sub und lege fest, ob und wenn ja welche(n) Parameter sie zum sinnvollen Betriebswechsel braucht.

VaporiZed schrieb:

Das Form ruft dazu eine (noch nicht von Dir erstellte) Sub der Klasse Kreuzung auf.

Das hast Du damit leider aber nicht umgesetzt. Das Form soll hier nur die Benutzereingaben an die Klasse Kreuzung weiterleiten. Keine Entscheidungen treffen. Und mit Strings fangen wir für Arbeitsmodi gar nicht erst an. Du hast ja selber schon ein weiteres Enum eingeführt: Betriebsmodus. Nutze es. Aber auch erst, sobald das Enum außerhalb der Formklasse ist! Keine Schachtelung!
Was noch dazukommt: AktuellerBetriebsmodus ist momentan Teil der Formklasse. Das bedeutet, Du triffst eine Aussage über den Betriebsmodus des Formulars! Du willst aber eine Aussage über den Betriebsmodus der Kreuzungssteuerung festlegen. Also gehört AktuellerBetriebsmodus nicht in die Formklasse, sondern in …?

kleiner Tipp bezüglich Benutzer -> Form -> Kreuzung: Wie hat denn der Benutzer bei Deiner Nicht-OOP-Version dem Programm mitgeteil, in welchem Modus sich die Kreuzung befinden soll? Den fast gleichen Weg musst Du auch jetzt gehen.

Sobald Du Dich über Friend, Public und Co. informiert hast und die Antwort auf die Fehlermeldung kennst und verstanden hast, kann es weitergehen. Solltest Du trotz Deiner Nachforschungen nicht weiterkommen, gib bescheid. Diese sogenannten Zugriffsmodifizierer sind für OOP wichtig und deren Bedeutung sollte klar sein.
Ansonsten bitte noch die Fragen aus Post#64 verarbeiten und beantworten.

Die Frage, wie der Benutzer der Kreuzungsklasse mitteilt, in welchem Zustand sie sein soll, hast Du zwar nicht mit Worten, aber mit Code beantwortet: Der Benutzer entscheidet durch einen Klick auf einen Button. Gut.
Nun bist Du noch etwas zu sehr in Formentscheidungscode verwickelt. Das Form soll gar nix entscheiden. Das ist die Aufgabe der Kreuzungsklasse. Daher: Sobald Du ein If im Form schreiben willst, kannst Du davon ausgehen, dass Du in der falschen Klasse bist. Zumindest, wenn nun änderungen an einer eigenen Klasse vorgenommen werden sollen.
Verfrachte also die If-Geschichte der Modusumschaltung von btn_anlage_ein_Click in die Kreuzungsklasse.
In btn_anlage_ein_Click darf erstmal nur folgendes stehen:

Mehr nicht. Das Form entscheidet gar nichts und soll auch keine Details der Kreuzungsklasse kennen. Das sind alles Aufgaben der Kreuzungsklasse. Je mehr das Form weiß und entscheiden soll, desto zentraler wird die Bedeutung des Forms. Und wenn dann das Form irgendwann alles wieder kann/weiß/tut, tja, dann bist Du wieder bei Deiner Nicht-OOP-Version.
Durch die genannte Änderung des Button-Click-Codes ergeben sich einige neue Punkte: Du brauchst in der Kreuzungsklasse eine zusätzliche Sub namens AnlagemodusUmschalten, die keinen Parameter hat. In der soll das passieren, was Du jetzt noch im Button-Code hast: Umschaltung des Betriebsmodus zwischen Normalbetrieb und Aus. Nichts anderes! Der Benutzer soll mit diesem Umschaltebutton nichts anderes können. So habe ich den Code und das frühere Nicht-OOP-Projekt verstanden. Daher ergibt es keinen Sinn, da mehr einzubauen.
Dann brauchst Du in der Kreuzungsklasse eine Function namens HoleBezeichungDesAktuellenBetriebsmodus. Diese Methode soll Dir nur einen String wie "Anlage ein", "Anlage aus" wiedergeben, abhängig vom Betriebsmodus.

Schau mal Deinen Kreuzungsklasse-AnlageEinschalten-Code an: Den kannst Du auf das hier reduzieren: AktuellerBetriebsmodus = Betrieb

Thema Benennung:

• TxTAnzeigen - warum TxT und nicht Text? Und: werd konkret. Welcher Text solll angezeigt werden? Benenne die Sub entsprechend.
  
• btn_anlage_ein: Das suggeriert, dass man damit die Anlage einschalten kann und nichts anderes. Das stimmt aber nicht. Man kann sie auch ausschalten.
  
• Kreuzungsklasse: AnlageEinschalten -> AnlageUmschalten
  

btw: ByVal kannst Du immer weglassen

Das ganze Select Case in AnlageUmschalten kann weg.
Das hier reicht:

Bei getAnlagenmodus kannst Du gerne Dein Select Case reinmachen. Da brauchst Du es.
AnlageEinAusschalten passt erstmal.
Nun können wir zu Störung kommen. Unter welchen Bedingungen soll die Kreuzung in den Störungsmodus gehen?

#########

(Formfehler korrigiert)

Korrekt. Die Anzeige brauchst Du auch noch später bei anderen Modi (Stöung, Nachtbetrieb).
Das musst Du jetzt für Dich festlegen, wann Dein Programm die Störung erzeugt oder auslöst.

Stop! In der Kreuzungsklasse wird nix am Form geändert. Das macht das Form schön selbst.

Vom Prinzip her ist das jetzt richtig.
Wenn es bis jetzt so funktioniert wie erwartet, ist alles gut. Dann kann es ja weitergehen. Bist Du eigentlich zu einem Ergebnis bei den Zugriffsmodifizierern gekommen? Alles klar jetzt?

Amelie schrieb:

Ja es funktioniert soweit, also kann man das so machen?

Aufbautechnisch passt das soweit. Klassen grapschen sich jetzt nicht gegenseitig ins Getriebe, sondern bitten um Ausführung von Anweisungen (Subs) bzw. bitten andere Klassen um Informationen (Functions). So soll es sein.

Amelie schrieb:

Zugriffsmodifizierer... Hatte ich eine für mich einigermaßen plausible Erklärung gepostet.

Naja, nach komplett verstanden klang das nicht. Nochmal kurz die drei für Dich momentan relevanten:
Public: alle Programme und Programmkomponenten (nicht nur der eigenen Code, sondern auch Fremdprogramme) können auf das mit Public Deklarierte zugreifen
Friend: alle Programmkomponenten des eigenen Projektes können auf den Codeteil zugreifen
Private: nur die Klasse, in der der Codeteil deklariert wird, kann darauf zugreifen.
Eine Public Sub AnlageUmschalten(Betrieb As Betriebsmodus) wirft automatisch einen Fehler, weil AnlageUmschalten zwar für andere Programme zugänglich ist, Betriebsmodus aber nur mit Friend deklariert wurde. Daher kann ein externes Programm die Sub nicht aufrufen, weil sie Betriebsmodus nicht kennt. Daher sagt der Compiler: Das geht so nicht.

Störung: Ok, wenn Du das nur als Button drinhaben willst, belassen wir es dabei. Vielleicht fällt Dir ja irgendwann noch eine Situation ein, bei dem noch eine Störung auftreten kann.

Dann kommen wir zu den Ampelphasenumschaltungen. Noch nicht mit Takt, sondern erstmal manuell. Der Taktgeber sollte aber bald folgen.
Wie würde eine Sub für eine manuelle Umschaltung einer Lichtanlage für Autos aussehen? Jeder Sub-Aufruf soll die Ampel in ihre nächste reguläre Phase bringen. Belassen wir es erstmal bei der Hauptstraßenampel. Da stellt sich eigentlich gleich die Frage: Wenn die Kreuzung und die Straßen erstellt werden, mit welcher Phase beginnen die Lichtkästen?

Mit den Ampelphasen wird es jetzt etwas komplizierter. Solange Du nur eine Haupt- und eine Nebenstraße hast, geht das. Aber spätestens wenn eine Straße dazukommt, könnte es vielleicht nicht mehr funktionieren. Kann ich noch nicht sagen.

Amelie schrieb:

Evtl Einen Zähler der bei jedem Buttonklick um eins erhöt wird und dabei aus der ENUM ausliest??

Gut! Denn die Enum-Werte sind ja nur sprechende Namen für Integer-Werte, die bei 0 losgehen. Versuch das mal.