Problem mit Action Listener in privater inneren Klasse

[FenchelT]

Hallo zusammen,

ich benoetige eine Tabbed Pane mit der Moeglichkeit diese ueber ein X pro Tab zu schliessen.
Dazu habe ich mich bei Sun durch dieses kleine Tutorial
http://java.sun.com/docs/books/tutorial/uiswing/components/tabbedpane.html
durch gearbeitet und es funktioniert einwandfrei.

Nun habe ich eine Controller KLasse, die die Tabs instanziiert und ein JPanel mit Textfeldern als Component uebergibt. Die Textfelder des Panels werden zusaetzlich in der Controller Klasse an einem KeyListener
angemeldet, um eine Instanzvariable auf true/false zu setzen, falls sich in einem Textfeld etwas
geaendert hat.
Das funktioniert auch einwandfrei.

Nun moechte ich aber, dass sobald auf das X des Tabs geklickt wird, die Variable in der Contr.-KLasse
abgefragt wird um abzufangen, dass Aenderungen bei schliessen verloren gehen,
falls nicht der Speichern Button gedrueckt wurde

Der ActionListener befindet sich aber im Sun-Tutorial in einer privaten inneren Klasse der Klasse, die das
erzeugen der schliessbaren Tabs realisiert.

Wie bekommt die Controller Klasse am einfachsten mit, wenn der ActionListener der ButtonTab-Klasse angesprochen wird?

Vielen Dank fuer eure Hilfe.

 

[SlaterB]

wo ist denn der Controller?

in der gleichen Operation als Parameter oder lokale Variable?
-> final machen, dann klappt der Zugriff

als Exemlarvariable?
-> Zugriff geht sowieso

Code posten..

 

[FenchelT]

Hallo Slater,

erstmal danke fuer DEine Antwort.

Nun, Du wolltest Code, hier kriegst Du ihn :wink:


Zuerst die ButtonTab KLasse

public class ButtonTabComponent extends JPanel {
    private final JTabbedPane pane;

    public ButtonTabComponent(final JTabbedPane pane) {
        //unset default FlowLayout' gaps
        super(new FlowLayout(FlowLayout.LEFT, 0, 0));
        if (pane == null) {
            throw new NullPointerException("TabbedPane is null");
        }
        this.pane = pane;
        setOpaque(false);
        
        //make JLabel read titles from JTabbedPane
        JLabel label = new JLabel() {
            public String getText() {
                int i = pane.indexOfTabComponent(ButtonTabComponent.this);
                if (i != -1) {
                    return pane.getTitleAt(i);
                }
                return null;
            }
        };
        
        add(label);
        //add more space between the label and the button
        label.setBorder(BorderFactory.createEmptyBorder(0, 0, 0, 5));
        //tab button
        JButton button = new TabButton();
        add(button);
        //add more space to the top of the component
        setBorder(BorderFactory.createEmptyBorder(2, 0, 0, 0));
    }

    private class TabButton extends JButton implements ActionListener {
        public TabButton() {
            int size = 17;
            setPreferredSize(new Dimension(size, size));
            setToolTipText("close this tab");
            //Make the button looks the same for all Laf's
            setUI(new BasicButtonUI());
            //Make it transparent
            setContentAreaFilled(false);
            //No need to be focusable
            setFocusable(false);
            setBorder(BorderFactory.createEtchedBorder());
            setBorderPainted(false);
            //Making nice rollover effect
            //we use the same listener for all buttons
            addMouseListener(buttonMouseListener);
            setRolloverEnabled(true);
            //Close the proper tab by clicking the button
            addActionListener(this);
        }

        public void actionPerformed(ActionEvent e) {
            int i = pane.indexOfTabComponent(ButtonTabComponent.this);
            if (i != -1) {
                pane.remove(i);
            }
        }

        //we don't want to update UI for this button
        public void updateUI() {
        }

