MVC Probleme bei SWING und EVENT

[Fry]

Hallo,

ich probiere gerade etwas mit dem MVC herum, und habe die Basics auch verstanden. Folgender Aufbau meiner vier Klassen, ich beschränke mich auf das Nötigste:

public class MyMain extends JFrame 
{

	public MyMain()
	{
		Container contentPane = getContentPane();
		contentPane.setLayout(new BorderLayout());
		
		//das wird die View
		MyView mv = new MyView();
		JScrollPane pane = new JScrollPane(mv);
		
		//Objekt vom Modell erstellen
		MyModel mm = new MyModel();
		
		//Controller erstellen mit "Modellübergabe"
		MyController mc = new MyController(mm);
		JScrollPane pane2 = new JScrollPane(mc);
		
		contentPane.add(pane2, BorderLayout.WEST);
		contentPane.add(pane, BorderLayout.CENTER);
		
		//Observer zum controller hinzufügen
		em.addObserver(mc);
		//Observer zum View hinzufügen
		em.addObserver(mv);
	}
	
	public static void main(String[] args) 
	{
		MyMain mm = new MyMain();
		mm.setVisible(true);

	}
}

Meine View ist einfach ein Panel in dem was abgebildet werden soll...

public class MyView extends JPanel implements Observer 
{
	public void update(Observable ob, Object arg)
	{
		MyModel m = (MyModel) ob;
		if(m.getChartPanel() == null)
		{
			//erstellt ein ChartPanel in der anderen ModellKlasse
			m.addBarChart();
                //holt sich das Panel
			panel = m.getChartPanel();
		}
		else
		{
			//Panel holen
			panel = m.getChartPanel();
			//alle Inhalte rauswerfen und neues ChartPanel einfügen
			panel.removeAll();
			panel.add(m.getChartPanel());
		}
		//neu zecihenn
		panel.repaint();
		
	}
	
	private ChartPanel panel;
}

und hier meine Controller Klasse

public class MyController extends JPanel implements Observer, TreeSelectionListener
{
	public MyController(MyModel m)
	{
		//Modelobjekt übergeben
		model = m;
[...]//hier wird halt nen Tree erstellt


		
		tree = new JTree(root);
		int mode = TreeSelectionModel.SINGLE_TREE_SELECTION;
		tree.getSelectionModel().setSelectionMode(mode);
		tree.addTreeSelectionListener(this);
		add(tree);

	}
	
	public void update(Observable ob, Object arg)
	{
		//Hier wird dann irgendwas im Model geändert
		
	}
	
	public void valueChanged(TreeSelectionEvent e)
	{
		TreePath path = tree.getSelectionPath();
		if(path.toString().equals("<Hier Name es Knoten....>"))
		{
			model.addBarChart();
		}
	}
	
	private JTree tree;
	private MyModel model;
}

Und zu guter letzt mein Modell

public class MyModel extends Observable 
{
	
	public void addBarChart()
	{
                //Diagramm generiern
                setCharPanel(...)
		setChanged();
		notifyObservers();

		
	}
	
	public void addPieChart()
	{
                 //Diagramm generiern		
                setCharPanel(...)
		setChanged();
		notifyObservers();
	
	}
	
	
	public void setChartPanel(ChartPanel p)
	{
		chartPanel = p; 
	}
	
	public ChartPanel getChartPanel()
	{
		return chartPanel;
	}
	
	private ChartPanel chartPanel;
}

1) Mich interessiert mal, ob ich das nun wirklich richtig verstanden habe, oder ob das kompletter Mist ist :meld:
2) Bis dato bekomme ich immer

java.lang.IllegalArgumentException: adding container's parent to itself

das passiert im View in Zeile 20 :roll: Verstehe ich nicht so ganz, wie soll ich denn sonst was neues hinzufügen?
3) Wenn ich Panels verschachtel, also z.B. ein Panel in eine JScrollPane und dann vielleicht noch in eine JSplitPane, dann bleibt für das Observable immer das "eigentliche" Panel interessant oder?

Danke für eure Antworten!

Fry

 

[Beni]

Ja guck dir die Zeilen genauer an:

         panel = m.getChartPanel();
         panel.add(m.getChartPanel());

"panel" ist "m.getChartPanel()". Der zweite Befehl lautet also "panel.add( panel )", und genau das geht nicht (Componenten sind wie ein Baum zusammengebaut, damit würdest du aber einen Kreis bilden).

Ich nehme an, du wolltest einfach

this.add( m.getChartPanel() );

schreiben?

 

[Fry]

Hallo Beni,

joa das hat gepaßt Aber jetzt komme ich trotzdem nicht so recht weiter.Es passiert nämlich kein "Update" des Panels wo was angezeigt werden soll.
Ich bin mir auch nicht so ganz sicher ob ich die update Methoden richtig "bestücke"
Also ich habe ja:

Controller--------------------- setData()------------------------> Model

Controller<-----update()--Model-----------update()-------> VIEW

Sprich wenn mein TreeSelectionListener was "merkt" dann soll der Daten im Modell ändern. Das will ich ja durch

   public void valueChanged(TreeSelectionEvent e)
   {
      TreePath path = tree.getSelectionPath();
      if(path.toString().equals("<Hier Name es Knoten....>"))
      {
         model.addBarChart();
      }
   }

machen. Zu dem gefundenen Eintrag wird eine Aktionausgelöst, hier wird eine Methode aufgerufen, die im Modell ein Panel mit einem Barchart hinzufügt.

Wenn das passiert ist, dann müßte nach meinem Verständis das Model bemerkbar machen. Das will ich durch

  public void addBarChart()
   {
                //Diagramm generiern
                setCharPanel(...)
      setChanged();
      notifyObservers();

      
   }

Durch setChanged() wird eine Veränderung verzeichnet und notifyObservers() "sagt" dann den Observern bescheid.
Im Controller möchte ich gar keine Änderung machen, deswegen lasse ich die public void update() einfach leer.

In der View soll ja das Panel praktisch neu gezeichnet werden, also:

public void update(Observable ob, Object arg)
	{
		MyModel m = (MyModel) ob;
		panel = m.getChartPanel();
		if(panel == null)
		{
			//wenns leer, dann hinzufügen
			this.add(m.getChartPanel());
		}
		else
		{
			//alles vom Panel runter
			panel.removeAll();
			this.add(panel);
		}
		//neu malen
		panel.validate();
		panel.repaint();
		
	}

(habs im Gegensatz zum ersten Posting nochmal verändert)
Aber so tut halt nichts. Es stürzt nicht ab oder so, es passiert aber genau auf dem entscheidenen panel nichts :roll:

Ich wollte jetzt beim notifyOberserver() mal das Panel mit übergeben, das ändert aber nichts...

Was ist da noch krumm?

Danke!
Fry

 

[Fry]

Hallo

:bahnhof: oh man, natürlich muss ich panel weiter hinten auch noch durch this ersetzen, war voll der denkfehler :bahnhof: also ich mach noch :### und wenn ich dann noch ne Frage habe, lass ich mich von euch :meld:

trotzdem danke
Fry