Swing Frage zu MVC

[Xenya]

Hallo,
um mich mit der Oberflächenprogrammierung vertraut zu machen, bastel ich Momentan an einem Übungsprojekt. Ich versuche mich dabei am MVC-Modell zu halten.

Ich habe also ein Model, für die Hauptlogik.
Einen Controller für die Kommunikation zwischen View und Model. Der Controller besitzt also eine Instanz des Models, an die er die Eingaben der View weitergibt.
Das Model bearbeitet die Anfragen, sollten Änderungen stattfinden, wird der Controller informiert. Den Controller bekommt er aber nicht über den Konstruktor, sondern es gibt eine addListener Methode, mit der beliebig viele Controller angmeldet werden dürfen.


Meine main erstellt erst das Model.
Anschließend den Controller, der über den Konstruktor das Model bekommt.

Wo ich mir nun nicht sicher bin, ist der Ort an dem die Listener hinzugefügt werden sollen.
Macht man dies im Konstruktor des Controllers, also addListener(this), damit man sich sonst niergends darum kümmern muss.
Oder macht man dies in der Main?
Also

m = new Model()
c = new Controller(m)
m.addListener(c)

Danke

 

[bERt0r]

Normalerweise addest du nicht den Controller als ModelListener sondern die View. Wenn nämlich der Controller nämlich die Änderungen an die View weitergeben muss, ist die Trennung zwischen Controller und View nicht mehr gegeben: Dann braucht jede View genau einen Controller und ist von diesem abhängig.
Es sollte solaufen: Benutzer macht Aktion (View ruft Controller auf). -> Controller ändert Model. -> Model meldet Änderung an View.

 

[hemeroc]

Hi,
sry das ich mich einmische aber ich bin mit einem Teil davon nicht wirklich einverstanden:

"Model meldet Änderung an View."

Es sollte unter keinen Umständen passieren, dass das Model überhaupt Informationen über die View hat vielmehr sollte das Model die Rückmeldung an den Controller geben und der Controller dann der View sagen, dass sie Updaten soll.

Liebe Grüße
Hemeroc

Edit: Beispiel

User klickt im View auf eine neue Datenansicht und ruft eine entsprechende Funktion im Controller auf, der Controller sagt dem Model dass es die Daten laden soll, sobald die Daten da sind wird der Controller davon informiert und der Controller sagt dann der View das Sie die Daten anzeigen soll.
Im besten Fall ist es noch so, dass hier alles über Interfaces kommuniziert wird.

 

[Xenya]

Hallo bERt0r,
danke für deine Antwort.
Wie sieht das dann bei Swing aus?

Habe mal gelesen, dass bei den Swing-Objekte die Unterscheidung zwischen Model-und Controller nicht ganz eindeutig ist. Habe sie deshalb in meinen Controller gesteckt.

Hallo hemeroc,
dafür musst du dich nicht entschuldigen. Ich denke "richtig" gibt es bei MVC nicht und ich freue mich immer, wenn ich verschiedene Meinungen höre. Nur so kommt man weiter und lernt was.




Ich habe mich dazu entschlossen, ein kleines Beispiel zu machen, um mal meine Vorgehensweise zu zeigen. Ich habe auch reingeschrieben, wo meine Fragen sind (beziehungsweise wo die Stellen bei meinem Vorgehen sind, ob mein Vorgehen stimmt ist etwas anderes und ihr könnt mich gerne korrigieren).

Imports habe ich weggelassen, ist eh schon viel genug.
In meiner echten Anwendung sind natürlich die Models und Controller noch Interfaces.

Frage 1:
Die, die ich im Hauptpost gestellt habe. addListener in das Hauptprogramm oder in den Controller
Frage 2:
Wo werdem Entscheidungen gefällt, die eigentlich nur die View betreffen?
Zum Beispiel, wenn nach der Bearbeitung durch einen Klick auf dem Button ein Eingabefeld zurückgesetzt werden soll. Wird dies direkt im Controller gemacht?
Oder, weil es eine Logik ist, in das Modul und das Modul ruft ein resetInput im Controller auf? Nachteil daran: Das Model weiß also, dass es ein Eingabefeld gibt. Was, wenn es in einer "neuen" View irgend wann dieses Feld nichtmehr gibt und somit ein Aufruf der resetInput unnötig ist?

