Discussion :

[Meline_]

Bonjour à tous,

Décidément, que d'incompréhensions cette semaine !
J'ai fracturé mon code en fonctions pour que celui-ci soit plus lisible, mais il y a un passage qui quand je le transforme en fonction ne...fonctionne plus

Je vous mets d'abord la version du main lorsque mon bout de code n'est pas transformé :

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
33
34
35
 
public static void main(String[] args) { 
 
      boolean rejouer = true;
      char reponse = ' ';
      while (rejouer == true){
 
        char[]tirage = donnerTirage();
        afficherTirage(tirage);
 
        System.out.print("Proposez votre mot : ");
        char[]mot = Terminal.lireString().toUpperCase().toCharArray();
        afficher(mot);
 
        char[]copie = new char[tirage.length];
        copieCases(tirage, copie, mot);
 
        recapEterreurs(tirage, mot);
 
        char[][]dico = getDictionnaire();
 
        int nbLettres = 0;
        compareCompte(dico, mot, nbLettres);
 
        //là, tout se déroule à merveille//
        while (reponse!='O' && reponse!='N'){
          System.out.println("Voulez-vous rejouer ? O/N");
          reponse = Terminal.lireChar();
          if (reponse=='N'){
            System.out.println("Au revoir !");
            boolean rejouer = false;
          }
        }
      }
    }

et maintenant le code avec la fonction :

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
 
public static void main(String[] args) { 
 
      boolean rejouer = true;
      char reponse = ' ';
      while (rejouer == true){
 
        char[]tirage = donnerTirage();
        afficherTirage(tirage);
 
        System.out.print("Proposez votre mot : ");
        char[]mot = Terminal.lireString().toUpperCase().toCharArray();
        afficher(mot);
 
        char[]copie = new char[tirage.length];
        copieCases(tirage, copie, mot);
 
        recapEterreurs(tirage, mot);
 
        char[][]dico = getDictionnaire();
        int nbLettres = 0;
        compareCompte(dico, mot, nbLettres);
 
        //appel de ma fonction, déclarée en dehors du main//
        nouveau(reponse);
 
      }
    }

Quelqu'un saurait-il me dire encore une fois ce que je fais de mal ? J'ai tout essayé, j'ai tout bougé dans tous les sens mais rien n'y fait, quand je réponds 'N' il m'affiche "Au revoir !" mais repart ensuite sur le tirage au sort du mot et me refait tout le programme

Merci pour vos lumières et bonne fin d'année à tous !

[thelvin]

Hello,

ben, 'faudrait peut-être penser à mettre "rejouer" à "false", tu crois pas ?

[Meline_]

Hello,

ben, 'faudrait peut-être penser à mettre "rejouer" à "false", tu crois pas ?

Salut !

Ben si je le crois, c'est l'une des choses que j'ai essayé de faire.
Accessoirement, je le mets déjà bien à false quand l'utilisateur saisit 'N' donc je vois pas où est le problème vu que dans mon main on ne lance le tirage que tant que rejouer est true.
Je ne peux pas vraiment le mettre à false tout au début de mon main, sinon mon programme ne démarre jamais.
Je ne peux pas non plus le déclarer à false juste avant d'appeler la méthode sinon c'est le 'O' qu'il ne prend plus en compte.
Je ne peux pas le mettre à false après la méthode sinon le 'O' n'est plus utile non plus.

Donc où exactement penses-tu qu'il faille mettre rejouer à false ? Je ne doute pas du fait que je rate quelque chose hein...


J'ai essayé ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public static void nouveau (char reponse){
      boolean rejouer = false;
      while (reponse!='O' && reponse!='N'){
      System.out.println("Voulez-vous rejouer ? O/N");
      reponse = Terminal.lireChar();
      if (reponse=='N'){
        System.out.println("Au revoir !");
        rejouer = false;
      }
      rejouer = false;
    }
    }

et ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public static void nouveau (char reponse){
      boolean rejouer = false;
      while (reponse!='O' && reponse!='N'){
        System.out.println("Voulez-vous rejouer ? O/N");
        reponse = Terminal.lireChar();
        if (reponse=='N'){
          System.out.println("Au revoir !");
          rejouer = false;
      }    
        } 
    }

En gros j'ai essayé de le mettre un peu partout, rien n'y fait.

Et encore une fois mon code de base fonctionne sans problème donc pourquoi j'aurais besoin de modifier quelque chose juste parce que je l'appelle maintenant en tant que méthode ? Pour mes autres méthodes déclarées j'ai fonctionné de la même manière et aucune ne m'a posé problème.

Merci en tout cas de m'avoir répondu.

[Drowan]

Ton problème est que Java passe les paramètres d'une fonction par valeur et non par référence.

Donc la variable qui s'appelle "reponse" au sein de la fonction nouveau() n'est pas la même que celle qui s'appelle "reponse" dans le main.

Lorsque tu modifie reponse dans ta fonction, ça ne modifie pas la valeur de reponse dans le main.

[Meline_]

Ton problème est que Java passe les paramètres d'une fonction par valeur et non par référence.

Donc la variable qui s'appelle "reponse" au sein de la fonction nouveau() n'est pas la même que celle qui s'appelle "reponse" dans le main.

Lorsque tu modifie reponse dans ta fonction, ça ne modifie pas la valeur de reponse dans le main.

Merci Drowan !

Y a tellement de réflexes tout bêtes à avoir

Problème résolu