Discussion :

[st4nisl4s]

Bonjour a tous,
j'ai une ArrayList<MaClasse>
MaClasse a un attribu id et un attribu name avec les getters et setter qui vont avec
MaClasse a aussi cette methode

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
@Override
	public boolean equals(Object obj){
		System.out.println("Comparaison de 2 objets");
		MaClasse maClasse = (MaClasse) obj;
		if(maClasse.getName().equals(name)){
			System.out.println("ils sont égaux");
			return true;
		}
		else{
			System.out.println("ils sont differants");
			return false;
		}
	}

Mon soucis est que lorsque j'utilise la méthode contains sur mon arrayList celle ci n'execute pas ma méthode equals, le débug n'a rien donné non plus :-(
Si vous avez des pistes je suis preneur

[OButterlin]

On peut voir le code de MaClasse ?

[st4nisl4s]

Voici le code complet de MaClasse, elle est on ne peut plus simple

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
public class MaClasse {
 
	private int id;
	private String name;
 
	public MaClasse(){
 
	}
 
	public MaClasse(String name){
		this.name = name;
	}
 
	public MaClasse(int id, String name) {
		super();
		this.id = id;
		this.name = name;
	}
 
	public int getId() {
		return id;
	}
 
	public void setId(int id) {
		this.id = id;
	}
 
	public String getName() {
		return name;
	}
 
	public void setName(String name) {
		this.name = name;
	}
 
	@Override
	public boolean equals(Object obj){
		System.out.println("Comparaison de 2 objets");
		MaClasse objet= (MaClasse) obj;
		if(objet.getName().equals(name)){
			System.out.println("ils sont égaux");
			return true;
		}
		else{
			System.out.println("ils sont differants");
			return false;
		}
	}
}

Le code de test :

Code :

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

1
2
3
4
5
6
 
Arraylist<MaClasse> maCollection;
//chargement de ma collection grace a Hibernate 
if(maCollection.contains("un text recherché")){
//un traitement qui n'est jamais executé
}

Mon code s'execute sans erreur, le soucis est que le programme n'entre jamais dans le if malgres les bonnes données vu dans le mode debug

[OButterlin]

Une remarque : il se pourrait que "name" soit null et ta méthode "equals" se planterait

Sinon, peux-tu également montrer ton code de test ?

[plawyx]

dans getName et setName, tu as une variable pName qui correspond à rien et donc la méthode "equals" rend toujours false

[st4nisl4s]

Une remarque : il se pourrait que "name" soit null et ta méthode "equals" se planterait

Sinon, peux-tu également montrer ton code de test ?

name ne peut etre null

[OButterlin]

Si, comme ceci

Code :

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

1
2
 
MaClasse c = new MaClasse();

Ton test est à l'envers

Code :

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

1
2
3
4
5

if (maCollection.contains("un text recherché"))
{ 
   //un traitement qui n'a jamais été executé
}

il faudrait

Code :

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

1
2
3
4
5

if ( ! maCollection.contains("un text recherché"))
{
   //un traitement qui n'a jamais été executé
}

(si c'est bien un traitement à faire une seule fois que tu veux)

[st4nisl4s]

merci pour la réponse,
j'avais déjà essayé la solution que tu m'a donné mais cela donne toujours le même résultat, le test est fait comme si j'avais pas ré-écris la methode equals. Je me demandais s'il fallait pas que j'implémente une interface ou que j’hérite d'une classe meme si normalement cela devrait fonctionner sans

[adiGuba]

Salut,


Ce comportement est normal puisque tu ne passes pas un objet de type MaClasse mais un objet de type String :

Code :

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

1
2
3
maCollection.contains("un text recherché")
// Il faut faire :
maCollection.contains(new MaClasse("un text recherché"))

Je suppose que contains() doit utiliser la méthode equals() de l'objet reçu en paramètre...


a++

[thelvin]

Je suppose que contains() doit utiliser la méthode equals() de l'objet reçu en paramètre...

Dans le cas d'une ArrayList, oui. Ça va de soi et c'est ce que dit la doc. (Correction : la doc ne le dit pas mais, par élimination, on peut logiquement conclure qu'il n'existe pas d'autre moyen de faire ce que dit la doc.)

[st4nisl4s]

Merci adiGuba ta solution est la bonne !