Discussion :

[CPI_en_mousse]

Bonjour à tous,

Je souhaiterais qu'un Boolean contenu dans l'objet du TreeSet change d'état lorsque j'essaie d'ajouter un objet identique.

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
 
public class CommuneDto implements Comparable<CommuneDto> {
    private String code;
    private String libelle;
    private Boolean indicActif;
 
@Override
    public boolean equals(Object o) {
        if (o == null || getClass() != o.getClass()){
            return false;
        }
 
        CommuneDto commune = (CommuneDto) o;
        if (this.getCode().equals(commune.getCode()) && this.getLibelle().equals(commune.getLibelle())) {
            return true;
        }
 
        return false;
    }
 
    @Override
    public int hashCode() {
        return code.hashCode() + libelle.hashCode();
    }
 
    /* (non-Javadoc)
     * @see java.lang.Comparable#compareTo(java.lang.Object)
     */
    @Override
    public int compareTo(CommuneDto o) {
        return this.getCode().compareTo(o.getCode());
 
    }

Code :

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

1
2
3
 
Set listeCommune = new TreeSet<CommuneDto>();
listeCommune.add(competenceGeographiqueService.chargerCommune(codeCommune));

Je ne sais pas si je suis très clair, lorsque j'ajoute une commune à mon treeSet et que cette commune existe déjà dans ce treeSet, je voudrais que indicActif soit vrai.
Je ne vois pas comment faire avec un treeSet.
J'avais pensé à inclure la mise à jour du boolean dans les methode hashCode ou equals que j'ai redéfini. Est une bonne idée?
Si c'est pas possible, je passerai par une arrayList mais je devrai gérer les doublons à la main.

Si quelqu'un à une idée sur comment faire....

Merci pour vos réponses

[eulbobo]

J'avais pensé à inclure la mise à jour du boolean dans les methode hashCode ou equals que j'ai redéfini. Est une bonne idée?

Non, pas bonne idée !

Pourquoi ne pas plutôt utiliser HashMap en plus?

Code :

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

1
2
3
4
5
6
7
8
 
Set listeCommune = new TreeSet<CommuneDto>();
CommuneDto myCommune = competenceGeographiqueService.chargerCommune(codeCommune);
if (listeCommune.add(myCommune)){
     mapCommunes.put(codeCommune, myCommune);
} else {
     mapCommunes.get(codeCommune).setIndicActif(Boolean.TRUE);
}

Ca implique juste d'avoir une HashMap<String, CommuneDto> qui stocke les liens vers les instances de commune dans ton TreeSet, et de créer un setter de indicActif


(la méthode add renvoie true seulement si l'élement a été ajouté au Set)

[CPI_en_mousse]

bon au final j'ai fait quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
 
 
        if (!competenceGeographiqueBacking.getSetCommunePersonne().add(communeAjout)) {            
            competenceGeographiqueBacking.getSetCommunePersonne().forEach(commune -> {
                if (commune.equals(communeAjout)) {
                    commune.setIndicActif(Boolean.TRUE);
                }
            });
        }

Merci pour ton aide eulbobo.

[eulbobo]

C'est une solution qui marche aussi !
J'espère juste que tu n'auras pas trop d'éléments dans ton set parce le parcourir en entier à chaque fois quand un seul élément doit être mis à jour, ça peut vite être long.

[CPI_en_mousse]

du coup j'ai optimisé ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
 
        Predicate<CommuneDto> predicate = commune -> commune.equals(communeAjout);
        if (!competenceGeographiqueBacking.getSetCommunePersonne().add(communeAjout)) {
            competenceGeographiqueBacking.getSetCommunePersonne().stream().filter(predicate).forEach(commune -> {
                if (commune.equals(communeAjout)) {
                    commune.setIndicActif(Boolean.TRUE);
                }
            });
        }

[eulbobo]

Y'a pas à dire, c'est beau ce qu'on peut faire en Java8 !