Discussion :

[soujava]

slt
voici une méthode qui permet de tester si un element s existe déja dans la liste list:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    public boolean testRep(ArrayList list, String s){
       Boolean b=false;
 
        for (Iterator i = list.iterator(); i.hasNext(); ) {
            if(i.next()==s){
                b=true;
               return (b);
            }
            return(b);
 
                        }
 
    }

et voici le message d'erreur:
Error(101,20): method does not return a value
a votre avis ou est le problème

[vermine]

On dirait que tu ne passes qu'une seule fois dans ton for. Je mettrais le second return hors du for, comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 public boolean testRep(ArrayList list, String s){
       Boolean b=false;
 
        for (Iterator i = list.iterator(); i.hasNext(); ) {
            if(i.next()==s){
                b=true;
               return b;
            }
 
 
                        }
             return b;
    }

[Seb19]

slt
voici une méthode qui permet de tester si un element s existe déja dans la liste list:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    public boolean testRep(ArrayList list, String s){
       Boolean b=false;
 
        for (Iterator i = list.iterator(); i.hasNext(); ) {
            if(i.next()==s){
                b=true;
               return (b);
            }
            return(b);
 
                        }
 
    }

et voici le message d'erreur:
Error(101,20): method does not return a value
a votre avis ou est le problème

Ta méthode doit renvoyer un "boolean" , or ton return renvoie b qui est déclaré en tant que "Boolean" et effectivement, sort le dernier return de la boucle for.

[JoloKossovar]

Tu fais un break pour sortir de ta boucle des que tu as trouvé ton élement et tu renvoit ton boolean.

[fr1man]

Utilise equals et non pas == pour comparer deux chaines.

[Tommy31]

Pas besoin d'affecter la constante booléenne true ou false à une variable. Il vaut mieux la retourner directement.

[jojodu31]

en résumé :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
                public boolean testRep(ArrayList list, String s){
 
		        for (Iterator i = list.iterator(); i.hasNext(); ) {
		            if(i.next().equals(s)){
		               return (true);
		            }
		       }
		        return(false);  
		 }

[vermine]

Je rassemble tout ce qui a été dit pour que tu t'y retrouves mieux. Voici le code:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 public boolean testRep(ArrayList list, String s){
 
        for (Iterator i = list.iterator(); i.hasNext(); ) {
            if(i.next().equals(s)){
               return true;
            }
        }
        return false;
    }

ou bien:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 public boolean testRep(ArrayList list, String s){
        Boolean b = false;
        for (Iterator i = list.iterator(); i.hasNext(); ) {
 
            if(i.next().equals(s)){
               b = true;
               break; //Pour sortir du for
            }
        }
        return b;
    }

[CheryBen]

Je vais peut-être dire une ânerie mais...est-ce que ça ne reviendrait pas à faire

Code :

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

1
2
3
public boolean testRep(ArrayList list, String s){
    return list.contains(s);
}

et dans ce cas autant utiliser la méthode contains au lieu de testRep ???

Vu toutes les réponses je me suis dit c'est pas possible j'ai du rater un truc, ça peut pas être aussi simple