Vilain bug ou mauvaise utilisation de ma part ?

Bonjour

Je développe une application avec une IHM, et quelques boutons, rien de très compliqué.
J'avais un soucis, lors d'un clic sur un bouton, l'action générée va vérifier si une fois l'action executée, le bouton doit toujours être enabled. La plupart du temps, d'ailleurs, une fois l'action terminée le bouton doit être disabled.

Seulement, lorsque je cliquais sur le dit-bouton, l'interface gelait complètement, processeur à fond. J'ai réussi à trouver en mode debug qu'une boucle de EventDispatch tournait en boucle...
Lorsque je met en commentaire ma ligne bouton.setEnabled(false), plus aucun problème !
Mais le bouton reste accessible, alors qu'il ne devrait pas l'être.

Mon bouton est un JButton, avec un MouseListener (besoin du survol du bouton), et mon action est codée dans MouseReleased (même erreur dans MouseClicked).


Est-ce que j'ai une erreur de codage qqpart, je fais qqchose qu'il ne faut pas faire ?




Merci

Pierre

L'action du bouton c'est pas avec un MouseListener qu'il faut la faire mais avec un ActionListener ou setAction(...).

Citation:

---

Envoyé par natha

L'action du bouton c'est pas avec un MouseListener qu'il faut la faire mais avec un ActionListener ou setAction(...).

---

C'est fait, j'ai mis mon code dans un ActionListener, relié à mon bouton...
Ca ne change rien au problème ! Ca boucle toujours au même endroit... désolé :?

Salut,


On pourrait voir le code de ton ActionListener ?
Sinon tu as d'autres listener sur ce bouton ?

a++

Citation:

---

Envoyé par adiGuba

Salut,
On pourrait voir le code de ton ActionListener ?
Sinon tu as d'autres listener sur ce bouton ?
a++

---

Je peux essayer de te filer des bouts de code, j'espère que ça ne sera pas trop décousu.

L'IHM comporte plusieurs boutons, tous instances de ActionButton

Le code de leurs ActionListener

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public void actionPerformed(ActionEvent e) { LisQuery newQuery = /* je passe ce passage */; try { LisMain.setQuery(newQuery); LisMain.getFrame().setEnabledActionButtons(); System.out.println("Action terminée"); } catch (ExceptionNoAction e2) { e2.printStackTrace(); } }

---

Explications :
- LisMain est ma classe principale, elle s'occupe aussi de faire le lien avec les données et l'IHM.
- L'action principale de ce bouton c'est le setQuery() (qui peut renvoyer une exception, mais normalement elle n'arrive jamais, je ne l'ai jamais eue)


Le code de setEnabledActionButtons() :

Code:

---

1 2 3 4 5 6

public void setEnabledActionButtons() { for (int i=0;i<actionButtons.length;i++) { actionButtons[i].setEnabledIfAction(); } }

---

Explications :
- actionButtons est un tableau comportant ma liste de boutons, pour chacun je vais voir s'ils sont toujours d'actualité après que l'action ait été effectuée.



Enfin, le setEnabledIfAction(), méthode de mon bouton :

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

public void setEnabledIfAction() { if (query != null) { try { LisQuery Q = query.getQuery(lisAction); ArrayList<Individual> indivs = query.getIndividuals(lisAction); if (indivs.size() > 0) { if (!indivs.equals(LisMain.getMemento().getCurrentQuery().getIndividuals())) { setEnabled(true); } else { setEnabled(false); } } else { setEnabled(false); } } catch (ExceptionNoAction e) { setEnabled(false); } } else { setEnabled(false); } }

---

