Récupération index JButton

Bonjour,

J'aimerai récupérer l'index d'un JButton instancié dans une liste[][]
Lorsque je clic dessus avec un ActionEvent j'obtiens (e.getsource()):

Code:

---

javax.swing.JButton[,185,156,37x26,alignmentX=0.0,alignmentY=0.5,border=javax.swing.plaf.BorderUIResource$CompoundBorderUIResource@b07f45d,flags=296,maximumSize=,minimumSize=,preferredSize=,defaultIcon=,disabledIcon=,disabledSelectedIcon=,margin=javax.swing.plaf.InsetsUIResource[top=2,left=14,bottom=2,right=14],paintBorder=true,paintFocus=true,pressedIcon=,rolloverEnabled=true,rolloverIcon=,rolloverSelectedIcon=,selectedIcon=,text= ,defaultCapable=true]

---

Je suppose qu'il y a un autre moyen de récupérer la donnée qui m'intéresse mais je ne vois pas lequel.

Voici comment se déroule la création de mon tableau :

Code:

---

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 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51

public Tableau(String Mode){   JLabel ltemp; tabAtt = new JButton[10][10]; p.setLayout(new GridLayout(11,11)); int posx = 0; int posy = 0; Boolean jpeuxClic = false;   if(Mode.equals("Attaque")){ jpeuxClic = true; }   for(int i = 0; i < 11; i++){   for(int j = 0; j < 11; j++){   posx = i - 1; posy = j - 1;   if(i == 0){ if(j == 0){ ltemp = new JLabel(getLettre(i)); ltemp.setHorizontalAlignment(JLabel.CENTER); p.add(ltemp); } else{ ltemp = new JLabel((j-1) + ""); ltemp.setHorizontalAlignment(JLabel.CENTER); p.add(ltemp); } } else if(j == 0){ ltemp = new JLabel(getLettre(i)); ltemp.setHorizontalAlignment(JLabel.CENTER); p.add(ltemp); } else{ tabAtt[posx][posy] = new JButton(" "); tabAtt[posx][posy].setBackground(Color.WHITE); tabAtt[posx][posy].setEnabled(jpeuxClic); tabAtt[posx][posy].addActionListener(this); p.add(tabAtt[posx][posy]); }     }   }   }

---

Pour plus d'informations complémentaires, je vous invite à rejoindre le sujet principal de mon projet :

http://www.developpez.net/forums/d11...x/#post6350313

Je ne comprends pas vraiment à quoi sert getX() et getY() du getSource() de l'ActionEvent.
Mais ce sont des multiples de 37 et 26 respectivement dans mon cas.
Quelqu'un peut m'éclairer pour ces données ?

En attendant, je peux continuer mon programme en faisant :

Code:

---

1 2 3 4 5 6

public void actionPerformed(ActionEvent e) {   System.out.print(getLettre(((JButton) e.getSource()).getY() / 26)); System.out.println(((JButton) e.getSource()).getX() / 37 - 1);   }

---

C'est moche, mais bon, j'ai pas le choix si je trouve pas autre chose pour m'aider à obtenir ce que je veux :/

C'est, comment dire, plutôt pas très propre comme façon de faire. Et ces valeurs pourraient bien changer selon le look&feel, la machine, la version de java et ainsi de suite...

A la limite, pour une solution bourrine il te suffit de parcourrir le contenu de tabAtt et de comparer avec le retour du e.getSource (en utilisant l'opérateur =, en effet, getSource et un des boutons sont censés pointer vers la même référence, donc le même objet, d'où l'utilisation directe de l'opérateur =).

Au moins tu seras sûr du résultat.

bonjour,

une solution sans doute plus élégante serait de créer ta classe Button dérivée de JButton :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

public class MyButton extends JButton { private int index; public MyButton() { super(); }   public void setIndex(int index) { this.index = index; }   public int getIndex() { return this.index; } }

---

Cette classe te permettra de préciser un paramètre index à ton bouton.


Puis au moment de la création :

Code:

---

1 2 3 4 5 6 7 8 9

