[MVC] Objekt bearbeiten

[UnkiDunki]

Hi,

ich formuliere das ganze noch mal um:

Ich möchte einen in einer Übersichtstabellen ausgewählten Kunden in einem Dialog bearbeiten...
Jetzt stell sich mir die Frage, wie ich nach dem MVC-Konzept, das Laden bzw. das Setzen der Kundendaten auf die Textfelder realisiere und ein potentielles Speichern meiner Änderungen umsetze...

Bisher habe ich es so, dass ich in der View zwei Methoden habe:

1.) Eine Methode, die sich nach dem Öffnen des Dialoges aus dem übergebenen Model das Kundenobjekt geben lässt und alle Daten dieser Person nacheinander auf die entsprechenden Textfelder setzt. Sprich:

tfName.setText(model.getCustomer()->getName())

und 2.) Nach Klick auf den Save-Button der Controller in der View eine apply-Methode aufruft, die umgekehrt, alle Inhalte der Textfelder zurück aufs Kundenobjekt im Model setzt und danach dieses Objekt mit seiner save-Methode in die Datenbank schreibt.

Frage: Ist das so i.O.? Ich habe nämlich gelesen, dass man jegliche Änderungen in den Textfeldern zB direkt in das Model schreibt. Ist das korrekt oder könnte ich es auf diese Weise auch realisieren?

Bei Punkt 2 wird ja auch der Umweg über den Controller gemacht, was natürlich logisch richtig ist, aber praktisch könnte man ja nach der Eingabe in der View direkt die eigene apply-Methode aufrufen, also wozu eigentlich über den Controller gehen? Nur aufgrund der Struktureinhaltung oder wo habe ich da einen Denkfehler?!?

Vielen Dank für Kritik und Anregungen

 

[UnkiDunki]

Ich hatte das im ersten Versuch ziemlich undeutlich oder verworren geschildert, was ich nun hoffentlich mit der Bearbeitung des Threads ein wenig verbessern konnte, so dass ihr euch meinem Problem bzw. meiner Frage vielleicht noch mal widmen könntet Wäre super.

 

[Michael...]

zu 2. Deine View sollte von der Datenbank eigentlich gar nichts wissen. Die apply Methode gehört also eher in den Controller.
Wenn es darum geht Änderungen in der View im Model nachzuziehen, kann dies auch direkt ohne Umweg über den Controller passieren. Bei MVC ist die Verbindung Model zu View ja bidirektional.

 

[UnkiDunki]

Hi,

meine View weiss aber doch garnichts von der Datenbank, nur vom Model bzw. vom Kundenobjekt in dem Falle...
Die apply-Methode in der View setzt ja nur die Eigenschaften aus den Textfeldern auf die entsprechenden Eigenschaften des Objektes...

Wenn ich die apply-Methode im Controller mache, stellt sich die Frage wie ich auf die ganzen Textfelder, etc in der View zugreifen soll?
Alle public machen oder etwa für jede Componente Getter und Setter? Das wird bei einem gewissen Unfang ja recht viel...

 

[Michael...]

meine View weiss aber doch garnichts von der Datenbank

Sorry, so hatte ich es ursprünglich verstanden.

Die View darf ja durchaus das Model ändern.

Wenn man jetzt strikt nach MVC vorgehen würde, würde bei einem Klick auf einen Save Button z.B. ein ActionListener anspringen (der evtl. direkt vom Controler implementiert wird) der Controler sagt dann zur View (per Methodenaufruf an der Referenzvariablen): Übertrage die Änderungen aus dem Textfeld auf das Model um dann anschliessend die neuen Werte aus dem Model zu erfragen und in die Datenbank zu speichern.

 

[UnkiDunki]

Sorry, so hatte ich es ursprünglich verstanden.

Ne, musst dich nicht entschuldigen. War mein Fehler. Hatte das oben nicht eindeutig genug geschildert.

Wenn man jetzt strikt nach MVC vorgehen würde, würde bei einem Klick auf einen Save Button z.B. ein ActionListener anspringen (der evtl. direkt vom Controler implementiert wird) der Controler sagt dann zur View (per Methodenaufruf an der Referenzvariablen): Übertrage die Änderungen aus dem Textfeld auf das Model um dann anschliessend die neuen Werte aus dem Model zu erfragen und in die Datenbank zu speichern.

