Element anzeigen / erstellen. Maus vs Controller

[filth]

Hallo,

ich habe eine Methode, die im TabbedPane Tabs anlegt:

    public void generateImageTab(String title) {
    	
        UUID id = UUID.randomUUID();
        setTabLayoutPolicy(JTabbedPane.WRAP_TAB_LAYOUT);

        ImageAreaComponent imageArea = new ImageAreaComponent(id, myView);
	imageArea.setBackground(new java.awt.Color(43,89,105));
	imageArea.setForeground(new java.awt.Color(43,89,105));
        imageAreaList.add(imageArea);
        
        //Pack pack = new Pack();
        //packList.add(pack);
        
        JScrollPane jScrollPane = new JScrollPane(imageArea);
        jScrollPane.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
        jScrollPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
	jScrollPane.setDoubleBuffered(true);
	jScrollPane.setBorder(new LineBorder(new java.awt.Color(21,60,70), 1, true));
	jScrollPane.setBackground(new java.awt.Color(43,89,105));
        jScrollPane.getVerticalScrollBar().setUnitIncrement(20);
        scrollPaneList.add(jScrollPane);

        title = title == null? "" : title;
        title = title.equals("") ? "Bildpaket" : title;
        addTab(title,jScrollPane);
    }

Die Methode wird über einen MouseListener in der gleichen Klasse aufgerufen:

        miAppend = new JMenuItem("Paket hinzufügen");
        miAppend.setOpaque(true);
        miAppend.setFont(new java.awt.Font("Segoe UI",0,14));
        miAppend.setBackground(new java.awt.Color(37,81,95));
        miAppend.setForeground(new java.awt.Color(255,255,255));
        
        miAppend.setToolTipText("Es können insgesamt 5 Bildpakete angelegt werden");
        miAppend.addActionListener(new java.awt.event.ActionListener() {
            @Override
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                if(imageAreaList.size()<5) {
                    generateImageTab("");
                }
            }
        });

Das funktioniert soweit und neue Tabs erscheinen.
Nun soll aber an einer weiteren Stelle, im Controller die gleiche Methode "manuell" getriggert werden:

	public void initTabbedPane(String eventName)
	{
		System.out.println("new tab");
		ImageTabbedPane myPane = new ImageTabbedPane(myView);
		myPane.generateImageTab("");
	
		myView.setTabbedPane(myPane);
	}

In diesem Fall passiert nichts, es erscheint kein neuer Tab. Hat jemand eine Idee, was ich falsch mache?

Danke!

 

[Simon_Flagg]

du addest die tabbedpane ja zu keinem frame... wie soll da was angezeigt werden?

lg

 

[Michael...]

Grundsätzlich ein merkwürdiges Konstrukt. Hast Du JTabbedPane überschrieben und darin ein JMenu inkl. anonymen ActionListner (!=MouseListener) definiert?

Wo wird denn das in der init-Metode erzeugte Tabbed Pane "verbaut"? Was macht die Methode setTabbedPane(...)?

 

[filth]

Grundsätzlich ein merkwürdiges Konstrukt. Hast Du JTabbedPane überschrieben und darin ein JMenu inkl. anonymen ActionListner (!=MouseListener) definiert?

Wo wird denn das in der init-Metode erzeugte Tabbed Pane "verbaut"? Was macht die Methode setTabbedPane(...)?

Also das ImageTabbedPane leitet vom JTabbedPane ab. Dort im Init ist ein JPopupMenu mit einem ActionListener.
In der View wird beim Initialisieren ein ImageTabbedPane angelegt, was später durch das neue mit der setTabbedPane überschrieben werden soll.

 

[Michael...]

In der View wird beim Initialisieren ein ImageTabbedPane angelegt, was später durch das neue mit der setTabbedPane überschrieben werden soll.

Was heisst überschrieben? Wie sieht denn der Code der Methode aus?

 

[filth]

So:

	public void setTabbedPane(ImageTabbedPane i)
	{
		this.imageTabbedPane = i;
	}

 

[Michael...]

So:

	public void setTabbedPane(ImageTabbedPane i)
	{
		this.imageTabbedPane = i;
	}

Hiermit referenziert nur die Variable imageTabbedPane auf die neue ImageTabbedPane - mehr nicht.
Das hat keine Auswirkungen auf die GUI oder sonst was. Nur auf Verwender die anschließend mit der Variable arbeiten.
Du müsstest die alte TabbedPane aus dem Container entfernen und die neue hinzufügen. Eventuell noch ein validate() auf auf den Container.

Aber soll wirklich die "alte" TabbedPane durch eine neue ersetzt werden? Eventuell reicht es ja aus einfach

 generateImageTab("")

an der bestehenden TabbedPane aufzurufen?

 

[filth]

Super! Vielen Dank!

	public void setTabbedPane(ImageTabbedPane i)
	{
		rightFrame.remove(imageTabbedPane);		
		this.imageTabbedPane = i;
		rightFrame.add(i, "0, 0, 3, 3");
	}

Es müssen in diesem Fall wirklich unterschiedliche Panels sein.
So funktioniert es. Ich hätte gedacht, dass nach dem Ändern der Referenz die View beim Repaint dann den neuen Inhalt automatisch nimmt.