Discussion :

[lilli1407]

Bonjour,

J'essaie de recuperer l'index d'un element dans une List et cela ne fonctionne pas. Pouvez vous m'aider ?

Voici mon code :

Code :

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

1
2
3
4
5
6
dossier1 = (DossierResultat) session.getAttribute("dossier");
 
structure = (StructureApplication) req.getSession().getServletContext().getAttribute("structureApplication");
 
int index = structure.getDossiers().indexOf(dossier1);
dossier = (Dossier) structure.getDossiers().get(index);

[adiGuba]

Salut,

et cela ne fonctionne pas.

C'est à dire ???


a++

[lilli1407]

Il me renvoit un index = - 1 alors que l'objet se trouve bien dans la liste

[adiGuba]

As-tu implémenté la méthode equals() de tes objets ?

a++

[lilli1407]

As-tu implémenté la méthode equals() de tes objets ?

a++

non, comment faut il faire ?

[adiGuba]

non, comment faut il faire ?

http://www.developpez.net/forums/sho...4&postcount=58

Par exemple

a++

[lilli1407]

Voila je vient de modifier ma classe dossier comme suit

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
public class Dossier {
 
	private String nom;
	private String description;
	private List familles;
	private ArrayList clefDossier;
 
	public String getDescription() {
		return description;
	}
	public void setDescription(String description) {
		this.description = description;
	}
	public List getFamilles() {
		return familles;
	}
	public void setFamilles(List familles) {
		this.familles = familles;
	}
	public String getNom() {
		return nom;
	}
	public void setNom(String nom) {
		this.nom = nom;
	}
	public ArrayList getClefDossier() {
		return clefDossier;
	}
	public void setClefDossier(ArrayList clefDossier) {
		this.clefDossier = clefDossier;
	}
 
 
	public boolean equals(Object obj) {
        // Vérification de l'égalité des références
        if (obj==this) {
            return true;
        }
 
        // Vérification du type du paramètre
        if (obj instanceof Dossier) {
            // Vérification des valeurs des attributs
            Dossier other = (Dossier) obj;
 
            // Pour les attributs de type objets 
            // on compare dans un premier temps les références 
            if (this.getNom() != other.getNom()) {
                // Si les références ne sont pas identiques
                // on doit en plus utiliser equals()
                if (this.getNom() == null || !this.getNom().equals(other.getNom())) {
                    return false; // les attributs sont différents 
                }
            }
 
            // Si on arrive ici c'est que tous les attributs sont égaux :
            return true;
        }
 
        return false;
    }
 
 
 
 
}

Comment dois-je faire maintenant ?

[adiGuba]

Comment dois-je faire maintenant ?

Ben c'est tout... (sauf qu'il est conseillé d'implémenté hashcode() dans le même temps)

Mais avec une List cela devrait marcher... si ce n'est pas le cas vérifie ce que te renvoi equals() entre ces deux instances...

a++

[lilli1407]

Mais je veux dire comment je l'appelle dans le code que j'ai mis mis au debut ? J'ai rien à changer ?

[adiGuba]

Mais je veux dire comment je l'appelle dans le code que j'ai mis mis au debut ? J'ai rien à changer ?

Non : equals() est utilisé par indexOf() (faut lire la doc ).


a++

[lilli1407]

Non : equals() est utilisé par indexOf() (faut lire la doc ).


a++

mais ca ne marche pas.

Petite precision je suis en version 1.4.2 et donc je n'ai pas le droit à @Override. Est-ce que c'est pour ça ?

Desolée je suis un peu perdu sur ce coup là

[adiGuba]

mais ca ne marche pas.

As-tu vérifié ce que te renvoi equals() entre ces deux instances ???

Petite precision je suis en version 1.4.2 et donc je n'ai pas le droit à @Override. Est-ce que c'est pour ça ?

Non @Override est juste à titre informatif, sa présence ne modifie en rien le code...

a++

[lilli1407]

As-tu vérifié ce que te renvoi equals() entre ces deux instances ???

Comment ça ?

[adiGuba]

Comment ça ?

Code :

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

tonObjet.equals( tonObject2QuidoitEtreIdentique );

a++

[lilli1407]

Code :

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

tonObjet.equals( tonObject2QuidoitEtreIdentique );

a++

Et bien ca semble renvoyer "false" tout le temps!!!! je comprends pas pourquoi!!

Peut-etre avez vous une idee ?

[BainE]

Salut,

DossierResultat herite bien de Dossier ?

et pis je sais ce qu'en pense adiGuba, mais je ferai une methode equals bien plus simple du style

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public boolean equals(Object obj) {
      boolean res = false;
 
        if (obj instanceof Dossier) {
            Dossier other = (Dossier) obj;
 
            if (this.nom.equals( other.nom ) ){
               res = true;
            }
        }
 
        return res;
}

Le fait de tester que l'instance1 soit en fait aussi l'instance2 ne nous interesse pas plus que ca, et pis ca rajoute des tests

[lilli1407]

Merci à tous les deux!!! Je viens de trouver mon erreur : les beans que je comparaient n'etaient pas du meme type (un dossier et un dossierresultat)

[adiGuba]

DossierResultat herite bien de Dossier ?

Bonne remarque : je n'avais pas vu qu'il s'agissait de type différent...

et pis je sais ce qu'en pense adiGuba, mais je ferai une methode equals bien plus simple du style

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public boolean equals(Object obj) {
      boolean res = false;
 
        if (obj instanceof Dossier) {
            Dossier other = (Dossier) obj;
 
            if (this.nom.equals( other.nom ) ){
               res = true;
            }
        }
 
        return res;
}

Le fait de tester que l'instance1 soit en fait aussi l'instance2 ne nous interesse pas plus que ca, et pis ca rajoute des tests

Cela permet d'éviter un instanceof et des vérifications supplémentaires (eventuellement couteuse) lorsqu'on compare la même instance (les comparaisons d'instanse sont bien plus rapide puisque cela équivaut à comparer deux "pointeurs").

Même chose pour les attributs... De plus dans ton cas tu présumes que this.nom n'est jamais null ce qui n'est pas forcément vrai (après cela dépend donc de tes classes).

a++