Discussion :

[vain_mich]

Bonjour tout le monde,

J'ai un fichier xml :
...
<error severity="warning" message="messageType1" source="type1"/>
<error severity="warning" message="messageType2" source="type2"/>
<error severity="warning" message="messageType1" source="type1"/>
...

J'ai crée une classe qui serait le reflet d'une ligne du fichier XML avec comme partie privée :

- private String source
- private String message
- private int nbItération

Il existe dans ce fichier xml des lignes qui sont identiques (voir erreur de type1 ci dessus). Je voudrais stocker dans une liste les instances (de la classe ci dessus) pour tout les types de sources (type1, type2...). Cependant lorsque je tomberai sur un "type" que j'ai deja rentrer je voudrais incrémenter de 1 "nbIteration" de l'instance qui correspond au "type" afin de savoir combien de fois apparait ce type de ligne dans le fichier XML. Ce qui donne pour l'exemple ci dessus deux instances dans ma liste :

{type1;messageType1;2}
{type2;messageType2;1}

Comment je peux faire pour savoir avant de rajouter dans ma liste un type si celui existe dejà sans avoir bêtement à parcourir celle-ci car il peut y'avoir 400 ou 500 lignes dans mon fichiers, donc 400 ou 500 type différents????

Merci d'avance

[Benouze]

Bonjour,

D'une manière ou d'une autre il faudra parcourir.
Maintenant, tu peux utiliser une HashMap en mettant le type de source en clef et tu auras ainsi à ta disposition la méthode containsKey qui permet de savoir si une clef est déjà présente ou pas.

[chtig]

Sinon tu peux aussi utiliser n'importe quelle Collection et te baser sur la méthode contains de Collection en redéfinissant les méthodes equals et hashCode de ta classe Ligne...

[vain_mich]

Merci d'avoir répondu aussi rapidement,

peux tu rentrer dans le detail de ton explication.

Merci d'avance.

[chtig]

Oui, je peux ^^
Je te mets un squelette, à toi après de le compléter :

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
 
public class ErrorMessage{
private String source;
private String message;
private int nbIteration;
 
public boolean equals(Object o) {
if !(o instanceof ErrorMessage)
  return false;
ErrorMessage other = (ErrorMessage) o;
return (source==null && other.getMessage()==null) || (source!=null && source.equals(other.getMessage());
}
 
public int hashCode(){
if (source==null)
  return 0;
return source.hashCode();
}
 
public void increment(){
  nbIteration++;
}
}

(ne pas oublier les getters/setters)
et après tu fais des (je ne mets pas de generics je ne sais pas en quelle version tu es)

Code :

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

1
2
3
4
5
6
7
8
9
10
 
ArrayList list = new ArrayList();
ErrorMessage currentLine;
for (...) {
  if list.contains(currentLine) {
    ((ErrorMessage) list.get(list.indexOf(currentLine))).increment();
}
else
  list.add(currentLine);
}

[vain_mich]

Merci chtig,

C'est exactement ce dont j'avais besoin. Cependant je me demande qu'elle est l'interet de redefinir la methode Hashcode puisque l'on redefini deja la méthode equals ? Est ce que c'est parce que celle-ci est utilisée lors de l'appel de la methode list.contain(...) ??

Dans tous les cas merci.

[chtig]

En fait c'est une bonne pratique de redéfinir hashCode quand tu redéfinis equals, parce que hashCode a un contrat de renvoyer le même hashCode pour deux objets qui sont "égaux" au sens de equals.

Sans le faire, j'ai déjà eu de belles surprises, mais de tête je ne saurais plus dire dans quel cas ^^
En fait dans les API Java, pour des raisons de perfos (je suppose), il est souvent codé
if (a.hashCode()==b.hashCode() && a.equals(b)) (pas forcément comme ça, mais c'est l'idée) au lieu de if (a.equals(b))

[vain_mich]

Ok j'ai compris je saurais m'en souvenir.

Merci pour tout,

A bientot, ^^

[chtig]

Pas de souci, n'oublie pas de mettre en résolu :-)