So hatte ich es ja auch bei Punkt 2 geschrieben:

und 2.) Nach Klick auf den Save-Button der Controller in der View eine apply-Methode aufruft, die umgekehrt, alle Inhalte der Textfelder zurück aufs Kundenobjekt im Model setzt und danach dieses Objekt mit seiner save-Methode in die Datenbank schreibt.

Mit "und danach dieses Objekt... in die DB schreibt" habe ich den Controller als "ausführendes Organ" gemeint.
Mein Kundenobjekt besitzt bereits eine eigene save-Methode, die Änderungen direkt in die DB schreibt.
Dann müsste es ja mehr oder weniger dann so aussehen:

// methode im controller, aufgerufen durch actionlistener
view.applyChanges();
model.getCustomer().save();

"Ginge" das dann so? Mehr oder weniger MVC-konform?

 

[Michael...]

Gehen tut das schon.

Ich würde allerdings das Model und das Kundenobjekt von der Datenbank unbehelligt lassen, sonst muss man, wenn man später evtl. die DB austauscht oder deren Design ändert die beiden Klassen wieder anpassen.

 

[UnkiDunki]

Ahh ok Guter Einwand. Also die Datenbankaufgaben in den Controller auslagern?
Wobei ich das aber nur teilweise nachvollziehen kann. Das Model hat ja in dem Sinne erst mal nichts mit der DB zu tun, da es nur das aktuelle Kundenobjekt beinhaltet, welches ja für sich die Datenbankaktionen durchführt und ob man jetzt bei Änderungen der DB den Controller anpasst oder die Kundenklasse bleibt doch eigentlich dasselbe. Es ist sogar ratsamer, wie ich finde, das komplett in der Kundenklasse zu lassen, denn so ändert man es bei DB-Änderungen wirklich nur an einer Stelle und nicht bei zig Controllern, falls man an anderer Stelle auch mit Kundenobjekten arbeitet, oder sehe ich das falsch?

Dachte, dass ich das mal geschickt gelöst hätte, dass ich meiner Kundenklasse gleich alle entsprechenden Methoden implementiert habe, also retrieve(kundenID), save() [bei bereits vorhandener kundenID update() sonst insert()]...

Das war ein Fehler?
Danke übrigens für dein Engagement!

 

[Michael...]

Ich würde da eine eigene Klasse verwenden, die mit der DB kommuniziert, die dann von den Controlern (hatte glaube ich noch nie mehr als ein Controler Klasse) verwendet wird.
Aber wenn Kundenobjekt die einzige Klasse ist, die was mit der DB zu tun hat, kann man das auch so machen, man muss ja nicht immer alles streng nach Vorschrift machen.

Danke übrigens für dein Engagement!

Hab gerade keinen Bock auf Arbeiten ;-) und es eh nicht allzu viel los.

 

[UnkiDunki]

Ich würde da eine eigene Klasse verwenden, die mit der DB kommuniziert, die dann von den Controlern (hatte glaube ich noch nie mehr als ein Controler Klasse) verwendet wird.

Ok, klar mit ner extra Klasse, an die Kundenklasse gebunden, würde das auch gehen. Direkt im Controller wäre ja schlecht, da man ja pro View einen Controller haben sollte. Das Model darf man gerne teilen, was ich übrigens tue, da ich vor dem Berarbeitungsdialog eine View mit einer Kundenübersicht als Tabelle habe. Bei Doppelklick auf einen Kunden setze ich 1.) auf das geteilte Model den angeklickten Kunden (ich instanziiere ein Kundenobjekt mit der aus der Tabelle gewonnenen ID

model.setCustomer(new Customer(id))

und öffne dann 2.) den Dialog zum Editieren, wo sich dann die View über das Model den Kunden holt und dann die Textfelder entsprechend belegt (sollte natürlich dann vom Controller initiiert sein das Ganze)... Auch ne Frage, ob das so gemacht wird

Hab gerade keinen Bock auf Arbeiten ;-) und es eh nicht allzu viel los.

Na dann...