Discussion :

[yolepro]

Bonjour,

Mon problème est le suivant. Je voudrais savoir s'il n'y a pas une maniere plus élégante pour faire le code que vais vous montrer plus loin (je pensais passer par une Map ou un iterator, mais je ne sais pas trop comment m'y prendre).

L'ennoncé :

J'ai deux ArrayList<Competence>.

Cette objet Competence contient un id<String>.

Je veux que les competences contenu dans la premiere list soit effacées de la deuxieme.

Voila ce que je fais :

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
 
//profilSelected est la première liste
//competencesComplete est la deuxième liste
 
            for (int j=0; j< profilSelected.size(); j++) {
 
                Competence compFromProfil= (Competence)profilSelected.get(j);
 
                for (int i=0; i<competencesComplete.size(); i++) {
                    Competence compFromCompl= (Competence)competencesComplete.get(i);
                    if (compFromProfil.getCode().equals(compFromCompl.getCode())) {
                        competencesComplete.remove(i);
                        i--;
                    }
                }
            }

Des idées?

[Janitrix]

Il me semble que ArrayList implement List, et que dans List , il y a une méthode

Code :

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

removeAll(Collection<e> c)

Ainsi, tous les éléments contenu dans la collection c seront supprimés !
L'explication de L'api doc :

removeAll(Collection<?> c)
Removes from this list all of its elements that are contained in the specified collection (optional operation).

[yolepro]

Bonjour Janitrix,

Merci pour ta réponse, effectivement il me semblait bien qu'il manquait une donnée à mon probleme.

Les deux liste contiennent bien des objets Competence mais ces objets ne sont pas strictement les même. Seule la propriete id de l'objet permet de les distinguer ("clé primaire" de la compétence).

[in]

je ne suis pas sur de ce que je vais dire mais ...

si tu faisais implémenter Comparable par tes Objets Compétence et que dans la méthode compareTo que tu redéfinirais tu faisais la comparaison sur l'id, à mon avis le removeAll fonctionnerait comme tu le souhaites ...

[Janitrix]

Autre remarque :

J'ai deux ArrayList<Competence>.[...]

Code :

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

Competence compFromProfil= (Competence)profilSelected.get(j);

As tu vraiment besoin de faire le cast ? Il me semble que non si tu as bien fait ArrayList<Competence>.
Sinon la remarque de in est intéressante, tu devrais la tester.
Bonne chance en tout cas .

[Sniper37]

Autre remarque :

As tu vraiment besoin de faire le cast ? Il me semble que non si tu as bien fait ArrayList<Competence>.
Sinon la remarque de in est intéressante, tu devrais la tester.
Bonne chance en tout cas .

et tu peux aussi utiliser la boucle for améliorée:

Code :

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

1
2
3
for (Competence  c : profilSelected){
...
}

[divxdede]

Bonjour Janitrix,

Merci pour ta réponse, effectivement il me semblait bien qu'il manquait une donnée à mon probleme.

Les deux liste contiennent bien des objets Competence mais ces objets ne sont pas strictement les même. Seule la propriete id de l'objet permet de les distinguer ("clé primaire" de la compétence).

La méthode removeAll(Collection) va utiliser la méthode contains(..) qui utilisera le equals(..)

Si tu surcharges la méthode equals(...) dans ton objet Competence afin de retourner "true" lorsque l'id est le meme alors le tour est joué.
Ensuite si ton objet est comparable, il est conseillé d'adapter la méthode compareTo afin d'être cohérent entre la méthode equals() et compareTo()

[yolepro]

Bonjour et merci a tout le monde pour vos réponses (que je n'ai d'ailleurs pas toutes comprises ).

J'ai finalement choisi la solution très adapté de divxdede qui est de surcharger la methode equals.

Cela fait exactement ce que je souhaite, super !!!

J'ai du coup mis en place un system de Comparable suffisement efficace pour mon utilité.

Cela donne donc cela :

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
 
public class Competence implements Serializable {
 
... blablabla propriete...
 
    private String code = ""; // a la fin il y a bien sur les setter et getter
 
    private static Comparator comparator = null;
 
    static {
        comparator = new Comparator() {
                    public int compare(Object o1, Object o2) {
                        Competence ov1 = (Competence) o1;
                        Competence ov2 = (Competence) o2;
 
                        int ret = 0;
                        ret = (new Integer(ov1.getCode())).compareTo(new Integer(
                                    ov2.getCode()));
 
                        if (ret != 0) {
                            return ret;
                        }
 
                        return ret;
                    }
                };
    }
 
    public static Comparator getComparatorOV() {
        return comparatorOV;
    }
 
    public boolean equals(Object o){
        return ((Competence)o).getCode().equals(this.getCode());
    }
 
... blablabla la suite ...
}

Du coup, je n'ai plus qu'à appeler maListe.removeAll(monAutreListe);

Merci

[divxdede]

il est de bon ton de toujours effectuer ces comparaisons et tests d'égalité suivant ce schéma:

Code :

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

1
2
3
4
5
6
7
public int compare(Object o1, Object o2) {
                         if( o1 == o2 )  return 0;
                         else if( o1 == null ) return -1;
                         else if( o2 == null ) return 1;
 
                         // your business comparaison (o1 and o2 aren't null)
                         // ...

Et

Code :

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

1
2
3
4
5
6
public boolean equals(Object o){
          if( this == o ) return true;
          else if( o == null ) return false;
 
          // your business identity test (o isn't null)
          // ...

Ca t'évitera des NullPointerException intempestifs !!!