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;
    }