tabAtt[posx][posy] = new MyButton();   tabAtt[posx][posy].setIndex(indexBouton); // index du bouton créé   tabAtt[posx][posy].setBackground(Color.WHITE); tabAtt[posx][posy].setEnabled(jpeuxClic); tabAtt[posx][posy].addActionListener(this); p.add(tabAtt[posx][posy]);

---

ensuite pour la gestion de l'événement récupère cet index pour savoir quel bouton a été actionné :

Code:

---

1 2 3 4 5 6

public void actionPerformed(ActionEvent e) { System.out.println(((MyButton)e.getSource()).getIndex());   }

---

Et un switch() case te permettra de réaliser l'action voulue en fonction de l'index du bouton.

Il est vrai que cette solution est vraiment propre.
Il est normal que

Code:

---

1 2	tabAtt[posx][posy] = new MonBouton(); tabAtt[posx][posy]).setIndex(indexBouton);

---

me sort qu'il faut définir un setIndex pour la classe JButton ?

Du coup je suis obligé de faire :

Code:

---

((MonBouton)tabAtt[posx][posy]).setIndex(indexBouton);

---

Pourtant, j'ai bien initialisé le tableau de MonBouton ...

Bah, ouais, tu as défini tabbAtt comme un tableau de JButton, pas comme un tableau de MyButton. Donc il ne te propose que les méthodes de JButton. Donc pas de setIndex. CQFD.

Bien vu ! :mrgreen:

Par contre, je ne vois pas comment je peux comparer 2 tableaux.

Code:

---

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 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46

public void actionPerformed(ActionEvent e) {   MonBouton caseClic = (MonBouton) e.getSource(); MonBouton caseOrdi = tabBatalOrdi[caseClic.getIndex() / 10][caseClic.getIndex() % 10];   int coupOrdi = (int) (Math.random() * 100); while(tabBatalJoueur[coupOrdi / 10][coupOrdi % 10].getBackground().equals(Color.BLUE) || tabBatalJoueur[coupOrdi / 10][coupOrdi % 10].getBackground().equals(Color.ORANGE)){ //L'ordinateur va choisir une case parmi celles non tirées coupOrdi = (int) (Math.random() * 100); } MonBouton caseJoueur = tabBatalJoueur[coupOrdi / 10][coupOrdi % 10];       //Coup par le joueur System.out.println("Votre coup : " + getLettre(caseClic.getIndex() / 10 + 1) + "" + caseClic.getIndex() % 10);       if(caseOrdi.getBackground().equals(Color.CYAN) || caseOrdi.getBackground().equals(Color.ORANGE)){ System.out.println("Etat : Touché !"); ((MonBouton) e.getSource()).setBackground(Color.ORANGE); ((MonBouton) e.getSource()).setEnabled(false); } else{ System.out.println("Etat : A l'eau !"); ((MonBouton) e.getSource()).setBackground(Color.BLUE); ((MonBouton) e.getSource()).setEnabled(false); }   System.out.println("Coup de l'ordinateur : " + getLettre(coupOrdi / 10 + 1) + "" + coupOrdi % 10);   if(caseJoueur.getBackground().equals(Color.CYAN)){ System.out.println("Etat : Touché !"); tabBatalJoueur[coupOrdi / 10][coupOrdi % 10].setBackground(Color.ORANGE); } else{ System.out.println("Etat : A l'eau !"); tabBatalJoueur[coupOrdi / 10][coupOrdi % 10].setBackground(Color.BLUE); }           }

---

Ca se situe tout en bas de ma classe tableau.
les différentes tables utilisées sont des copies de tabAtt (copiées après le placement des mes bateaux).
J'aimerai placer cet ActionPerformed dans la classe qui a créé les panels possédant ces tables afin de pouvoir comparer les tables pour savoir si oui ou non je touche quand je clique sur un bouton.

Du coup, je ne dois plus effectuer d'add.ActionListener sur les boutons je suppose ?
Mais je ne peux pas ajouter d'ActionListener directement sur le tableau dans la classe des panels ...

Je sèche du coup.
A moins que je ne déclare tous les tableaux dans une même instance de classe Tableau.

Quelqu'un a-t-il des suggestions ?