Explications :
- Ne cherchez pas la signification de toutes les vérifications, en fait il fait soit un setEnabled(true) soit un setEnabled(false), on fonction de plusieurs paramètres
(j'ai beaucoup simplifié le code, normalement on met en mémoire un message expliquant pourquoi le bouton a été disabled).



Sinon pour répondre à ton autre question, oui j'ai aussi mon MouseListener comme j'expliquais dans mon 1er message, avec seulement

Code:

---

1 2 3 4

public void mouseEntered(MouseEvent arg0) { this.setToolTipText(message); }

---

Voilà

Merci


Pierre

A priori il n'y a rien d'anormal. Essayes de logger un peu pour voir s'il n'y a pas une récursion dans les appels qui ferait qu'il y aurait une boucle infini dans les appels de méthode...

a++

Citation:

---

Envoyé par adiGuba

A priori il n'y a rien d'anormal. Essayes de logger un peu pour voir s'il n'y a pas une récursion dans les appels qui ferait qu'il y aurait une boucle infini dans les appels de méthode...
a++

---

Mais je sais très bien où est cette boucle infinie !
Seulement, elle est dans EventDispatchThread.class !

Code:

---

1 2 3 4 5 6 7 8 9 10

void pumpEventsForFilter(int id, Conditional cond, EventFilter filter) { addEventFilter(filter); while (doDispatch && cond.evaluate()) { if (isInterrupted() || !pumpOneEventForFilters(id)) { doDispatch = false; } } removeEventFilter(filter); }

---

Le programme tourne en boucle sur ce while...
Alors par contre, pour arriver à cette méthode, je dois, en débug, après la fin de mon action, passer par une bonne 30aine de méthodes...

Cette boucle infini est normale : c'est la boucle de traitement des événements.

Quelque part dans ton code tu dois générer involontairement un grand nombre d'événements ce qui fait que tu remplis cette pile de traitement...

a++

Ta boucle infinie, c'est pas plutot que tu passes sur le breakpoint donc la tu vas sur eclipse, apres tu rafiches ton ihm, re breakpoint, re ihm...
Je sais pas si je suis clair, mais met un log au niveau où tu penses que c'est en boucle. Et fait tourner en normal, voir si le log s'affiche en boucle.
S'il s'affiche pas en boucle c'est l'erreur que j'essaie de t'expliquer

Citation:

---

Envoyé par ganga

Ta boucle infinie, c'est pas plutot que tu passes sur le breakpoint donc la tu vas sur eclipse, apres tu rafiches ton ihm, re breakpoint, re ihm...
Je sais pas si je suis clair, mais met un log au niveau où tu penses que c'est en boucle. Et fait tourner en normal, voir si le log s'affiche en boucle.
S'il s'affiche pas en boucle c'est l'erreur que j'essaie de t'expliquer

---

Je n'ai pas très bien compris ton explication.
Mais sous eclipse, je place un breakpoint juste après mon action, puis je fais du pas à pas, et je vois nettement le pointeur d'instruction faire des allers-retours. Si par ailleurs j'essaye de cliquer sur "next" (je ne sais plus le nom du bouton, mais ça execute jusqu'à fermer la méthode en cours), ça re-plante !

Citation:

---

Envoyé par adiGuba

Quelque part dans ton code tu dois générer involontairement un grand nombre d'événements ce qui fait que tu remplis cette pile de traitement...

---

OK merci pour cette explication. J'ai cherché tout ce qui pouvait produire des évènements, et j'ai trouvé dans un recoin de mon code une classe copiée sur un autre site, qui permettait de placer une flèche à côté d'un bouton...
Quand je place cette ligne en commentaire, ça fonctionne (mais ça ne change plus la propriété du coup, mais ce n'est pas trop important)

Code:

---

1 2 3 4

public void propertyChange(PropertyChangeEvent evt){ // arrowButton.setEnabled(mainButton.isEnabled()); }

---

Voilà, je pense que je suis sur la bonne voie...


Merci

Citation:

---

Envoyé par ForgetTheNorm

Code:

---

1 2 3 4

public void propertyChange(PropertyChangeEvent evt){ // arrowButton.setEnabled(mainButton.isEnabled()); }

---

Voilà, je pense que je suis sur la bonne voie...

---

Le voilà ton bug !
Faire setEnabled provoque un propertyChange du composant de la propriété "enabled", puis tu refais un setEnabled dedans => boucle infinie !

Voilà, le bug est résolu, j'ai enlevé l'implementation de PropertyChange, et placé ce bout de code :

Code:

---

1 2 3 4 5 6

@Override public void setEnabled(boolean b) { super.setEnabled(b); arrowButton.setEnabled(b); }

---

Merci à vous


Pierre