Discussion :

[GDMINFO]

Bonjour,
j'ai déjà eu des soucis avec la méthode remove() de la classe LinkedList et grâce à des personnes de ce forum j'ai tout réglé en redéfinissant la méthode equals() dans la classe des éléments de ma liste. Seulement voilà, si à un endroit de mon code tout fonctionne parfaitement, plus loin (alors qu'il s'agit d'une LinkedList contenant des éléments de même type que précédemment, en l'occurence des ModuleTree) j'ai la même erreur que la première fois, à savoir il ne supprime rien du tout.
A quoi peut être dû cette différence pour le moins étrange ?

voici le premier endroit où j'utilise remove() et où tout fonctionne :

ModuleTree Mr2 = TmSauve.MTmod(rI);
ModuleTree Ntemp = (ModuleTree) MtN.clone();
LinkedList EnfantN = (LinkedList) Ntemp.getChildren().clone();
EnfantN.remove(Mr2);

et voici le second qui ne supprime rien de la liste :

ModuleTree MtNTh = Th.MT(N,"P").clone();
ModuleTree pereMtNTh = Th.pere(MtNTh);
LinkedList enfantsMtNThP = (LinkedList) pereMtNTh.getChildren().clone();
enfantsMtNThP.remove(MtNTh);

Il est bien entendu que j'ai vérifié ce que renvoyait chaque sous fonction utilisée. MtNTh contient bien un ModuleTree présent dans la liste.

Merci de votre aide.

[chtig]

As-tu redéfini la méthode hashCode() ? En général, il est conseillé quand on redéfinit equals(Object o) de redéfinir hashCode() pour que deux objets égaux au sens de equals aient le même hashCode...

[®om]

Pourquoi tu fais des clone() à tout va?

[GDMINFO]

chtig : As-tu redéfini la méthode hashCode() ? En général, il est conseillé quand on redéfinit equals(Object o) de redéfinir hashCode() pour que deux objets égaux au sens de equals aient le même hashCode...

Non, j'avoue que je n'ai pas redéfini hashcode, ça me paraissait un peu compliqué... mais je vais essayer.

®om : Pourquoi tu fais des clone() à tout va?

Je pensais que c'était mieux pour éviter les problèmes avec les références... Ca n'est pas utile ?

[GDMINFO]

Redéfinir la méthode hashcode() ne donne rien...

[adiGuba]

Non, j'avoue que je n'ai pas redéfini hashcode, ça me paraissait un peu compliqué... mais je vais essayer.

Les spécifications de equals() et de hashCode() sont fortement lié, et ces deux méthodes doivent toujours être cohérentes...

L'implémentation de hashCode() nest pas très difficile : Pourquoi et comment redéfinir la méthode hashCode() ?

Toutefois cela ne devrait pas avoir d'impact sur les LinkedList...

Je pensais que c'était mieux pour éviter les problèmes avec les références... Ca n'est pas utile ?

Quels problèmes avec les références ???? Clone() sert à dupliquer l'objet en mémoire...




Il faudrait également voir le code de tes méthodes equals() et clone(), ainsi que l'objectif exacte de ce bout de code...

a++

[GDMINFO]

J'étais en train de répondre et puis j'ai essayé d'enlever tous les clone()... ça marche !
Pourquoi ?

Ma méthode clone() est :

public ModuleTree clone(){
ModuleTree clone = new ModuleTree();
clone.setLeaf(this.leaf);
for(Iterator it = this.getChildren().iterator();it.hasNext(){
ModuleTree m = (ModuleTree) it.next();
clone.addChild(m);
}
clone.setLabel(this.getLabel());
clone.representant = (Vertex) this.getRepresentant().clone();

et les attributs de ma classe ModuleTree sont :

boolean leaf = false;
LinkedList children=new LinkedList();
String label="";
Vertex representant=null;

Mais ma plus grande interrogation reste que la suppression d'un ModuleTree d'une LinkedList dans un autre contexte (voir message 1) fonctionne très bien (il y a des clone() aussi)...