MVVM Umsetzung mit WinForms

@Amelie

Versuch einfach ein adneres Framework einzustellen. Im Projektmappenexplorer rechtklick auf das Projekt(das mit dem VB Icon in der Mappe), dort dann auf Eigenschaften klicken. Schau dann nach "Zielframework" und stell da was ein. Kannst natürlich mit einem Doppelklick die Datei öffnen, dort musst du aber selbst den Text bearbeiten. Im Zweifel die UI nutzen an die du wie gesagt kommst.

Welches Studio?
Wenn du ein neues Projekt anlegst, kannst du denn da NET auswählen? Dazu musst du aber ein NET Projekt auswählen kein NET-Framework.
Wenn du kein NET(ohne Framework) ProjektTemplate findest, ist dein Studio evtl. zu alt.

Amelie schrieb:

Und wie es scheint, kann VS2017 wohl nur .Net Consolen Anwendungen

Quatsch.
Das hast du sicher die falsche Einstellung erwischt.
Es gibt z.B. die Projektvorlage Windows-Forms-App (.NET Framework).

Oder du hast dein Studio so kastriert installiert, dass alles abgewählt wurde.
Dann starte den Visual Studio Installer und lade den gewünschten Rest nach.

Amelie schrieb:

Wenn ich das Projekt lade, kann ich kein anderes FW auswählen. Ist alles leer!

vmtl. hast du gar kein .Net SDK installiert.
Hier scheints die zu geben: dotnet.microsoft.com/en-us/dow…dio-sdks?cid=getdotnetsdk
Du könntest es mit .net.9 versuchen

Amelie schrieb:

Unter anderem konnte ich keine eigenen Usercontrols auf die Form ziehen.

Installier dir einfach mal das aktuelle Studio -> Visual Studio Community 2022, da funktioniert das mit den Usercontrols. Ich versteh nicht warum du immer noch an den alten Studios festhälst. Fehlt was, startest du den "Installer", da kannste dann alles was du brauchst installieren, auch im Nachhinein.

.NET Core (das ist .NET bis 3.1, Du brauchst mindestens 7.0) ist genauso falsch wie .NET Standard. Schau mal bei Windows Desktop rein, vielleicht ist es dort.

Dein Bild enthält persönliche Daten. Entfern es lieber, wir glauben Dir auch so, dass Du VS22 installierst.

Amelie schrieb:

Obwohl ich .Net 8 installiert hatte war VS2017 nicht dazu zu bewegen etwas derartiges zu zeigen.

Macht Sinn.
Hier steht:

​.NET SDK-Version 8: Visual Studio 2022 Version 17.8 oder höher

Amelie schrieb:

Aufjedenfall, muss ich da sehr viel umdenken, weil alles so anders ist..

Wie geschrieben, fast alles wäre mit .NET-Framework auch schon möglich gewesen. Dass Du bisher DataBinding noch nicht so genutzt hast, liegt schonmal nicht an .NET. Sondern wahrscheinlich eher daran, dass Du den Thread WinForms Projektentwicklung bzgl. der Rolle des Formulars vielleicht gelesen, aber womöglich noch nicht die einzelnen Stufenprojekte ausprobiert und nachvollzogen hattest
Aber macht nix. Dann eben jetzt.

Anbei die Korrektur.

Amelie schrieb:

Das kommt mir so "doppeltgemoppelt" vor???

Das ist aber tatsächlich notwendig. Für ein korrektes DataBinding benötigen Auflistungen eine eigene BindingSource (BS). Allein schon für die BS-Eigenschaften Current und Position. Wie sonst sollte eine Property, die aus mehreren Einträgen besteht (Drives als ObservableCollection) so abgebildet werden, dass der ausgewählte Wert nachvollziehbar ist?
Weil ich grad beim Thema bin: Ich habe bei der ComboBox noch die Eigenschaft ausgewählter Wert festgelegt. Das erspart Dir den Code, dem Du im MainForm-CodeBehind hattest. Da soll nämlich gar nix stehen
Das Ärgerliche, was ich auch nochmal @loeffel schreiben werde: Der Designer unterstützt bisher keine Direktauswahl der DataBinding-Eigenschaften, ich musste es in die FrmMain.Designer.vb manuell eintragen

Ansonsten habe ich die VmMain.vb noch etwas gekürzt, weil Du die ausführliche Schreibweise der Propertys nicht brauchst. Die einzeilige AutoProperty-Schreibweise reicht, solange Du nix spezielles in den Gettern/Settern machst.

Ich schrieb ja. Nimm sie zur Kenntnis, aber anfänglich solltest Du sie ignorieren. Aber bitte: Die RelayCommand-Klasse ist eine allgemeine Klasse, die das ICommand-Interface implementiert. Damit wird sie für das Button-DataBinding an die Command-Property zugänglich. Eine VmMain-Property, die eine RelayCommand-Klasseninstanz ist, kann also an die Command-Property eines Button gebunden werden. Klar soweit? Also DataBinding geht grundsätzlich nur mit Propertys P und das DataBinding an die Button-Command-Property geht zusätzlich nur, wenn die Property P das ICommand-Interface implementiert. Da im Code steht Property Blablabla As New RelayCommand, sind beide Bedingungen erfüllt: Property und ICommand-Implementierung.

