Discussion :

[azard]

Bonsoir,

Erreur: Votre programme a levé une exception car il a lu trop de caractères!

-------------------------------------------------------------------------------------------------

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
public class penduu{
    public static void main(String[] args){
 
           char lettre;
           int counter = 0;
           int i=0;
           char[] unmot =new char[14];
           char[] motdeviner = new char[14];
         for  (i = 0; i<counter; i ++){
         motdeviner[i]=' ';
         unmot[i]=' ';}  
       unmot = MOOC.motAuSort();
 
      for  (i = 0; i<14; i ++){
       if  (unmot[i]!=' '){
           counter=counter+1;}
           }
       for  (i = 0; i<counter; i ++){
         motdeviner[i]='-';}
 
     while (motdeviner[i] != unmot[i])   {   
           System.out.println("Entrer une lettre"); 
           lettre = MOOC.readChar();
       for (i=0;i<counter;i++) {
        if (unmot[i] == lettre){
            motdeviner[i] = lettre;}
          } System.out.println(motdeviner);
        } System.out.print("bravo tu as trouvé le mot");
 } 
}

[joel.drigo]

Salut,

Ce serait mieux si tu nous donnais l'exception et la stacktrace de celle-ci.

Tu obtiens probablement une ArrayIndexOutOfBoundException dans cette ligne (surlignée en jaune) :

Code :

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

1
2
3
4
5
6
7
unmot = MOOC.motAuSort();

		for (i = 0; i < 14; i++) {
			if (unmot[i] != ' ') {
				counter = counter + 1;
			}
		}

En effet, il est peu probable que le mot tiré au sort fasse 14 lettres. Il faut toujours parcourir les tableaux en utilisant leur taille :

Code :

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

1
2
3
4
5
6
7
unmot = MOOC.motAuSort();

		for (i = 0; i < unmot.length; i++) {
			if (unmot[i] != ' ') {
				counter = counter + 1;
			}
		}

C'est vrai également pour la boucle suivante (le for sur motdeviner) : il y'a peu de probabilité pour que motdeviner ait la même taille que unmot.

Quant au while while (motdeviner[i] != unmot[i]) {, il plantera à coup sûr. i vallant la taille de unmot lors du premier passage. On évite de réutilisr des variables (ici i) pour éviter ce genre de problèmes.

[azard]

Merci pour vos réponses . Le problème c'est que la boucle While ne s’arrête pas quand le mot est trouvé.
Je crois que la condition (unmot != motdeviner) se concrétise toujours.Le problème persiste.
J'ai tenté ce code mais en vain .

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
public class pendu{
    public static void main(String[] args){
 
           char lettre;
           int i=0;
 
           char[] unmot =new char[14];
           char[] motdeviner = new char[14];
 
           unmot = MOOC.motAuSort();
       for  (i = 0; i<unmot.length; i ++){
         motdeviner[i]='-';
       }System.out.println(motdeviner);
 
     while (motdeviner != unmot)   {   
           System.out.println("Entrer une lettre"); 
           lettre = MOOC.readChar();
       for (i=0;i<unmot.length;i++) {
        if (unmot[i] == lettre){
            motdeviner[i] = lettre;}
          } System.out.println(motdeviner);
        } System.out.print("bravo tu as trouvé le mot");
 } 
}

[joel.drigo]

Merci pour vos réponses . Le problème c'est que la boucle While ne s’arrête pas quand le mot est trouvé.
Je crois que la condition (unmot != motdeviner) se concrétise toujours.Le problème persiste.

Avec != tu compares les références des 2 tableaux. Il s'agit de 2 tableaux différents donc toujours de références différentes, donc la condition est toujours vraie.

Il faut utiliser cette expression !Arrays.equals(unmot, motdeviner);.

[azard]

J'ai essayé ce code et voila ce que le compilateur me dit:
AssertionError: Votre programme a levé une exception java.lang.ArrayIndexOutOfBoundsException: 14 à la ligne 29 de votre fichier


voila le 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
29
30
31
public class pendabc{
    public static void main(String[] args){
 
           char lettre;
           int i=0;
           int counter=0; 
           char[] unmot = new char[14];
           unmot = MOOC.motAuSort();
 
           char[] motdeviner = new char[unmot.length];
 
           for (i = 0; i < unmot.length; i++) {
               if (unmot[i] !=' ') {
                counter = counter + 1;
                 }
                } 
 
       for  (i = 0; i<counter; i ++){
         motdeviner[i]='-';
       }System.out.println(motdeviner);
 
     do {   
           System.out.println("Entrer une lettre"); 
           lettre = MOOC.readChar();
       for (i=0;i<unmot.length;i++) {
        if (unmot[i] == lettre){
            motdeviner[i] = lettre;}
          } System.out.println(motdeviner);
        }while (motdeviner[i]=='-' ) ;
 } 
}

[joel.drigo]

J'ai essayé ce code et voila ce que le compilateur me dit:
AssertionError: Votre programme a levé une exception java.lang.ArrayIndexOutOfBoundsException: 14 à la ligne 29 de votre fichier

Toujours le même problème : une gestion incohérente des index.

Code :

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

1
2
3
4
5
6
7
for (i = 0; i < unmot.length; i++) {
        if (unmot[i] == lettre) {
            motdeviner[i] = lettre;
        }
    }
    System.out.println(motdeviner);
} while (motdeviner[i] == '-');

Le for fait varier i de 0 à longueur de unmot. La longueur de motdeviner doit être de la même que unmot a priori. Donc la condition du while teste un caractère en dehors des bornes.
La condition du while devrait être : tant qu'il reste un '-' dans motdeviner. Mais autant utiliser !Arrays.equals(motdeviner, unmot), sinon il faut que tu écrives une méthode pour tester s'il y a encore un '-' dans le tableau.


PS : indente correctement ton code. C'est quoi cette façon (foireuse) de coller une accolade fermante juste derrière un ; ?