Model:

public class TestModel
{
	Set<TestController> controllers = new HashSet<TestController>();
	
	private StringBuilder word = new StringBuilder();
	
	public void addListener(TestController controller)
	{
		controllers.add(controller);
	}
	
	//Getter und Setter
	public String getWord()
	{
		return word.toString();
	}
	
	//Listener-Methoden die vom Controller aufgerufen werden
	public void addStrings(String newPart)
	{
		word.append(newPart);
		
		for(TestController controller: controllers)
			controller.outputTextChanged();
	}
}

Controller:

public class TestController
{
	private final TestModel model;

	private JFrame frame;
	private JTextField input;
	private JLabel output;

	public TestController(TestModel model)
	{
		this.model = model;
		
		initialize();
		frame.setVisible(true);
		
		//zu Frage 1
		//oder hier?
		model.addListener(this);
	}

	private void initialize()
	{
		frame = new JFrame();
		frame.setBounds(100, 100, 450, 300);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.getContentPane().setLayout(new FlowLayout(FlowLayout.CENTER, 5, 5));
		
		input = new JTextField();
		frame.getContentPane().add(input);
		input.setColumns(10);
		
		JButton buttonAdd = new JButton("add Word");
		buttonAdd.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				clickedAddButton();
			}
		});
		frame.getContentPane().add(buttonAdd);
		
		output = new JLabel();
		frame.getContentPane().add(output);
	}
	
	//Listener die von der View aufgerufen werden, könnte auch direkt oben stehen
	protected void clickedAddButton()
	{
		model.addStrings(input.getText());
		
		//zu Frage 2
		input.setText("");
	}
	
	//Listener die vom Model aufgerufen werden
	public void outputTextChanged()
	{
		output.setText(model.getWord());
	}
}

Die Main, die alles zusammenfügt:

public class TestMain
{
	public static void main(String[] args)
	{
		TestModel model = new TestModel();
		TestController controller = new TestController(model);
		
		//zu Frage 1:
		//an dieser Stelle?
		model.addListener(controller);
	}
}

 

[Wildcard]

IMO wurde diese ganze MVC Geschichte mittlerweile totgekaut und ist stark überschätzt. Das wichtige ist das Model, das ist der Kern von allem.
Das Verbindung zwischen View und Model schafft man am elegantesten mit Databinding und dann muss man sich auch nicht mehr darüber streiten was nun in welche Schicht gehört.
Und nicht vergessen, nichtmal Swing ist MVC. Swing ist mehr DMSMJCVC (Domain Model -> Swing Model -> JComponent -> View -> Controler).
Vielleicht bin ich auch nur SWT/JFace verwöhnt...

 

[bERt0r]

Überleg doch mal, es macht doch gar keinen Sinn wenn das Model den Controller benachrichtigt:
Wenn der Contorller was am Model ändert weis er das doch, wieso wird er dann nochmal benachrichtigt? Da ist doch eine direkte Kommunikation mit der View schneller und auch logischer.

Sämtliche Komponenten in Swing sind so aufgebaut, Beispiel Table:
Im TableModel gibt es die Methode addTableModelListener. Und wer implementiert TableModelListener? JTable, also die View. Jedes mal wenn im TableModel was geändert wird, werden sämtliche TableModelListener abgeklappert und ihnen ein TableModelEvent geschickt.

Wenn du vom Controller aus die View steuern willst, kannst du das ja gleich so machen:

public void benutzerAktion(){
model.changeStuff();
view.updateView();
}

Nur ist mit so einer Konstruktion die Unabhängigkeit von View und Controller nicht mehr gegeben. Du bist auf auf eine View beschränkt Eine Konstruktion wie 1 Model mit 5 Views kannst du so nicht realisieren.

Du kannst dir gerne das beispiel von GUIProgrammierer ansehen, was ich in diese Richtung überarbeitet habe, da wird das ganze anschaulich demonstriert: MVC Beispiel