        //paint the cross
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            Graphics2D g2 = (Graphics2D) g.create();
            //shift the image for pressed buttons
            if (getModel().isPressed()) {
                g2.translate(1, 1);
            }
            g2.setStroke(new BasicStroke(2));
            g2.setColor(Color.BLACK);
            if (getModel().isRollover()) {
                g2.setColor(Color.MAGENTA);
            }
            int delta = 6;
            g2.drawLine(delta, delta, getWidth() - delta - 1, getHeight() - delta - 1);
            g2.drawLine(getWidth() - delta - 1, delta, delta, getHeight() - delta - 1);
            g2.dispose();
        }
    }

    private final static MouseListener buttonMouseListener = new MouseAdapter() {
        public void mouseEntered(MouseEvent e) {
            Component component = e.getComponent();
            if (component instanceof AbstractButton) {
                AbstractButton button = (AbstractButton) component;
                button.setBorderPainted(true);
            }
        }

        public void mouseExited(MouseEvent e) {
            Component component = e.getComponent();
            if (component instanceof AbstractButton) {
                AbstractButton button = (AbstractButton) component;
                button.setBorderPainted(false);
            }
        }
    };
}

Und hier noch das was in meinem Controller passiert:

// Fuegt einer TabbedPane einen neuen Tab hinzu
private void tabHinzufuegen()
	{
		this.kundeTab.getTabPane().add(title, this.asp[this.countAsp]);    // title, ist die Beschriftung des Tab, asp ist ein JPanel
        initTabComponent(this.countAsp);
	}


// Aendert das "Aussehen" des Tab so dass es mit einem x versehen wird
private void initTabComponent(int i) {
		this.kundeTab.getTabPane().setTabComponentAt(i+1,
	             new ButtonTabComponent(this.kundeTab.getTabPane()));
	}


// Meldet alle Textfelder am KeyListener an
private void initTabListener()
	{	
		Component[] comps = this.kundeGui.getComponents();	
        
        for (int x=0; x <=comps.length-1;x++)
        {
        	if(comps[x] instanceof JTextField)
        	{	
        		JTextField textfield = (JTextField) comps[x];
        		textfield.addKeyListener(this);
        	}
        }
		
	}

// Aendert bei Eingabe in ein Textfeld eine Boolsche Variable Instanzvariable 
public void keyPressed(KeyEvent arg0) 
	{
		// TODO Auto-generated method stub
		this.changeCtrl = true;
		
	}

Danke Dir nochmals

 

[SlaterB]

du hast Erklärungen und den Bezug zur Frage vergessen,
oder soll ich jetzt selber interpretieren, was davon 'Der ActionListener befindet sich aber im Sun-Tutorial in einer privaten inneren Klasse der Klasse' ist?

das mit der inneren Klasse hatte ich aber tatsächlich nicht so genau gelesen,
dann hilft nur, den Controller im Konstruktor als Parameter zu übergeben und im ActionListener als Exemplarvariable abzuspeichern

 

[FenchelT]

Hallo Slater,

habe das ganze gestern noch ein wenig umgestrickt; der Action- und die MouseListener befinden sich nun in meiner Controller Klasse, den das Kontroller-Objekt wollte ich eigentlich nicht an eine andere
Klasse uebergeben, damit dort weitergearbeitet werden kann; vielmehr wollte ich, dass der Controller
die Faeden in der Hand haelt.

Das funktioniert auch soweit, nur weiss ich im Moment noch nicht genau,
wie ich diesen Teil aus der inneren Button Klasse in die Controller Klasse ausgelagert bekomme:

public void actionPerformed(ActionEvent e) {
            int i = pane.indexOfTabComponent(ButtonTabComponent.this);
            if (i != -1) {
                pane.remove(i);
            }
        }

Ich werde noch ein wenig weiterbasteln; wenn Dir noch etwas einfaellt wuerde ich mich ueber weitere Tips freuen.

Vielen Dank nochmal

 

[Tom299]

du könntest dir z.b. ein Interface definieren und im Controller einen Listener, der das ActionEvent von deinem TabButton abfängt und das entsprechende Event aus dem Interface feuert.
in deiner TabButton Klasse brauchste nur noch das Interface auszuimplementieren und beim Controller anzumelden. in deiner Interface-Implementierung haste dann vollen zugriff auf all deine internen komponenten und du brauchst z.b. deine klasse oder deine buttons nicht mit getter-methoden rauszugeben.

 

[FenchelT]

Hallo zusammen,

nach einigen Tests habe ich mich dann doch dazu entschieden, es so zu machen, wie Slater es vorgeschlagen hatte.

dann hilft nur, den Controller im Konstruktor als Parameter zu übergeben und im ActionListener als Exemplarvariable abzuspeichern

Laeuft nun.

Danke nochmals.
Werde mir das mit den Interfaces fuer diesen Fall aber nochmal in Ruhe anschauen.