Amelie schrieb:

3 Etagen mit Lastenauswertung

Was ist eine "Lastenauswertung"?

Amelie schrieb:

Wenn Alles fertig ist, soll das alles automatisiert ablaufen.

Was alles soll ablaufen?

Amelie schrieb:

darf der Lift nicht fahren.

Hat der Lift eine Waage?

Ich denke, das Ganze soll nur eine Animation sein, ja? Sprich, eine PictureBox bewegt sich vertikal auf der GUI hin und her. Höhe proportional zur Etage. Ich denke nicht, dass @'Amelie' einen Motor ansteuert.

Die "Lastenauswertung" – also eine Simulation, dass zu viele Personen eingestiegen sind – würde ich dann mit einer TextBox simulieren, das heißt du schreibst eine Zahl rein und dann


Und das Ganze möglichst mit Klassen, da sie sich im Bereich Klassen weiterentwickeln will.

Wenn der Lift von x nach y gefahren ist, dass sich die Türen allein öffnen und wieder schließen ...

Übersetz diesen Satz in Code:
Wenn PictureBox an einer bestimmten Höhe angekommen, dann Tür auf. Tür nach Zeitraum x schließen. Weiterfahren.

Bartosz schrieb:

VB.NET-Quellcode

1. Debug.WriteLine("zu viel")

Was soll das?

@RodFromGermany habs geändert. Wollte ausdrücken, dass der Fahrstuhl dann nicht fährt

Bartosz schrieb:

habs geändert.

Der Vorschlag ist akzeptabel.
Auf jeden Fall sollte dort noch ein Return drin stehen.

@Amelie: Nein, dem Code kann ich überhaupt nicht zustimmen.
Class Haus hat bei Dir eine Property StockwerkStr und StockwerkInt. Dass man die Datentypen nicht in den Variablentypen einbauen sollte, ist eine Sache (obwohl ich das bei Forms und Controls selbst mache). Aber was sollen diese Propertys bedeuten? Die Frage wird bei weiterer Recherche im Code eher schlecht begründet: Class Kabine: Inherits Haus - das bedeutet, dass Kabine eine spezialisierte Art eines Hauses ist. Bitte was? Eine Kabine ist kein Haus. Und keiner kommt mir hier jetzt mit The Cabin in the Woods! Eine Kabine ist ein Raum und kein Haus. Daher ist Class Kabine: Inherits Haus erstmal Unfug, es sei denn, Du lieferst eine sehr gute Begründung.

Die Kabinenerstellmethode ist m.E. komplett in die falsche Richtung abgebogen. Da eine PicBox einzubauen gehört für mich zumindest nicht in die Klasse. Eine Kabine in der Realität kennt keine PicBox. Vielleicht hat ne Kabine ein Gemälde an der Wand, dafür wär ne PicBox akzeptabel. Aber das ist bei Dir nicht der Fall. Auch der Zugriff auf's Form Frm_Aufzug hat da überhaupt nix verloren. Du willst eine Kabine von ihren Eigenschaften her modellieren. Dass diese Kabine dabei vom Form als PicBox dargestellt wird, muss der Kabine vollkommen egal sein! Wenn diese Kabinenklasse warumauchimmer in einem Konsolenprogramm verschoben wird, muss sie genauso funktionieren wie in einem WinForms- oder WPF-Programm. Das würde aber bei Deiner Kabinenklasse aufgrund von PicBox und Formbezug überhaupt nicht klappen. Eine Modellklasse darf ihre Darstellung auf dem Bildschirm (normalerweise) nicht kennen.

Die Property MaxGewicht ergibt so auch keinen Sinn. Du schreibst rein: Wenn der Wert > 100 ist, dann gilt die Kabine als überladen. Das ist falsch. MaxGewicht ist das maximal für diesen Lift von Dir festgelegte, zulässige Gewicht. Dein Code sagt: Wenn das maximal zulässige Gewicht > 100 (Einheit?) ist, dann ist das Ding überladen. Warum? Es muss doch gelten: Wenn das aktuelle Gewicht größer als das zulässige Gewicht ist, dann gilt der Lift als überladen.