Was die RelayCommand-Klasse jetzt macht: Dem Konstruktor gibt man die Adresse einer Methode, die dann ausgeführt werden soll, wenn die Execute-Methode des RelayCommands aufgerufen wird. Wann passiert das? Wenn der RelayCommand an einen Button gebunden ist und der Button geklickt wird. Also quasi: »Ich erstelle mir eine VmMain-Property SaveDataRelayCommand vom Typ RelayCommand. Diese Property soll sich mal die Aktion SaveData merken. Wenn der Button, an den ich SaveDataRelayCommand binde, geklickt wird, kümmert sich .NET darum, dass dann automatisch die hinterlegte Aktion SaveData aufgerufen wird.«

Belassen wir es erstmal dabei. Wenn das sitzt, können wir uns um den Rest der Klasse kümmern. Dein Hauptaugenmerk sollte darin liegen, das Form codefrei zu halten und all Deinen Code in der VmMain-Klasse zu belassen.
Der spätere Schritt ist natürlich, dass die VmMain-Klasse (ist ja das ViewModel) nicht alles weiß, sondern dass die Modelklassen hinzukommen. Das ViewModel vermittelt ja nur zwischen GUI/View und Modelklassen.

##########

Was meinst Du mit »Controls formatieren«? Das DataBinding einzurichten?

Amelie schrieb:

Die Textbox ist ja auf dem Form. Da fehlt mir der Ansatz, wie das nun in die VmMain kommt.

Die TextBox gar nicht. Was Du in VmMain haben wirst und willst: Den Text der TextBox. Du erstellst Dir ne String-Property im VmMain und bindest im Designer die TextProperty der TextBox an die String-Property. Fertig. Den Leave-EventHandler brauchst Du nicht. Denn was passiert in dem EventHadler bisher? »Wenn die TextBox verlassen wird, prüfe die Daten.« Und da sind wir bei dem, was ich in Post#20 geschrieben habe. Wenn die Daten einer TextBox überprüft werden sollen, wenn sie verlassen und damit validiert werden, muss das Binding nicht extra verändert werden, da das StandardBinding besagt: »aktualisiere die DataBinding-Property OnValidation«. Das bedeutet: Wenn eine Validierung durchgeführt der TextBox-TextProperty durchgeführt wird (nämlich beim Verlassen der TextBox), dann erst aktualisiert .NET die String-Property in VmMain. Und wie kannst Du jetzt die eingegebenen Daten in VmMain prüfen? Indem Du die Datenprüfung im Setter der gebundenen String-Property machst. Also gibt es nicht mehr Code zu schreiben als bisher, sondern es ist nur eine Verschiebung von Form-CodeBehind nach VmMain.
rudimentäres Beispiel innerhalb von VmMain:


Zum Rest schreib ich später noch was.

##########

Amelie schrieb:

Naja und wie ich glaube schon erwähnte, bekommt man gefühlt nun noch mehr Code. Frage mich ob das wirklich bei jedem Projekt Sinn macht?

Das ist immer die Frage. Wenn Du ein Programm hast, was auf eine Bildschirmseite passt, wäre es nicht sinnvoll, MVVM anzuwenden. Je größer das Projekt, dest mehr ergibt es Sinn, mithilfe von MVVM oder anderen Strukturdesignpatterns das Programm auzuteilen bzw. zu gestalten, um u.a. die Übersicht zu behalten. Wenn Du alles in die Formklasse packst (Stichwort Klumpenform/Monolith), dann ist das zwar alles in einer Datei. Aber Du hast ggf. 50000 Codezeilen. Das wird's übel mit der Übersicht. Aber ja, je mehr Klassen beteiligt sind, desto mehr müssen die auch miteinander kommunizieren. Aber das ist ja auch OOP.

DataSet und MVVM gehen bei mir nicht zusammen. Entweder tDS oder MVVM. Ja, richtig gelesen.

Blättern von Daten? Ich versteh nicht, wo das Problem ist. Dass die Daten quasi gefiltert werden? In VmMain sind die Daten verfügbar. Und wenn das GUI sagt: Ich brauch andere Daten, weil der User jetzt nen anderen Monat sehen will, geht der Befehl an VmMain und VmMain kümmert sich darum, dass die gewüschten Daten dem GUI zur Verfügung gestellt werden. So läuft das immer: User verwendet GUI, GUI meldet Wünsche ans ViewModel, ViewModel meldet die gewünschten Daten ans GUI, GUI stellt die Daten dar. Das ist das Grundprinzip von MVVM.

Amelie schrieb:

Wann ziehe ich z.B. ein DGV nicht einfach aus den "Datenquellen" auf das Form, mache den 2-Zeiler zum speichern. Ein paar Subs zum formatieren, Fehler abfangen und Validierung und gut ist.

Tschuldigung, aber Du wolltest MVVM kennenlernen. Wenn doch wieder das GUI Aufgaben übernehmen soll, die nicht zum Aufgabenbereich des GUIs gehören (und genau das ist der springende Punkt!), dann hat das nix mehr mit MVVM zu tun. Das GUI hat die Aufgabe, Daten darzustellen und Benutzereingaben weiterzugeben. Sonst gar nix. Dass man es vor allem als Programmieranfänger (ich zähl Dich nicht mehr dazu) so macht, dass das Form alles kann/macht/weiß, ist der Anfang vom Ende eines komplexen Projektes. Stichwort Spaghetticode.