Discussion :

[comme de bien entendu]

Bonjour,
J'ai envie de séparer un peu mes actionPerformed de ma classe
principale, et pour ca j'ai lu qu'il était conseillé de passer par
des classes internes.
Seulement lorsque je veux lancer des methodes de la classe interne en faisant reference à la classe principale ca marche pas car elles sont
de type non static, est ce que vous auriez une solution vu mon code ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
public class MonGraphique extends JPanel 
{
JPanel CePanel;
MonGraphique()
{
JButton MonButton = new JButton("test");
this.setLayout(new GridLayout(4, 3));  
this.add(MonButton);
MonButton.addActionListener(new UnListener() );
}
 
 
// classe interne du listener
 
private class UnListener implements ActionListener
{
      public  void actionPerformed(ActionEvent e) {
        MonGraphique.AfficheDesBoutons();
        MonGraphique.repaint();
      }
protected void AfficheDesBoutons()
{
JButton MonButton = new JButton("cliqué");
MonGraphique.add(MonButton);
}
}

D'avance merci de vos réponses

[schneidb]

Salut,

par convention les noms de variables en java commencent tous par une minuscule

Si tu veux accèder au bouton à partir de ton écouteur il faut déclarer
JButton MonButton en attribut comme ton panel CePanel

Créer une classe interne comme écouteur ce n'est pas forcément ce qu'il y a de mieux, cela dépend des besoins. Avoir deux classes distinctes c'est bien aussi.

[comme de bien entendu]

J'ai fait la modif en passant le JPanel en attribut au constructeur, mais j'ai un NullPointerException sur leGraphique.add(monButton);
Mon JPanel etant bien instancié je ne vois pas l'erreur que je commets.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
private class UnListener implements ActionListener
{
JPanel leGraphique;
UnListener(JPanel cePanel)
{
leGraphique=cePanel;
}
 
 
public  void actionPerformed(ActionEvent e) {
        AfficheDesBoutons();
         }
 protected void AfficheDesBoutons()
        {
      System.out.println("======================test1");
      JButton monButton = new JButton("cliqué");
       leGraphique.add(monButton);
       }
}

[schneidb]

Quand on utilise un object il faut le déclarer et l'instancier par un new JPanel par exemple ici.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public class MonGraphique extends JPanel
{
JButton monButton;
public MonGraphique()
{
monButton = new JButton("test");
this.setLayout(new GridLayout(4, 3)); 
this.add(MonButton);
monButton.addActionListener(new UnListener() );
}
 
// classe interne du listener
 
private class UnListener implements ActionListener
{
      public  void actionPerformed(ActionEvent e)
      {
        monButton.setText("clique!");
        // ou pour faire un écouteur qui modifie le text de n'importe quel boutton:
        Object o = e.getSource ();
        ((JButton)o).setText("clique!");
      }
}

[comme de bien entendu]

Merci de ta réponse mais, ca ne fait pas ce que je souhaite, car je voulais créer un nouveau JButton à chaque fois que je clique sur le bouton de ma classe MonGraphique. Cela marche tres bien si j'instancie mon bouton de cette maniére là depuis ma classe MonGraphique

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
MonButton.addActionListener(new ActionListener() {
          public  void actionPerformed(ActionEvent e) {
              JButton MonButton = new JButton("cliqué");
            this.add(MonButton);
})

Mais sans vouloir "enculer les mouches", je souhaitais déporter mon ActionListener pour avoir quelque chose de plus lisible.
Mais ce n'est peut être pas possible....
(NB mon JPanel lui est instancié par une classe JFrame au préalable)

[xavlours]

Pour ton code, si tu veux déplacer le listener dans une classe interne, tu peux remplacer :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

this.add(monButton);

Par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

MaClasseExterne.this.add(monButton);

Par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public class MonPanel extends JPanel {
 
  int nbButton;
  MonActionListener monActionListener;
 
  public MonPanel() {
    super(new FlowLayout());
    monActionListener = new MonActionListener();
 
    nbButton = 1;
    JButton b = new JButton("bouton " + nbButton++);
    b.addActionListener(monActionListener);
    add(b);
  }
 
  private class MonActionListener implements ActionListener {
 
    public void actionPerformed(ActionEvent e) {
      JButton b = new JButton("bouton " + nbButton++);
      b.addActionListener(this);
      MonPanel.this.add(b);
    }
  }
}

[comme de bien entendu]

Merci ca marche bien. J'ai appri un truc. J'aurais pas pensé au
ClasseExterne.this.... pour appeler depuis la classe Interne
Bonne continuation