une méthode qui retourne une valeur

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

Code:

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

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

Code:

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

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

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.

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

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

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

en résumé :

Code:

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

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

Code:

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:

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

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

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 :P