Bonjour à tous.

Je suis actuellement en train de réaliser un projet de jeu et je m'attarde, après mise en place de beaucoup de fonctionnalités, sur un menu propre et basique : 

 - Il y a 6 choix (de 1 à 6) et pour selectionner la branche d'un menu (jouer, règles, charger une partie..) on doit entrer au clavier son choix (donc avec des scanf)

Tout semble simple, donc j'ai réalisé un blindage type : - un while qui exclut tout ce qui n'est pas compris entre 1 et 6 inclus ;

                                                                                - des if qui donnent des instructions pour 1,2,3,4,5,6 ;

Seulement voilà, tout fonctionne, mais un truc me pose problème.

Si on tape, lors du menu, "1a" "1b" "1dfjqjzdklqjsdkq" enfin bref tout ce que vous voulez comme chaine de caractères mais qui comporte un des choix entre 1,2,3,4,5,6 ; on accède quand même à la branche 1,2,3,4,5,6 en console. 

+ Déja je voulais savoir si c'est normal et est ce que mon blindage est bon pour cette action

+ Et sinon, ya t'il une possibilité de contourner ce problème et de faire que seul "1", "2", "3".... ect soit accepté dans le scanf ?

Voici un extrait du programme entier, sur le blindage :

while (choix < 1 || choix > 6)
     {
        printf("Entrez votre selection : ");
        fflush(stdin);
        scanf("%d", &choix);
     }

     switch(choix)
     {
        case 1 :

            // Je vous épargne les commandes car ça n'a rien a voir avec le blindage
            break;

        case 2 :
            // Je vous épargne les commandes car ça n'a rien a voir avec le blindage
            break;

        case 3 :
            // Je vous épargne les commandes car ça n'a rien a voir avec le blindage
            break;

        case 4 :
            // Je vous épargne les commandes car ça n'a rien a voir avec le blindage
            break;

        case 5 :
           // Je vous épargne les commandes car ça n'a rien a voir avec le blindage
            break;
         case 6 :
           // Je vous épargne les commandes car ça n'a rien a voir avec le blindage
            break;
        }
}

Merci aux courageux qui voudront bien m'aider

-
Edité par Clément 2910 3 janvier 2022 à 18:21:15

Ton cas est assez simple.
Utilises  fgets() avec un espace assez long (je suppose que tu ne t'amuseras pas à taper 300 caractères)

https://koor.fr/C/cstdio/fgets.wp

Tu regardes le premier et la longueur
si la longueur n'est pas 1, il y a erreur.

(il faut enlever le caractère '\n' avant, utilises strlen pour savoir la longueur)
Si le caractère n'est pas '1', '2', ... c'est une erreur
tu fais ensuite le calcul  choix = caractere - '0';

-
Edité par PierrotLeFou 3 janvier 2022 à 19:06:34

Hello,

Clément 2910 a écrit:

ok super merci, et comment on enleve le caractère \n ?

Comme ceci par exemple

char str[....];
fgets(str,sizeof(str),stdin);
char *p=strchr(str,'\n');
if(p)
    *p=0;

-
Edité par edgarjacobs 3 janvier 2022 à 19:32:36

Clément 2910 a écrit:

En fait j'ai l'impression que si je veux que ça marche, il ne faut pas que j'utilise "switch"... car on me renvoie l'erreur comme quoi switch n'accepte que les integer....

Evidemment. Si tu as écrit

char str[....];
fgets(str,....)
switch(str) {

ça ne va pas être possible. Par contre, écrire

char str[....];
fgets(str,....);
switch(*str-'0') {    // ou str[0]-'0'
    case 1:
    case 2:
....

ça va être ok.

*str-'0', ça fait quoi ? Ça prend le premier caractère de str ('1', '2', etc) et lui retire la valeur du caractère '0'. Donc, '1'-'0', c'est (en ascii) 49-48, soit 1. Bingo.

Et tu peux aussi écrire (écriture que je préfère)

switch(*str) {
	case '1': 
	case '2':
....

-
Edité par edgarjacobs 3 janvier 2022 à 19:46:42

bah ça fonctionne mais mon programme crashe dès que j'entre une séléction... je comprend pas vraiment d'où ça vient (alors qu'avec le blindage simple marchait bien entendu) :

while (strlen(choix) != 1 && choix != 1 && choix != 2 && choix != 3 && choix != 4 && choix != 5 && choix != 6)
     {
        system("cls");
        affichage_choix();
        gotoligcol(16,70);
        printf("Entrez votre selection : ");
        fflush(stdin);
        fgets(choix,sizeof(choix),stdin);
        char *p=strchr(choix,'\n');
        if(p)
            *p=0;

     }
     system("cls");

     switch(*choix-'0')
     {
        case 1 :

            // Bloc d'instruction...
            // Case 2 case 3 ... ect 

Heuuu.... tu as dû avoir une pelletée de warning pour ta ligne 1. Tu compares une adresse (choix est un pointeur) avec un entier....

Et on ne fflush() pas stdin, cela peut avoir un comportement indéterminé, voir ici.

-
Edité par edgarjacobs 3 janvier 2022 à 20:02:12

Il s'agit d'un programme qui

• pose des questions
• attend une reponse. L'utilisateur tape quelque chose puis [Entrée]

// lire la ligne

char ligne[1000];
fgets(ligne, 1000, stdin;

// essayer de lire un nombre + un caractère

int nombre;
char ch;

int r = sscanf(ligne, "%s%c", &nombre, &ch);

-
Edité par michelbillaud 3 janvier 2022 à 20:02:31

-
Edité par edgarjacobs 3 janvier 2022 à 20:08:42

Merci les gars, j'ai réussi, tout marche bien et j'ai supprimé les warnings sur la ligne.

Je laisse ouvert pendant quelques heures si j'ai de nouveau, un pb, mais sinon merci bcp !

 gotoligcol(16,70);
    printf("Entrez votre selection : ");
    fgets(choix,sizeof(choix),stdin);
    char *p=strchr(choix,'\n');
    if(p)
        *p=0;

     while (strlen(choix) != 1 && *choix != 1 && *choix != 2 && *choix != 3 && *choix != 4 && *choix != 5 && *choix != 6)
     {
        system("cls");
        affichage_choix();
        gotoligcol(16,70);
        printf("Entrez votre selection : ");
        fgets(choix,sizeof(choix),stdin);
        char *p=strchr(choix,'\n');
        if(p)
            *p=0;

     }
     system("cls");

     switch(*choix-'0')
     {
        case 1 :

            system("cls");
            fflush(stdin);
            system("cls");
            partie_en_cours(repere);