Discussion :

[weautus]

Bonjour,

Pour un projet que j'ai en Java, je dois vérifier si les éléments d'une Collection, sont bien complet. Je m'explique, j'ai des Livres dans une Collection livresBases. Ces Livres sont composés de titre, auteur, isbn, et disponibilité.

Ce que je voudrais, c'est parcourir ma collection, mais ne pas vérifier le dernier élément, car je sais qu'il est d'office vide (c'est en fait un champ de saisie, donc par définition, il doit être vide). Le problème, c'est que l'intitulé du projet ne me permet pas d'utiliser une ArrayList, et donc un livresBases.get(element).

Voici ma méthode, sans la transformation voulue :

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
    public boolean isInfoComplete() {
        Iterator it=livresBases.iterator();
        if (super.isInfoComplete()) {
            if (dateAcquisition == null) {
                return false;
            }
            for (LivreBase livre : livresBases) {
                if (!livre.isInfoComplete()) {
                    return false;
                }
            }
            return true;
        }
        return false;
    }

J'aimerais donc pouvoir arrêté ma boucle un tour avant ce qu'elle fait à présent. Pourriez-vous m'aider ?

Merci

[thelvin]

Ça m'étonnerait beaucoup que tu sois obligé de mélanger des pommes et des poires dans la même Collection, et que si oui tu sois obligé de faire un truc particulier avec le dernier élément mais sans que la Collection puisse y accéder... Mais voici ce que tu peux faire :

Solution #1 - Garder une propriété qui indique que le dernier élément, qui doit être vide, est le dernier élément qui doit être vide. Comme ça, quand tu tombes dessus, tu ne le vérifies pas.

Solution #2 - Appeler la méthode size() de ta collection pour déterminer combien il y a d'élément. Dans ta boucle, compter combien d'éléments tu as vérifiés. Ne pas vérifier le dernier.

[Rei Ichido]

Sinon, tu déclares un itérateur sans l'utiliser.

C'est un des rares cas où, à part les suggestions de thelvin, qui sont pleines de bon sens, un itérateur peut servir par rapport au foreach : tu peux tester .hasNext après ton next :

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
 
    public boolean isInfoComplete() {
        Iterator<LivreBase> it=livresBases.iterator();
        if (super.isInfoComplete()) {
            if (dateAcquisition == null) {
                return false;
            }
            while(it.hasNext) {
                LivreBase livre = it.next();
                if (it.hasNext() && !livre.isInfoComplete()) {
                    return false;
                }
            }
            return true;
        }
        return false;
    }

[weautus]

Ok merci, c'est ce dont j'avais besoin. J'ai néanmoins encore un problème pour la suite, mais normalement, c'est la méthode dont j'avais besoin.

Merci