Discussion :

[Auteur]

bonjour,

dans une fonction j'ai une boucle qui me permet de créer 3 boutons et leur ActionListener. Je voudrais passer en paramètre à la fonction appelée lors du clic sur le bouton le contenu de la variable de boucle :

Code java :

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
 
        JButton btn[]= new JButton[3];
int i;
 
for (i=0; i<3; i++)
{
    btn[i]=new JButton("Bouton " + i);
    btn[i].addActionListener(new ActionListener()
    {
 
        @Override
        public void actionPerformed(ActionEvent e)
        {
            System.out.println(i);
        }
    });
}

Malheureusement au moment de la compilation j'ai cette erreur au niveau du println() :

error: local variable i is accessed from within inner class; needs to be declared final
System.out.println("bouton n°" + i);

Si je déclare i en final int mon erreur se trouve au niveau du for


D'où ma question : comment je peux passer le contenu de i ?

[oneagaindoguys]

Quand tu fais

Code :

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

new ActionListener()

tu es dans une classe interne, et la variable i, n'est plus visible. Il faut alors déclarer ta variable i en variable globale. C'est à dire avec les attributs de la classe.

Code :

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

1
2
3
4
5
6
public class MaClasse {
int i;
...
 
public void taMethode() {
for (i=0...

A ta place je ne m'y prendrais pas comme ça. Ta classe devrait implémenter ActionListener et dans la méthode actionPerformed, tu boucles sur le tableau, tu intercepte le bouton source et avec un switch case tu gère l'action du bouton en fonction de sa position dans le tableau.

[Auteur]

Quand tu fais

Code :

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

new ActionListener()

tu es dans une classe interne, et la variable i, n'est plus visible. Il faut alors déclarer ta variable i en variable globale. C'est à dire avec les attributs de la classe.

Code :

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

1
2
3
4
5
6
public class MaClasse {
int i;
...
 
public void taMethode() {
for (i=0...

ça ne fonctionne pas seule la dernière valeur de i est prise en compte...

A ta place je ne m'y prendrais pas comme ça. Ta classe devrait implémenter ActionListener et dans la méthode actionPerformed, tu boucles sur le tableau, tu intercepte le bouton source et avec un switch case tu gère l'action du bouton en fonction de sa position dans le tableau.

Dans ce cas quelle propriété du bouton est testée dans le switch case ? Comment je peux les différencier les uns des autres ? En utilisant setName() ?

[babycart]

Bonsoir,
Tu peux faire comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int i=0; i<3; i++)         {
 final int iTmp = i;
 btn[i]=new JButton("Bouton " + i);             
 btn[i].addActionListener(new ActionListener() {

   @Override
   public void actionPerformed(ActionEvent e) {
       System.out.println(iTmp);                 
   }             
});         
}

ou encore :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int i=0; i<3; i++)         {
 btn[i]=new JButton("Bouton " + i);  
 btn[i].setActionCommand(Integer.toString(i));           
 btn[i].addActionListener(new ActionListener() {

   @Override
   public void actionPerformed(ActionEvent e) {
       System.out.println(e.getActionCommand());                 
   }             
});         
}

Mais tout dépend de ce que tu veux faire au final (?)

[oneagaindoguys]

Bonsoir,
Tu peux faire comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int i=0; i<3; i++)         {
 final int iTmp = i;
 btn[i]=new JButton("Bouton " + i);             
 btn[i].addActionListener(new ActionListener() {

   @Override
   public void actionPerformed(ActionEvent e) {
       System.out.println(iTmp);                 
   }             
});         
}

ou encore :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int i=0; i<3; i++)         {
 btn[i]=new JButton("Bouton " + i);  
 btn[i].setActionCommand(Integer.toString(i));           
 btn[i].addActionListener(new ActionListener() {

   @Override
   public void actionPerformed(ActionEvent e) {
       System.out.println(e.getActionCommand());                 
   }             
});         
}

Mais tout dépend de ce que tu veux faire au final (?)

Le mieux, est que la classa implémente ActionListener

[oneagaindoguys]

ça ne fonctionne pas seule la dernière valeur de i est prise en compte...



Dans ce cas quelle propriété du bouton est testée dans le switch case ? Comment je peux les différencier les uns des autres ? En utilisant setName() ?

Lorsque tu instancies un bouton la JVM lui attribue une place en mémoire. C'est l'adresse mémoire qui est prise en compte.

Si tu veux envoies ton code, je vais essayer de l'adapter

[Auteur]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int i=0; i<3; i++)         {
 final int iTmp = i;
 btn[i]=new JButton("Bouton " + i);             
 btn[i].addActionListener(new ActionListener() {
 
   @Override
   public void actionPerformed(ActionEvent e) {
       System.out.println(iTmp);                 
   }             
});         
}

j'y ai pensé et testé : effectivement cela fonctionne. Mais, j'avoue avoir une certaine réticence à déclarer des variables dans des boucles (sans doute parce que j'ai dans le passé utilisé des langages qui interdisent ce cas de figure).

Par ailleurs, au niveau de la mémoire comment sont gérées les déclarations dans les boucles ?

ou encore :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int i=0; i<3; i++)         {
 btn[i]=new JButton("Bouton " + i);  
 btn[i].setActionCommand(Integer.toString(i));           
 btn[i].addActionListener(new ActionListener() {
 
   @Override
   public void actionPerformed(ActionEvent e) {
       System.out.println(e.getActionCommand());                 
   }             
});         
}

pas mal Je testerai dès que possible Mais une question qu'est-ce que la méthode setActionCommand() ? La javadoc n'est pas très explicite

public void setActionCommand(String actionCommand)
Sets the action command for this button.

Lorsque tu instancies un bouton la JVM lui attribue une place en mémoire. C'est l'adresse mémoire qui est prise en compte.

je testerai donc cette possibilité avec le méthode equals de getSource().

Si tu veux envoies ton code, je vais essayer de l'adapter

non, je préfère le faire moi même à partir des suggestions postées

[oneagaindoguys]

non, je préfère le faire moi même à partir des suggestions postées

Eh bien si tu veux coder correctement fait ta classe implémenter ActionListener.

Code :

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

1
2
3
4
5
6
7
8
 
public void actionPerformed(ActionEvent e) {
   for (int i=0; i<btn.length; i++) {
      if (e.getSource().equals(btn[i])) {
         switch...
         //ici tu gères l'action en fonction du boutton
   }
}

ActionListener est une interface et instancier une interface n'est pas ce qui est de plus cohérent. Imagine tu 30 boutons dans ton tableau, tu instancie 30 ActionListener, c'est lourd.

[Logan Mauzaize]

j'y ai pensé et testé : effectivement cela fonctionne. Mais, j'avoue avoir une certaine réticence à déclarer des variables dans des boucles (sans doute parce que j'ai dans le passé utilisé des langages qui interdisent ce cas de figure).

C'est bien comme cela qu'il faut faire rapidement.

Par ailleurs, au niveau de la mémoire comment sont gérées les déclarations dans les boucles ?

Quand tu crées une classe anonyme, le compilateur génère une vraie classe.
Si tu utilises des variables locales à l'instanciation de celle-ci alors le compilateur va simplement rajouter des attributs finaux dans la classe générée et les alimenter au niveau de l'instanciation de celle-ci.
C'est pourquoi les attributs doivent être finaux pour que cela fonctionne.
Le même principe s'applique si tu fais appels à des attributs et/ou méthode de "this". Le compilateur ajoute un attribut final du type de la classe qui génère la classe anonyme et alimente cet attribut au moment de l'instanciation.
Le même principe est également appliqué aux inner class non static !

pas mal Je testerai dès que possible Mais une question qu'est-ce que la méthode setActionCommand() ? La javadoc n'est pas très explicite

Il s'agit d'une simple String permettant d'identifier le comportement du bouton. Par exemple une ActionCommand "delete" pourrait être utilisé pour plusieurs boutons mais un seul listener est nécessaire pour le gérer.

Dans ce cas il ne faudrait pas instancier un action listener pour chaque élément de la boucle :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
ActionListener listener = new ActionListener() {
  public void actionPerformed(ActionEvent e) {
    switch (Integer.parseInt(e.getActionCommand()) {
    }
  }
}
for (int i=0; i < 10; i++) {
  JButton button = new JButton("Button #" + i);
  button.setActionCommand(""+i);
  button.addActionListener(listener);
  myPanel.add(button);
}

[adiGuba]

Salut,

Mais, j'avoue avoir une certaine réticence à déclarer des variables dans des boucles (sans doute parce que j'ai dans le passé utilisé des langages qui interdisent ce cas de figure).

Pourquoi ??????????

Au contraire même c'est plus propre si la variable n'est utilisé que dans la boucle...

Par ailleurs, au niveau de la mémoire comment sont gérées les déclarations dans les boucles ?

Il n'y a aucun soucis avec cela...



a++