Discussion :

[misou83]

Bonjour,

J'ai une classe "UnderGroup" qui contient un vecteur V et un int cpt. J'ai un deuxième vecteur "Base" qui contient des éléments de type "UnderGroup".
Pour éviter d'ajouter plusieur fois le même vecteur V dans le vecteur "Base", je veux tester si V existe déjà dans "Base".
la méthode que j'utilise pour comparer s'appelle Existe:

public boolean Existe(UnderGroup v) { return Base.containsAll(v.getData()); }

dans la classe UnderGroup
public Vector getData() { return V; }

Le problème est que la méthode Existe me renvoie toujours false, même si le vecteur existe déjà dans le vecteur "Base". Quelqu'un saurait il me dire où est mon erreur?

merci!!

[anitshka]

as tu redéfini ta méthoe equals() ou compareTo() je ne sais plus laquelle des 2 pour vérifier l'égalité de tes éléments dans ton vecteur ? Sinon j'ai peur qu'il ne vérifie les adresses de tes objets.

[SEMPERE Benjamin]

as tu redéfini ta méthoe equals() ou compareTo() je ne sais plus laquelle des 2 pour vérifier l'égalité de tes éléments dans ton vecteur ? Sinon j'ai peur qu'il ne vérifie les adresses de tes objets.

c est equal qu il faut implementer, compareTo, c est pour les SortedSet ou SortedMap tels que treeSet et hashSet

[miloux32]

as tu redéfini ta méthoe equals() ou compareTo() je ne sais plus laquelle des 2 pour vérifier l'égalité de tes éléments dans ton vecteur ? Sinon j'ai peur qu'il ne vérifie les adresses de tes objets.

il me semble que c'est equals() ( c'est le meme probleme qu'entre les chaines de caractere )

[anitshka]

as tu redéfini ta méthoe equals() ou compareTo() je ne sais plus laquelle des 2 pour vérifier l'égalité de tes éléments dans ton vecteur ? Sinon j'ai peur qu'il ne vérifie les adresses de tes objets.

c est equal qu il faut implementer, compareTo, c est pour les SortedSet ou SortedMap tels que treeSet et hashSet

En effet confirmation par la javaDoc

public boolean contains(Object o)Returns true if this collection contains the specified element. More formally, returns true if and only if this collection contains at least one element e such that (o==null ? e==null : o.equals(e)).

[TabrisLeFol]

Pour éviter d'ajouter plusieur fois le même vecteur V dans le vecteur "Base", je veux tester si V existe déjà dans "Base".

Les Set le font automatiquement non?
De toute facon il faut eviter d'utiliser les Vector, regarde la FAQ pour les informations pratiquessur les Collections.
Mais oui dans ce cas la aussi faut definir ce que tu appelele "egale" par une nouvelle implementation de equals()

[barksy]

Quoi utiliser alors si les Vecteurs sont a eviter ? ArrayList ?

Edit: c bon c'est dans la FAQ, honte a moi.
Par contre l'arrayList n'est pas thread safe, ne faut t'il pas mieu utiliser Vector alors lors d'utilisaiton en multi-thread ?

[SEMPERE Benjamin]

Quoi utiliser alors si les Vecteurs sont a eviter ? ArrayList ?

Edit: c bon c'est dans la FAQ, honte a moi.
Par contre l'arrayList n'est pas thread safe, ne faut t'il pas mieu utiliser Vector alors lors d'utilisaiton en multi-thread ?

si tout a fait le vector doit etre utilise dans l environnement multi threadé a la place de l ArrayList

[SEMPERE Benjamin]

Pour éviter d'ajouter plusieur fois le même vecteur V dans le vecteur "Base", je veux tester si V existe déjà dans "Base".

Les Set le font automatiquement non?
De toute facon il faut eviter d'utiliser les Vector, regarde la FAQ pour les informations pratiquessur les Collections.
Mais oui dans ce cas la aussi faut definir ce que tu appelele "egale" par une nouvelle implementation de equals()

Les set le font effectivement mais il faut au prealable que les objets que tu ajoutent au set implementent Comparable

[misou83]

Merci pour votre aide!

Je vais essayer comme çà

[SEMPERE Benjamin]

penses au tag resolu

[TabrisLeFol]

petit Tip pour la methode egale, meme si parfois c'est pas respecter meme dans l'API de sun...

Elle est sensée etre reflective, symetrique, transitive, coherente et pour tout parametre null renvoyer false...

En general, son implementation commence par

Code :

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

1
2
3
 
if(this==obj) return true; // meilleure performance
if(obj == null) return false; // repecte les specification

et

Code :

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

if(getClass()!=obj.getClass()) return false;

ou

Code :

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

if(!(obj instanceof MyClass)) return false;

egalement

Code :

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

if(!super.equals(obj)) return false;

peut servir si on connait bien les tests effectuer par la classe parent...

Et avec le jdk 5.0, il faudrait rajouter dans la doc

Code :

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

@Override public boolean equals(Object obj)