References sur meme objet

Version imprimable

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:

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:

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:

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

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:

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:

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:

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 ...

21/02/2007, 11h31
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 ?
21/02/2007, 11h35
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.
21/02/2007, 11h50
®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.
21/02/2007, 11h56
Bardack

Citation:

Envoyé par Sanguko

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.

Citation:

Envoyé par Elendil_BzH

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

Meme remarque que ci dessus

Citation:

Envoyé par ®om

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 :|
21/02/2007, 12h13
®om

Citation:

Envoyé par Bardack

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 :lol:

Citation:

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
21/02/2007, 12h16
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 ... :/
21/02/2007, 12h18
Sanguko
Citation:

Envoyé par Bardack

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:

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

Envoyé par Bardack

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.
21/02/2007, 12h19
Bardack
Citation:

Envoyé par ®om

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:

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.
21/02/2007, 12h23
Bardack

Bah c est le meme probleme alors Sanguko.
La classe Materiel n' a pas de reference vers Tableau. Et quand bien meme elle en aurait une, les references serait diff d'un Materiel a l autre (donc taille des vecteurs differents et tout ) ...

Mais effectivement en passant par ce que tu propose, ca peut aller mieux ... mais donc toujours passer par getInstance plutot qu'apeller tableau?
Je vois pas trop quand utiliser cette methode en fait
21/02/2007, 12h32
Bardack

Ok , j 'ai piger l'histoire de getInstance.

Dans tout mon code donc je n' ai plus nul part d attribut de type tableau, je passe toujours par Tableau.getInstance.

Et effectivement ca marche, ca delete bien du vecteur comme avant. Mais ca reste toujours dans le JPanel ...

