Discussion :

[Bardack]

Bonjour,

voilà une chose bien inquiétante, dans l'application que je développe, il y a donc un JPanel (reagissant au DnD) et on peut droper dedans des objets (Materiel) qui sont en fait des JPanel aussi.
OnClick sur un Materiel, un JPopupMenu apparait et propose de 'delete' le Materiel.

Jusqu' ici tout va bien.

Mais onClick sur l'item delete, rien ne se passe ... Pourtant le code est bon ...

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
 
private void creerPopup() {
        monPopup = new JPopupMenu ();
        liaison = new JMenu("liaison to");
        delete = new JMenuItem ("delete");
        delete.addActionListener (new ActionListener () {
            public void actionPerformed (ActionEvent e) {
                afficheDialogue();
           }
        });
        proprietes = new JMenuItem ("properties");
        monPopup.add(liaison);
        monPopup.add(delete);
        monPopup.add(proprietes);
    }

Donc voilà, onClick le JPopupMenu apparait bien mais le actionPerformed ne fait rien (il ne rentre jamais dedans en fait).

J'ai trouvé une parade (laide), a savoir :
Faire en sorte que ma classe implements ActionListener,
puis remplacer:

Code :

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

1
2
3
4
5
6
 
delete.addActionListener (new ActionListener () {
            public void actionPerformed (ActionEvent e) {
                afficheDialogue();
           }
        });

par

Code :

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

1
2
 
delete.addActionListener (this);

Et ensuite ajouter la methode dans la classe :

Code :

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

1
2
3
4
5
6
 
public void actionPerformed(ActionEvent e) {
        if (e.getActionCommand().compareTo("delete")==0) {
            afficheDialogue();   
        }
    }

C'est a n y rien comprendre. Bon au final ca marche, mais je trouve ca un peu laid et ne voit surtout pas quelle est la raison du non fonctionnement de la premiere facon ...

Serait ce possible que cela soit un bug de JRE ? Ou moi qui ai codé ca n'importe comment?

Merci de reflechir a la question

[Epistocles]

salut,

Code :

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

1
2
3
4
5
public void actionPerformed(ActionEvent e) {
        if (e.getActionCommand().compareTo("delete")==0) {
            afficheDialogue();   
        }
    }

serait avantageusement remplacé par :

Code :

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

if (e.getSource()==this.delete) { ....}

ou, si comparaison itérative :

Code :

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

1
2
3
Object trigger=e.getSource();
if (trigger==this.delete) {....}
if (trigger==this.etc ...

enfin bon, ce n'est qu'un conseil.

sur quel éditeur développes-tu ? le problème persiste-t-il si dans ton code tu remplaces afficheDialogue() par this.afficheDialogue() ?

[Epistocles]

en passant, pourquoi as-tu mis la création du popUp dans une méthode, et non dans le constructeur de ton JPanel ? afin de bien renseigner le compilateur sur la valeur globale de tes champs, je te conseille :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
private void creerPopup() {
        this.monPopup = new JPopupMenu ();
        this.liaison = new JMenu("liaison to");
        this.delete = new JMenuItem ("delete");
        this.delete.addActionListener (new ActionListener () {
            public void actionPerformed (ActionEvent e) {
                this.afficheDialogue();
           }
        });
        this.proprietes = new JMenuItem ("properties");
        this.monPopup.add(this.liaison);
        this.monPopup.add(this.delete);
        this.monPopup.add(this.proprietes);
    }

et d'en prendre l'habitude.

j'espère que les a déclarés en champs globaux de ta classe

[Bardack]

Alors, j' utilise Netbeans comme IDE.
Pour ce qui est des this, bah c'est une habitude (mauvaise) car je suis développeur C++ a la base, donc on s'en fout du this , meme si tu as raison.

la methode creerPopup est appelée dans le constructer, et comme tu peux le voir c est une methode private. En gros c est simplement une facon pour moi de me retrouver si le constructeur est trop conséquent.

Sinon, je viens de modifier en passant en e.getSource , effectivement c est + classieux ... mais cela reste laid par rapport a la facon habituelle de faire.

Pour ce qui est d'ajouter this. pour afficheDialogue(), pas la peine, j' avais tester avec un System.out.println, et meme combat, n affiche rien.

Enfin c est deja plus propre a present , c est deja ca

Merci bien

Si t as une idée pour le comment du pourquoi ca mrche pas autrement, je suis preneur

[Epistocles]

honnêtement, je reste circonspect. peut-être une piste ?

ta méthode actionPerformed est bien publique comme l'exige la syntaxe, mais elle est enfermée dans une méthode privée, peut-être que c'est ça le souci.

essaie de reprendre ta syntaxe originale

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
monPopup = new JPopupMenu ();
        liaison = new JMenu("liaison to");
        delete = new JMenuItem ("delete");
        delete.addActionListener (new ActionListener () {
            public void actionPerformed (ActionEvent e) {
                afficheDialogue();
           }
        });
        proprietes = new JMenuItem ("properties");
        monPopup.add(liaison);
        monPopup.add(delete);
        monPopup.add(proprietes);

mais au lieu de l'enfermer dans une méthode privée, mets-la dans ton constructeur, et dis-nous ce qu'il en est ?

[Bardack]

Meme combat ...

Ayant avancé plus loin dans le projet, je retombe sur le meme probleme plus loin ... mais dans une JDialog ce coup ci ... je tilt vraiment pas :s

[Epistocles]

ajoute ton delete au popup avant d'ajouter ton écouteur dessus

[Bardack]

????

C'est censé marché un brol pareil? Non quand meme? ...

[Epistocles]

non honnêtement c'est une piste perdue, mais je ne vois pas où est le souci avec ta classe anonyme ..

au hasard, vérifie que tu as bien défini

Code :

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

private JMenuItem delete

et pas JMenu ?

[Bardack]

Tout cela est ok , j ai verifier le code 10 fois et des potes aussi ... On comprend pas pourquoi en passant par this ca passe, et pas en passant par le cas habituel ... Incompréhensible ... Bug de l'ami swing peut etre ...