Discussion :

[Bardack]

Bonjour à tous,
comme tout le monde sait, en java, tout est passé par référence, donc a chaque coup qu'on envoie en param. a une méthode un objet, celui - ci est copié.

J'en viens a ma question, voici la situation:
La notion de JPanel intervient, donc de notre ami swing, mais vu que la question est plutot basé java pur et non un problème graphique, je pense plus raisonable de l'avoir posée ici.

Un JPanel (Tableau) sur lequel peut venir se dropper d'autres JPanel (Materiel) (qui sont en fait des images).
La classe Tableau a dans ses attributs:

Code :

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

private static Vector<Materiel> vecteur;

Le but etant de pouvoir travailler avec les 'Materiel' sur le Tableau, et les retrouver facilement.

Jusque là pas de problème, maintenant voyons au niveau du drop sur le tableau comment ca se passe:

Nous sommes dans la classe Tableau donc :

Code :

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

1
2
3
4
5
6
7
8
    /* Faire une copie de l'objet qui vient d'être droppé , drag depuis une 
     *   liste d'images
     */
    Materiel nouveau = (Materiel)droppedMateriel.clone();
    // J'ajoute l'objet a mon vecteur
    vecteur.add (nouveau);
    // Je l'ajoute ensuite au Tableau pour l'afficher sur le JPanel
    this.add (nouveau);

OnClick sur un Materiel, un petit popup apparait et OnClick sur l'item delete de ce popup, j'appelle :

Code :

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

1
2
// Le but etant de supprimer l'objet clické dans le vecteur et le Tableau
Tableau.delElement (this);

Cette methode statique contient:

Code :

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

1
2
3
public static void delElement(Materiel m) {
        vecteur.remove(m);
    }

Un Materiel contient un id unique (static int id) qui est incrémenter a chaque appel du constructeur Materiel, j'ai donc réécrit la methode equals de la classe Materiel de telle facon que :

Code :

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

1
2
3
public boolean equals(Object o) {
        return (this.id == ((Materiel)o).id);
}

remove de vecteur va donc utilisé ma methode equals, et va bien supprimer de mon vecteur le Materiel (pour preuve, il n'est ensuite plus clickable sur le Tableau).
Par contre, il apparait toujours sur le Tableau (JPanel).
Dans cette methode delElement de Tableau, je ne peux pas utiliser this, et donc ne peut pas faire:

Code :

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

this.remove(m);

Vous allez me dire, je n'ai qu'a ajouter une reference vers mon Tableau, bla bla bla, mais c'est bien laid, ca va surcharger mon code et de + c'est pas approprié.
Le but est donc de pouvoir remove le Materiel du Tableau aussi, et je vois trop pas comment faire ...
Le but final etant bien sur de pouvoir se limiter a faire :

Code :

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

vecteur.remove(m);

et donc si plusieurs references sur meme objet, cela aura comme effet de delete le Materiel du Tableau aussi ... Mais notre bon ami Java ne l'entend pas de la sorte.

Merci de reflechir a la question car là ca pose un sacré problème ...

[Sanguko]

Ce qui est surprenant, c'est la methode delElement soit static, alors que la methode qui ajoute le panel ne l'est pas. Ca n'est pas symetrique, ce qui, en général, est un symptome d'une conception inadéquate.
Peux tu donner quelques precisions sur le sujet ?

[Bardack]

Ce qui est surprenant, c'est la methode delElement soit static,

Cette méthode est en statique pour pouvoir l'apeller de ma classe Materiel sans avoir besoin d'une reference vers le tableau dans Materiel.

Pourquoi ne tout simplement pas faire de la méthode delElement, une méthode non-statique ?

Meme remarque que ci dessus

pour résumer, c'est que tu voudrais lorsque tu supprimes un objet de ton tableau, que l'objet "graphique" associé se supprime, c'est ça?

Exactement. En deletant du vecteur, que l'objet soit deleter du JPanel aussi.



Pour tout le monde:

J'ai tout de même essayer en passant en non-static la methode delElement (donc reference dans chaque Materiel vers l'objet de la classe Tableau), mais ca ne change rien au probleme. Vu que ca passe par reference il ne trouve pas l'objet en question et donc ne le delete pas ...

Me voilà un peu perdu ... alors qu'en C++ tout aurait ete comme sur des roulettes ... Faut vraiment que je me remette au java ... et surtout que je solutionne ce probleme

[®om]

Cette méthode est en statique pour pouvoir l'apeller de ma classe Materiel sans avoir besoin d'une reference vers le tableau dans Materiel.

En fait, tu n'as pas trouvé comment récupérer la bonne référence, donc tu n'utilises plus d'instance mais directement une méthode statique? Ca n'est pas correct

Exactement. En deletant du vecteur, que l'objet soit deleter du JPanel aussi.

Pour cela, il faut que tu te crées un listener qui écoute ton modèle (ta liste -ou pour l'instant vector-), et qui prévient la vue quand il y a un changement.

Voici un tuto pour faire cela:
http://rom.developpez.com/java-listeners

[Bardack]

Mmmm ,j 'ai du mal a comprendre j'avoue ...

Je vois pas trop que faire et dans tous les cas ca me bloque plutot pas mal ... :/

[Bardack]

En fait, tu n'as pas trouvé comment récupérer la bonne référence, donc tu n'utilises plus d'instance mais directement une méthode statique? Ca n'est pas correct

Bah c'est pas vraiment ca.
On Drag de ma liste d'image, Drop sur le Tableau, je fais un clone de l'image (Materiel) et ensuite je fais:

Code :

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

1
2
3
 
this.vecteur.add(nouveau);
nouveau.setTableau (this);

Meme probleme, chaque Materiel aura une reference differente sur le Tableau, ... je comprend trop pas

et je fais ca a chaque fois.

[Sanguko]

Cette méthode est en statique pour pouvoir l'apeller de ma classe Materiel sans avoir besoin d'une reference vers le tableau dans Materiel.

Ce qui n'est pas cohérent, c'est que vecteur soit static, mais que tu puisses avoir plusieurs instances de Tableau.
La solution proposée par Elendil_BzH (qui est sans doute la bonne), consiste à rendre toutes les methodes et attributes non statique (meme vecteur), et ajouter une methode du type

Code :

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

1
2
3
4
5
6
7
8
 
   static Tableau tableau = null;
 
   static Tableau getInstance() {
       if (tableau==null)
             tableau = new Tableau();
       return tableau;
   }

Me voilà un peu perdu ... alors qu'en C++ tout aurait ete comme sur des roulettes ... Faut vraiment que je me remette au java ... et surtout que je solutionne ce probleme

Je pense pas que ce soit un probleme de langage, mais plutot de conception.

[Elendil_BzH]

Bonjour,

Pourquoi ne tout simplement pas faire de la méthode delElement, une méthode non-statique ? Vu que tu l'as déclarée en statique, c'est que tu ne dois avoir qu'une seule instance de la classe Tableau à tout moment. Utiliser le design pattern Singleton devrait résoudre ton problème.

[®om]

Ton problème, pour résumer, c'est que tu voudrais lorsque tu supprimes un objet de ton tableau, que l'objet "graphique" associé se supprime, c'est ça?

Ou alors j'ai rien compris...

Au passage, Vector date de l'ancienne API de Collections (avant Java 1.2 !), depuis il y a de nouvelles classes, et en pratique, ce qui se fait c'est d'utiliser List<TonType> en type apparent et ArrayList<TonType> en type réel.