Discussion :

[wolfenjava]

Bonjour,

J'aimerais solliciter votre aide pour résoudre un dysfonctionnement de mon code Java. Voici le résultat que je souhaite obtenir :

1. L'utilisateur entre un prénom pour savoir si ce dernier figure dans un tableau (prédéfini dans le code).

2. Si le résultat est positif, le script lui indique la position dans le tableau du prénom.

Voici mon code :

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
import java.io.*;
 
class chap5_recherche {
  public static void main(String[] args) {
    String [] tab={"jean","pierre","paul","jack"};
    int i=0;
    String txt="";
    boolean trouve=false;
    BufferedReader saisie;
 
    saisie=new BufferedReader(new InputStreamReader (System.in));
    try{
      System.out.println("Entrez le prenom recherche :");
      txt=saisie.readLine();
    }
    catch (Exception excp) {
      System.out.println("Erreur");
    }
 
    while(i<tab.length && !trouve) {
    	    if(tab[i]==txt) trouve=true;
    	    i++;
    }
 
    if(trouve) System.out.println("Trouve a la position "+(i-1));
    else System.out.println("Ce nom ne figure pas dans la liste !");
  }
}

Lors de la compilation je n'ai pas d'erreur. Par contre, le script ne récupère pas ce que je saisi et me retourne à chaque fois "Ce nom ne figure pas dans la liste".

Merci d'avance pour votre aide !

[Mides]

Bonjour,

tout d'abord ton if ne fonctionne pas, tu compares une chaine, tu récupères la valeur i en sortie de boucle et non la position dans le tableau du champ trouvé, et le "&& !trouve", à mon avis, ne sert pas à grand chose dans l'état.

Peut être que cela peut t'aider :

Code :

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

1
2
3
4
5
6
7
8
9
10
while (i < tab.length) {
    if (tab[i].equals(txt)){
        trouve = true;
	break;
	}
     i++;
}
 
if (trouve)
    System.out.println("Trouve a la position " + (i));

[dev_2007]

Voila, ton code modifié et qui donne le résultat souhaité

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
 
import java.io.*;
 
class chap5_recherche {
  public static void main(String[] args) {
    String [] tab={"jean","pierre","paul","jack"};
    int i=0;
    String txt="";
    boolean trouve=false;
    BufferedReader saisie;
 
    saisie=new BufferedReader(new InputStreamReader (System.in));
    try{
      System.out.println("Entrez le prenom recherche :");
      txt=saisie.readLine();
    }
    catch (Exception excp) {
      System.out.println("Erreur");
    }
 
    while(i<tab.length && !trouve) {
    	    if(tab[i].equals(txt)){ 
    	    	trouve=true;
    	    break;
    	    }else
    	    	i++;
    }
 
    if(trouve) System.out.println("Trouve a la position "+(i));
    else System.out.println("Ce nom ne figure pas dans la liste !");
  }
}

Le principe est quasiment simple, dans ton while une fois tu trouve l'élément recherché alors tu mets ton boolean à true puiske quitter la boucle puiske ça ne sert à rien de continuer à chercher, d'ou le break.

les tableaux sont indexés à partir de 0 donc ton (i-1) donnera toujours la position - 1 de ton élément dans la tableau si trouvé, d'ou aussi le i au lieu de i-1.

voilou voilou.

[wolfenjava]

Merci pour vos deux réponses ultra rapide ! C'est parfait.

[Mides]

Et si vraiment tu tiens à utiliser "!trouve" dans le test, il te faudra réaffecter la valeur "false" à la variable "trouve" à un moment ou un autre.

Quand au "else" dans la boucle, je ne suis pas trop sûr de sa raison d'être.

[dev_2007]

déjà la variable trouve est bien initialisée au moment de sa déclaration à false

Code :

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

boolean trouve=false;

Donc si le test abouti alors trouve aura comme valeur true si non il sera tout le temps à false.

Pour le "else" je suis d'accord avec toi de sa présence puisque le break fait l'affaire au cas ou on rentre dans la boucle si non on incrémente le i pour passer à l'élément suivant.

[Mides]

Donc si le test abouti alors trouve aura comme valeur true si non il sera tout le temps à false.

Tout à fait, et comme dans l'exemple qu'il donne il ne rentre qu'une seule fois dans la boucle et que la valeur de "trouve" est à "false", nul besoin de tester cette valeur dans la condition "&& !trouve".

Code :

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

1
2
3
4
5
6
7
8
9
  while(i<tab.length) {
     if(tab[i].equals(txt)){ 
    	trouve=true;
    	 break;
       }
    i++;
    }
 
    if(trouve) System.out.println("Trouve a la position "+(i));