Au moins maintenant c est cohérent :) , mais va tjr pas :(
21/02/2007, 12h49
Bardack
Chaque matériel a un ID (static cpt incrémenté a chaque fois le constructeur Materiel est apellé (ou methode clone)).

J'ajoute 3 materiels dans le Vector et affiche les id.
Je parcoure aussi les Components du JPanel et affiche les id de cette facon:
Code:

1 2 3 for (int i = 0; i< this.getComponents().length; i++) System.out.println(((Materiel)this.getComponents()[i]).getId());
Les id sont identiques donc c'est bien les memes objets. Donc c'est censé etre correct.

Mais dans la methode delElement de la classe Tableau,
Code:

1 2 3 4 5 6 // Ca ca marche vecteur.remove(m); // Ca ca marche pas this.remove(m);
Pourtant ma methode equals de Materiel donne :
Code:

1 2 3 4 public boolean equals(Object o) { return (this.id == ((Materiel)o).id); }
Donc c'est censé fonctionner. Comprend vraiment pas là :s , j approche mais non :)
21/02/2007, 12h52
®om

Peux-tu nous expliquer ce que représente Tableau, Matériel, etc... (les classes utiles), et pour chacune, nous dire si c'est une classe du modèle ou une classe de la vue.
21/02/2007, 12h56
Bardack

Donc il y a une JFrame, dans laquelle il y a :
a gauche: Liste extends JList, qui est composée d'images (étant en fait des Materiel extends JPanel).

Materiel est une classe mere, chaque image differente etant une fille extends Materiel.

a droite: Tableau extends JPanel.

Voilà pour + de clarté.

Le probleme c est que dans un projet comme celui ci, je ne vois pas du tout comment implémenter le principe du MVC ...

Pour ce qui est du problème:

la méthode remove de Vector passe par equals de Materiel (que j ai redefinie), par contre remove de this (Tableau), ne passe pas par equals. Logique donc qu'il ne trouve pas d'équivalence. Je pense devoir etre obligé de passer par un for et tester l'id et remove ensuite ... a tester

Effectivement cela marche en faisant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public void delElement(Materiel m) {
        vecteur.remove(m);
        int i = 0;
        boolean find = false;
        while (!find && i<this.getComponents().length) {
            if ((((Materiel)this.getComponents()[i]).getId()) == m.getId()) {
                this.remove(i);
                find = true;
            }
            i++;
        }
        this.repaint();
    }

21/02/2007, 13h02
®om

Citation:

Envoyé par Bardack

Donc il y a une JFrame, dans laquelle il y a :
a gauche: Liste extends JList, qui est composée d'images (étant en fait des Materiel extends JPanel).

Materiel est une classe mere, chaque image differente etant une fille extends Materiel.

a droite: Tableau extends JPanel.

Voilà pour + de clarté.

Le probleme c est que dans un projet comme celui ci, je ne vois pas du tout comment implémenter le principe du MVC ...

Pour ce qui est du problème:

la méthode remove de Vector passe par equals de Materiel (que j ai redefinie), par contre remove de this (Tableau), ne passe pas par equals. Logique donc qu'il ne trouve pas d'équivalence. Je pense devoir etre obligé de passer par un for et tester l'id et remove ensuite ... a tester

Bon, pour faire simple (même si on pourrait encore rajouter un niveau d'abstraction en créeant tes propres listeners), il suffit que tu fasses une classe étendant AbstractListModel, dans laquel tu définis getSize() et getElementAt(int) (ayant un attribut List<Materiel> et renvoyant les éléments Matériel correspondant aux bons indices), et définissant également une méthode remove(int), et quand tu fais un remove, tu le transmets à ta List<Materiel>, et tu fais également un appel à fireIntervalRemoved pour "prévenir" ta JList dans ta vue qu'un élément a été supprimé.

Citation:

Envoyé par Bardack

Effectivement cela marche en faisant :

Code:

1
2
3
4
5
6
7
8
9
 
public void delElement(Materiel m) {
        vecteur.remove(m);
        for (int i = 0; i< this.getComponents().length; i++) {
            if ((((Materiel)this.getComponents()[i]).getId()) == m.getId())
                this.remove(i);
        }
        this.repaint();
    }

Mais la résolution du problème n'est pas dans le bon sens, c'est le modèle qui doit prévenir la (ou les) vues.

21/02/2007, 13h58
Bardack

En gros grace au mvc, plus de this.repaint a ce niveau.

Je vais voir ca ;)

Grand merci d'avance pour votre aide
21/02/2007, 14h07
®om

Citation:

Envoyé par Bardack

En gros grace au mvc, plus de this.repaint a ce niveau.

Je vais voir ca ;)

Grand merci d'avance pour votre aide

Et au lieu de créer des JPanel pour chaque image, si ces images sont associées à un autre type (par exemple, si ton image représente un objet de type MonObjet), il vaut mieux utiliser un ListCellRenderer, ça gère pas mal de choses à ta place (le cache, etc...). Il faut bien sûr être capable d'associer une image à MonObjet, par exemple par une méthode quelque part Image getImage(MonObjet);
Dans ce cas getElementAt(int) devra renvoyer un MonObjet :)
21/02/2007, 14h29
Bardack

Oki merci, je regarde a ca en ce moment :)

Enfin là je regarde pour mvc, j essaie de l'implémenter comme je l'ai appris et wow, cho ... très ...

Pour ce qui est de ListCellRenderer je le fais déja.

Dans le constructeur de la Liste

Code:

1
2
 
this.setCellRenderer(new CustomCellRenderer());

Et ensuite:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
class CustomCellRenderer implements ListCellRenderer {
        public Component getListCellRendererComponent (JList list, Object value, int index,
                                                       boolean isSelected,boolean cellHasFocus) {
            Component component = (Component)value;
            if (isSelected)
                component.setBackground(Color.LIGHT_GRAY);
            else 
                component.setBackground(null);
            return component;
        }
   }

Le problème est que si les images n'ont pas la meme taille, ca s' adapte a la plus grande image et donc une image plus petite sera de la taille de la plus grande image ... dommage