Class Etagen - Eine Klasse sollte genau ein Objekt beschreiben, nicht mehrere. Aber Du beschreibst eine Etagenklasse, die mehrere Etagen (auch noch als Integer) und PicBox-Türen hat. Das ergibt für mich auch keinen Sinn. Wofür die Methoden PruefeEtagenaster() und EtagenTuerAufZu() erschließt sich mir aufgrund der Namen auch nicht.

Das erstmal grob für den Anfang. Mit diesen Klassen wird das meines Erachtens nix.

eine Klasse mit den "GewichtEinheiten" wäre sinnvoll, dann wer/was (Mensche;Hund Katze;Vogel...)
steigt.

Ein Mensch wiegt z.b. 80,6Kg
ein Wellensittich wiegt 22gramm
ein Wellensittich Käfig wiegt ....

Klasse Etage
Klasse Aus/Einsteigen
......

für Gewichte


viel Erfolg mit deinem Projekt

Amelie schrieb:

Irgendwie muss ich ja mit etwas simulieren was die Kabine abbildet. Was soll ich denn da benutzen? […] Packe ich eine Picbox gleich auf die Form und greife dann von der Klasse aus darauf zu ist es auch falsch.

Die anderen dürfen mich gern korrigieren, wenn ich falsch liege. Korrekt ist es, wenn das Form eine PicBox hat und diese PicBox aufgrund der Daten der Kabinenklasseninstanz ändert. Das heißt, das Form holt sich die Daten von der Kabine und ändert daraufhin die PicBox. DIe Form ändert die PicBox, die Kabinenklasse macht selbst nix an der PicBox.

Amelie schrieb:

PruefeEtagenaster() und EtagenTuerAufZu()
Was ist daran nicht zu verstehen?

Ich kenne das Wort Aster nicht. Was soll das sein? Und dann eben Etagenaster. Sagt mir noch weniger. Was ist das? Und EtagenTuerAufZu enthält kein Verb, um einen Befehl zu beschreiben.

Amelie schrieb:

Das mit den Klassen...

- Diese Wortgruppe taucht in meinem Text nicht auf, von daher weiß ich nicht, worauf Du Dich beziehst.

Amelie schrieb:

Dann gibt es hier eben NULL Vererbungen... Aber irgendwie muss ich ja den Zusammenhang zwischen Haus / Stockwerk usw bringen.

Ein Haus hat Stockwerke, ja, das ist was, was EdR als Beziehung 3. Art bezeichnet hat: Hat-Viele

Amelie schrieb:

Naja ich lass es wohl sein.

Da ich Kritik an Deinem Code offensichtlich nicht so formulieren kann, ohne dass es für Dich destruktiv rüberkommt, bin ich aus diesem Thread wohl besser raus.

Amelie schrieb:

Es hieß doch das Form soll NICHT in den Klassen rumgrabschen?

Richtig, aber ich hab wohl damals nicht klar gemacht, was das bedeutet. Das Form soll keine Daten Deiner Klassen direkt ändern. Also Formcode wie Kabine.AktlGewicht = 120 ist tabu. Aber die Kabine anweisen, dass sie selbst ihre Werte ändert, ist ok: Kabine.SetzeAktuellesGesamtgewichtAuf(120). Das Daten-Holen ist auch ok

Amelie schrieb:

Also die Klasse ist der Bauplan. Dann baue ich Objekte in der Klasse und lasse dieses Objekt auf dem Form anzeigen.
Dann weise ich diesen Objekten, in der Klasse Methoden zu.
In meinem Fall das die Kabine Auf oder Ab fahren soll.
Das Form bekommt von der Methode aus der Klasse gesagt (in meinem Fall) zeige die PicBox nun an der Position yx...
Alles falsch??

Nö, meines Erachtens fast richtig. Nur die letzte Aussage muss eben auf das o.g. geändert werden: Das Form bekommt von der Klasse entweder über Events gesagt, wie der Kabinenstatus ist. Oder das Form holt sich die Kabinendaten, z.B. über Kabinen-Functions. Aber die Kabine ändert weder was am Form noch an der Form-PicBox direkt.

