ActionListener und Singleton

[tobi193]

Hallo,

ich habe hier ein Problem mit dem ActionListener. Und zwar wird die actionPerformed Methode nicht aufgerufen. Da mein Quellcode etwas zu kompliziert ist um ihn hier zu zeigen, habe ich das ganze vereinfacht mal nachgebaut. Auch in der simplen Fassung funktioniert es nicht:

public class GUI extends JFrame{
    public GUI(){
        super("bla");
        JButton bu = new JButton("BLUBB");
        bu.addActionListener(Test.getInstance());
        this.add(bu);
        this.setVisible(true);
    }
}




public class Test implements ActionListener{

    private static Test instance = new Test();
    private GUI gui = new GUI();

    private Test(){

    }

    public static void main(String[] args){
        Test.getInstance();
    }

    public static Test getInstance(){
        return instance;
    }

    public void actionPerformed(ActionEvent e) {
       System.out.println("bla");
    }
}

Bei einem Buttonclick passiert einfach nichts ("bla" wird nicht ausgegeben). Woran liegt das ?

Vielen Dank für eure Hilfe !

 

[Runtime]

Wie nichts? Reagiert der Button nicht?

 

[tobi193]

Nein, die actionPerformed Methode wird nicht aufgerufen.

 

[dehlen]

soweit ich das beurteilen kann (nur vermutung) muss

public class Test implements ActionListener{

hinter diese Zeile die action performed Methode

 

[z-mon]

Also der Code sieht soweit ok aus. Konnte ihn auch ohne weiteres compilieren (die Sichtbarkeit des Fensters muss natürlich noch gesetzt sein).

Bekommst du auch keine Fehlermeldung oder ähnliches?

 

[eRaaaa]

bu.addActionListener(Test.getInstance());

instance

ist null!
Mache mal

 GUI gui

auch static , wobei das alles etwas merkwürdig verdrahtet ist

 

[z-mon]

Vom Prinzip sollte deine Geschichte eigentlich in etwa so aussehen:

public class GUI extends JFrame{
    public GUI(){
        JButton bu = new JButton("BLUBB");
        bu.addActionListener(new Test());
        add(bu);
        pack();
        setVisible(true);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
    }
    
    public static void main(String[] args) {
		new GUI();
	}
}
 
 
class Test implements ActionListener{
	public void actionPerformed(ActionEvent e) {
		System.out.println("blubb");
	}
}

 

[tobi193]

@ z-mon: Klar, das ist natürlich die einfachste Möglichkeit, aber ich benötige eben die Klasse "Test" als Singleton. Wie gesagt ist das Beispiel nur ein (stark) vereinfachter Code aus meinem eigentlichen Programm.

@ eRaaaa: Mit static scheint es im Beispiel zu funktionieren (bei richtiger Beachtung der Reihenfolge bei der Initialisierung. Das werde ich gleich mal im eigentlichen Programm ausprobieren.

 

[tfa]

Bei einem Buttonclick passiert einfach nichts ("bla" wird nicht ausgegeben). Woran liegt das ?

Einen netten Bug hast du da produziert. Die Initialisierung und der "Start" des Programms durch die gui-Membervariable in Test ist auch wirklich kompliziert und ungeschickt.

Und versuche um Himmels willen dieses Singleton weg zu kriegen. Die sind auf 100 Arten gefährlich und als ActionListener braucht man sie garantiert nicht.

 

[tobi193]

Ja, ich habe mich total verwurstelt.

Mein Programm ist in etwa so aufgebaut:

GUI - Oberfläche

Controller - alle steuernden Geschichten

Client - Netzwerkverbindung

Bis vor kurzem hatte ich die GUI und den Controller in einer Klasse. Aber bei zunehmender Komplexität ist das sehr unsauber und unübersichtlich. Daher muss getrennt werden. Aber daran scheitert es nun.
Ich habe in der GUI einige Buttons auf die ein ActionListener registriert ist sowie mehrere Labels, welche von einem Thread im Controller ständig bearbeitet werden.
Somit habe ich gegenseitige Beziehungen, welche ich irgendwie nicht gelöst bekomme.

Wie baue ich jetzt das ganze optimal auf ? Mit der Singletonvariante wohl nicht.

 

[z-mon]

Mal so eine Frage am Rande .. weißt du eigentlich was genau ein Singleton ist und zu welchem Zweck es eingesetzt wird?

 

[tfa]

In der Controller-Klasse definierst du für jeden Button ein Modell (z.B. DefaultButtonModel oder auch einfach nur ein ActionListener). Dieses Modell bindest du bei der Initialisierung des GUI-Objektes an den betreffenden Button.
Ungefähr so:

public class Gui {
	
	public Gui(Controller ctrl) {
		getOkButton().setModel(ctrl.getOkButtonModel());
	}
}

public class Controller {
	
	private DefaultButtonModel okButtonModel;
	
	public DefaultButtonModel getOkButtonModel() {
		if (okButtonModel==null) {
			okButtonModel = new DefaultButtonModel();
			okButtonModel.addActionListener(new ActionListener() {
				
				public void actionPerformed(ActionEvent e) {
					okButtonPressed();
				}
			});
		}
	}
}

Das wäre eine ganz einfache Möglichkeit.

 

[tagedieb]

Da hast du ein nettes kleines Problem konstruiert ;-)

Die Loesung duerfte jetzt inzwischen klar sein.
Das Problem hat nichts mit Swing/AWT zu tun sondern mit 'Concurrency'.
Du musst umbedingt darauf achten, dass du deine Objekte erst VOLLSTAENDIG initialisiert bevor du sie benutzt (publizierst).
Ansonsten ist das Ergebnis unvorhersehbar wenn du mit halb initialisierten Objekten arbeitest!!

Versuche auch bidirektionale Beziehungen zu vermeiden, dann ist der Code gleich viel verstaendlicher und erhoeht die Erweiterbarkeit deiner Software.

 

[tfa]

Das Problem hat nichts mit Swing/AWT zu tun sondern mit 'Concurrency'.

Nö. Hier gab es ja nur einen Thread.

 

[tagedieb]

Nö. Hier gab es ja nur einen Thread.

Da sieht man mal wieder, dass nicht mal ein eiziger Thread threadsafe ist ;-)
Recht hast du - es geht um die Initialisierung von Objekten. Habs im Concurrency Handbuch nachgeschlagen, deshalb ist das hier reingerutscht.

 

[Tente]

Jap Initialisierungsfehler. Mach den getter lazy und die instanz threadsafe, dann sollte es gehen:

public class GUI extends JFrame{
    public GUI(){
        super("bla");
        JButton bu = new JButton("BLUBB");
        bu.addActionListener(Test.getInstance());
        this.add(bu);
        this.setVisible(true);
    }

   public static void main(String args[]){
        new GUI();
   }
}
 

 
 
public class Test implements ActionListener{
 
    private static volatile Test instance;
     
    private Test(){
 
    }
 
    public static Test getInstance(){
        if(instance == null)
            instance = new Test();
        return instance;
    }
 
    public void actionPerformed(ActionEvent e) {
       System.out.println("bla");
    }
}

Nicht getestet, sollte aber funktionieren. Und du hast dein Singleton....