actionPerformed funktioniert nicht

[ioannis]

Hallo,

ich habe folgenden code,

/ Create an action
        Action action = new AbstractAction("Start") { 

        // This method is called when the button is pressed
        public void actionPerformed(ActionEvent evt) {
          TotalCountPanel panel = new TotalCountPanel();
          frame.getContentPane().add(panel, BorderLayout.CENTER);
        }

und die Klasse die ich aufrufen will ist folgende

 class TotalCountPanel extends JPanel 
   {
       public void paintComponent(Graphics g)
       {
           super.paintComponent(g);
           g.drawString("Hallo", MESSAGE_X, MESSAGE_Y);
       }
       public static final int MESSAGE_X=150;
       public static final int MESSAGE_Y=150;
   }

nun die übergabe an das JFrame will nicht funktionieren und ich weiß nicht warum. Kann mir da jemand weiterhelfen. WEnn ich den Klassenaufruf vor dem Action Action Befehl eingebe funktioniert die Ausgabe! Was mache ich falsch?

Grüße

Ioannis

 

[SlaterB]

validate();
fehlt

public class TestGUI
    extends JFrame
{

    public TestGUI()
        throws Exception
    {
        JButton b = new JButton("Start");
        b.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent evt)
                {
                    TotalCountPanel panel = new TotalCountPanel();
                    getContentPane().add(panel, BorderLayout.CENTER);
                    validate();
                }
            });
        getContentPane().add(b, BorderLayout.NORTH);

        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(800, 400);
        setVisible(true);
    }

    public static void main(String[] args)
        throws Exception
    {
        new TestGUI();
    }
}


class TotalCountPanel
    extends JPanel
{
    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);
        g.drawString("Hallo", 150, 150);
    }
}

 

[ioannis]

Hallo,

ich weiß zwar nicht wie und warum aber jetzt funktioniert es!

Danke

 

[licht400]

Hallo,
habe auch ein kleine ActionPerformed Problem!
Habe ein Programm was schon lief, aber ich jetz in klassen abgeändert habe.
Da habe ich eine Klasse Aktionen wo der actionPerformed und der valueChanged vorhanden sind.
Dieser valueChanged funktioniert, aber meine actionPeformed leider nicht..sprich, es reagiert kein button.

die action liegt in der klasse a!

 public void actionPerformed(ActionEvent e) {
        
        //close
        Object quelle = e.getSource();
        if (quelle == d.bitem){
            if (JOptionPane.showConfirmDialog(this, "Möchten Sie wirklich aufhören","Frage",
                    JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE)
                    == JOptionPane.YES_OPTION) {
                    System.exit(0);
                    }

}

dieser button liegt in der klasse d (wie design) und habe ihm auch dem actionListener übergeben. wieso reagiert er aber nicht?

 

[SlaterB]

zum einen, weil er aus 10 Zeilen unsäglich komischen Code besteht,
zur Frage, ob ein ActionListener funktioniert, gibt es nur eine Art von actionPerformed-Operationen:

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


alles andere stört nur,

----------

so, wenn diese Meldung kommt, dann funktionierts, ansonsten nicht,
aber an der actionPerformed kann es dann nicht mehr liegen,
muss wohl mehr Code posten , am besten ein vollständiges Beispielprogramm < 100 Zeilen

falls die Meldung schon kommt, aber dein ursprünglicher ActionListener nicht geht,
dann wahrscheinlich, weil e.getSource() doch nicht das ist was du vermutest,
gib von beiden Objekten den hashcode() aus, vielleicht ist die Exemplarvariable gar null?
wo wird sie gesetzt?

 

[licht400]

da er mir nix ausgibt, wenn ich system.out.println("funkst") mache, gehe ich von aus, das die actionPerformed nix macht.

so..habe 2 klassen...einmal design und einmal die aktionen.

aktionen

public class aktionen extends JFrame implements ActionListener{
private design d;

  public void init(design d){  //die init wird in der main aufgerufen
        this.d=d;
      }

 public void actionPerformed(ActionEvent e) {
        System.out.println("funkst");
        //close
        //.....ist ja erstmal egal...hier wäre dann der zugriff auf bitem zum beispiel
}
}

design

public class design extends JFrame {

private aktionen a;

 public void init(aktionen a){
        this.a=a;        
    }
public design(){

//Menu
        menuBar = new JMenuBar();
        menu = new JMenu("suchen");
        menuBar.setLayout(new BorderLayout());
        menuBar.add(menu);
        sitem = new JMenuItem("suchen");
        bitem = new JMenuItem("beenden");
        menu.add(sitem);
        menu.add(bitem);
        
        menuBar.add(menu, BorderLayout.EAST);
        setJMenuBar(menuBar);
        bitem.addActionListener(a);
        sitem.addActionListener(a);
//...

}
}

so hat es ja schon mal funktioniert, bevor ich das ganze in klassen gemacht habe ;-)!

 