@Amelie:
Falls der Unterschied nicht klar ist, ein Beispiel aus der Realität (möge es Dir was bringen): Giro-Bankkonto und dessen Eigentümer. Der Eigentümer darf den Kontostand abfragen. Er darf Überweisungen, Einzahlungen und Auszahlungen tätigen. Dazu nutzt er Funktionen des Kontos, indem er mit der Bank kommuniziert oder einen Geldautomaten oder Online-Banking nutzt. Wenn der Eigentümer 20000 € Guthaben hat (Girokonto-Guthaben = 20000) und er 30000 € abheben will, wird das zurecht verweigert. Würde er direkt Einfluss auf die Daten des Kontos haben, würde er ja festlegen können: Girokonto-Guthaben = -10000, mein Bargeld +30000. Er würde in die internen Daten des Kontos reingrapschen und die Daten ändern. Das kann er aber nicht. Und das ist auch gut so. Er kann nur die Girokonto-Methoden nutzen, um das zu bekommen, was er will: ZeigeKontostand, ZahleBetragInEuroEin(500), HebeBetragInEuroAb(10000). Und wenn das Konto bei irgendeinem der Vorgänge die Ausführung verweigert (HebeBetragInEuroAb(30000)), dann kann der Eigentümer nix dagegen tun.
Aber auch das Konto sollte nix von seinem Eigentümer wissen oder diesen gar manipulieren. Das Konto ändert weder was am Barbestand des Eigentümers, noch sonst irgendwelche Daten. Wenn der Eigentümer eine Auszahlung veranlasst, muss er das Geld aus dem Geldautomat nicht mitnehmen. Er kann es stecken lassen (dann wird es irgendwann wieder eingezogen) oder er erlaubt, dass andere das Geld nehmen. Das ist Entscheidung des Eigentümers, was er mit dem Geld macht, nicht die Entscheidung des Kontos. Das Konto ändert nicht die Daten des Eigentümers.
Selbst wenn beim Konto die Adressdaten hinterlegt sind, hat das Konto kein Recht diese Daten zu ändern und damit den Eigentümer zu einem Umzug in die nächste Stadt zu zwingen.

Zu Deinem Projekt zurück: Das Form ist dabei der Eigentümer (in dem Fall Eigentümer der Kabinenklasseninstanz, da das eine hat-eine-Beziehung ist, da in Frm_Aufzug steht: Private WithEvents Kabine As Kabine) und die Kabine ist wie das Konto. Der Eigentümer kann mit der Kabine vieles machen, was die Kabine gewollt zur Verfügung stellt, aber der Eigentümer sollte nicht alles machen können, was der Eigentümer will, sondern nur, was die Kabine erlaubt. Denn was wär, wenn das Form plötzlich festlegt: Kabine.AktlStandort = -666 'Hölle? und Kabine.AktlGewicht = -1000 'Antischwerkraft?

Aussage 1 (Haus): richtig
Aussage 2 (Etage): richtig
Aussage 3 (Etagenbauplan): richtig; das Haus legt die Reihenfolge der Etagen fest
Aussage 4 (Etage erbt von Haus): falsch; erben bedeutet immer ist-eine-spezialisierte-Form-von. Da eine Etage aber kein Haus ist, darf die Etagenklasse nicht von der Hausklasse erben/abgeleitet sein. Der Code Class Etage: Inherits Haus funktioniert, ist aber inhaltlich falsch.
Gegenbeispiele, die inhaltlich richtig sind:

• Class Bungalow: Inherits Haus
  
• Class Krankenhaus: Inherits Haus
  
• Class Einfamilienhaus: Inherits Haus
  
• Class Wolkenkratzer: Inherits Haus
  

Zur Klarstellung: Enums sind nur Zahlen mit Namen. Nix weiter.

VS und OOP sind für den Lift sehr gute Werkzeuge
Ich mach mal einen einfachen, inhaltlichen Gegenvorschlag:

Du brauchst diesen Teil, um innerhalb des Hauses festzulegen, wieviele Ebenen Dein Haus tatsächlich hat. Nehmen wir meine Hausspezialisierungen:


Die Klasse Haus sollte dann natürlich noch eine Property Aufzug As Aufzug haben, sonst würde es ja gar nicht zu Deinem Projekt passen. Ein Haus ohne Aufzug soll ja gar nicht vorkommen. Aber auch ein Aufzug ohne Zugehörigkeit zu einem Haus kann nicht existieren.
Keiner kommt mir jetzt mit Aufzugsfabriken!

Mir käme dann natürlich eine Umbenennung von Class Haus in Class Gebäude in den Sinn, aber das ist nebensächlich …