Discussion :

[Roud9]

Bonjour,

J'aimerais déterminer si une chaîne entrée par l'utilisateur contient uniquement les caractères dans le tableau lettresAleatoires. Si une lettre diffère, on rejette ce qu'il a écrit et demande d'entrer une autre chaîne. Voici ce que j'ai écrit jusqu'à maintenant, le code dans la méthode verifierSiValide() est manquant et c'est dans cette méthode que je voudrais faire la vérification.

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
     private final static char[] lettresAleatoires =
    {
        'R', 'B', 'O', 'V', 'J', 'F', 'I', 'N'
    };
 
   private static void demanderCombinaison() throws IOException
    {
        do
        {
            System.out.println("Entrez votre combinaison pour l'essai " + essai
                    + " <enter> pour arrêter");
            CombinaisonEntree = r.readLine();
        } while (CombinaisonEntree.length() != 4);
    }
 
    private static void verifierSiValide()
    {
    }

Merci d'avance

[thelvin]

Sans la bibliothèque commons-lang, c'est un peu longuet :

le seul moyen de savoir si un char est présent dans un tableau de chars, c'est de parcourir le tableau et de s'arrêter quand on a trouvé le char recherché. Si on ne l'a pas trouvé, c'est qu'il n'y est pas.

Pour te simplifier la vie, tu peux écrire une méthode :

Code :

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

1
2
3
4
/** Indique si le tableau donné contient le char donné. */
public static boolean arrayContains(char[] array, char c) {
  // à toi d'implémenter
}

[Roud9]

Je ne comprends pas vraiment comment faire la validation

[MigouW]

Dans une méthode bien particulière, je convertirai ton tableau lettresAleatoires , en une String contenant les lettres de celui-ci => lettresAleatoiresString = String.valueOf(lettresAleatoires); (par exemple).

Puis je parcourerais ta chaine en entrée lettre à lettre. et a chaque lettre j'utiliserai une des fonctions fourni par String.
Soit
lettresAleatoiresString.contains(lettre); qui te renverra un booléen
ou
lettresAleatoiresString.indexOf(lettre); qui te renverra la position ou -1 si la lettre n'est pas trouvée.

[thelvin]

Je ne comprends pas vraiment comment faire la validation

Personnellement je ne réponds pas aux "vous voulez bien faire mon travail s'il vous plaît ?"

Montre-nous ce que tu as essayé, comment tu penses que tu pourrais faire mais que tu trouves pas en Java, bref, d'une manière générale, montre-nous que tu bosses et on t'aidera à te dépatouiller.

[Roud9]

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
private static void demanderCombinaison() throws IOException {
          System.out.print("Entrez votre code : ");
          String combinaison = scanner.readLine();
          boolean end = false;
          for (int i = 0; i < combinaison.length; i++) {
               end = false;
               for (int a = 0; a < lettresAleatoires.length && end == false; a++) {
                    if (combinaison.charAt(i) == lettresAleatoires[a]) {
                          end = true;    
                    }
               }
 
               if (end == false) {
                    System.out.println("Erreur");
                    demanderCombinaison();
               }
          }
 
          System.out.println("Code OK.");
     }

[thelvin]

La raison pour laquelle ça ne fait pas ce que tu veux, c'est que, quand il y a une lettre invalide, tu rappelles un nouveau demanderCombinaison() alors que tu es encore dans ta boucle de vérification.
Il faut quitter la boucle d'abord. En fait, le mieux serait que demanderCombinaison() ne s'appelle pas elle-même. Il faudrait séparer le fait de faire taper un truc et vérifier si c'est valide, du fait d'afficher si c'est valide ou pas, et de recommencer si ça ne l'est pas.


Sinon, elle était pas mal ton idée de faire une méthode verifierSiValide(). Seulement, il faut qu'elle renvoie la réponse, boolean, et qu'elle prenne en paramètre la combinaison à vérifier, String.

Code :

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

1
2
3
private static boolean verifierSiValide(String combinaison) {
  // faire return false si c'est pas valide, ou return true si c'est valide.
}

Autres remarques :
- pour quitter une boucle, il y a l'instruction

Code :

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

break;

La boucle cesse immédiatement de s'exécuter, et le programme reprend à la première instruction après la boucle. S'il y a plusieurs boucles imbriquées, break; quitte la boucle la plus imbriquée, celle contenue par toutes les autres.

- l'instruction

Code :

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

return valeurDeRetour;

quitte immédiatement la méthode en cours, ignorant les instructions suivantes. Faire un return dans une boucle est donc un moyen de quitter la boucle (et la méhode) dès qu'on connaît la réponse.