[SlaterB]

ach du Schande, das aktionen-design-tree-Trauma..

du hast wie immer
bitem.addActionListener(a);
im Konstruktor, wobei a noch null ist und doch erst SPÄTER in der init()-Operation gesetzt wird,
a ist im Konstruktor noch NULL, glaub das doch endlich mal,

und ist so leicht zu prüfen:
statt
bitem.addActionListener(a);

schreibe

System.out.println("a ist: "+a);
bitem.addActionListener(a);

wieso schaffst du es nie, so eine kleine Ausgabe einzubauen?
na dir ist wohl nicht mehr zu helfen..

 

[licht400]

das dieser leer ist, ist mir ja bewusst, da er nicht reagiert und vorher es funktioniert hat.
aber wenn mein listener nun mal in a ist, muss ich denn ja mit a weiter geben..
so habe ich es eigentlich schon mal woanders gemacht!

 

[licht400]

wie übergebe ich dem button nun dann den actionlistener, da a ja eh null ist
edit: ja bin in dieser sache vll schwer von begriff, aber könntest mir sagen, wie ich der komponente den actionlistener zuweise, ohne das dieser null ist?

 

[SlaterB]

wie schon tausendmal gesagt:
jeder Code der a benötigt darf nicht im Konstruktor stehen,
sondern erst in init() wenn a da ist!

gaaaanz einfach,
kaufe nie im Supermarkt ein bevor du das Geld vom Bankautomaten abgehoben hast/ Taschengeld von Eltern bekommen hast?

---------

wahrscheinlich solltest du einfach das Themal init() ein für alle mal begraben und wieder zu ordentlichen ganz normalen Konstruktoren mit Parametern zurückkehren,
im damaligen Thread habe ich dir ja die richtige Reihenfolge vorgeschlagen

 

[licht400]

gut, ich möcht es aber gern mit dem init machen.

gut, wenn das a drine steht, funktioniert es nicht, da zu dem zeitpunkt a null ist.
hab ausporbiert new aktionen() , statt a..da geht zumindest die performed wieder, aber bekomm dann bei meinen buttons ein nullpointer..

also kannst mir da jetz keine lösung sagen, wie ich dem bitem den actionlistener hinzufüge (auch wenn ich mich gerade vll blöd stelle)

 

[SlaterB]

new aktionen() statt a wäre fatal, darum gehts doch die ganze Zeit bei init(),
dass du nur ein Objekt a hast usw.

> also kannst mir da jetz keine lösung sagen, wie ich dem bitem den actionlistener hinzufüge

habe ich schon längst, schon zig mal, immer das gleiche Problem, IMMER die gleiche Lösung:

jeder Code der a benötigt darf nicht im Konstruktor stehen,
sondern erst in init() wenn a da ist!

alles rein in init, aber nicht als Komplettlösung sehen, kann auch zu anderen Problemen führen,
wie bei den JPanels in einem der anderen Threads

nach Möglichkeit also allen Code, der nicht von einer der init-Variablen abhängt, im Konstruktor,
der Rest dann in init,

public design(){ 

//Menu 
        menuBar = new JMenuBar(); 
        menu = new JMenu("suchen"); 
        menuBar.setLayout(new BorderLayout()); 
        menuBar.add(menu); 
        sitem = new JMenuItem("suchen"); 
        bitem = new JMenuItem("beenden"); 
        menu.add(sitem); 
        menu.add(bitem); 
        
        menuBar.add(menu, BorderLayout.EAST); 
        setJMenuBar(menuBar); 
//... 

} 

public void init(aktionen a){ 
        this.a=a;        

        bitem.addActionListener(a); 
        sitem.addActionListener(a); 

    }

 

[licht400]

sprich, wenn ich was aus der klasse benötige wie den actionlistener muss ich das in die init schreiben!

danke, hat geklappt und ich glaube mir ist jetz ein licht aufgegangen ;-)!

 

[SlaterB]

ich glaube nicht.. (dein 'sprich, ..' stimmte aber)