Discussion :

[Neocid]

Bonjour, j'ai écrit une fonction qui me permet de choisir une option. J'aimerai blinder le programme en permettant à l'utilisateur de faire une erreur de choix et de recommencer. Seulement la boucle que j'ai écrite une fois lancée (utilisateur entre un caractère invalide != (1 || 2 || 3) :p) elle se relance indéfiniement, l'instruction de mon scanf n'est plus prit en compte.

Une idée de mon erreur ? merci d'avance !
ps: Code édité après correction. la boucle fonctionne mais pas la vérification de 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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
void introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
		getchar();
		if (isdigit(i))
		{
 
			if(i==1)
			{
				saisie_repertoire(bla);
				lecture_repertoire(bla,NULL);
			}
			if(i==2)
			{
				credit();
			}
			if(i==3)
			{
				break;
			}
			else
			{
				printf("L'option choisie n'est pas valide, veuillez recommencer");
			}
		}		
		else
		{
			printf("L'option choisie n'est pas valide, veuillez recommencer");
		}
	}while(!((i==1)||(i==2)||(i==3)));
}

[emprex]

Après ton scanf, il me semble qu'il faut manger le retour chariot.
tu peux ecrire juste apres ton scanf
getchar();

[Neocid]

Super merci ça marche.

[mailaka]

D'apres un tuto, mieux vaut ne pas melanger scanf et getchar a priori. Et privilegier fgets pour avoir des entrees fiable si on ne veut pas rentrer dans les arcanes du fonctionnement de scanf.

[emprex]

Lorsqu'on saisit un caractere ou une donnée avec un scanf ou un getchar on doit appuyer sur enter qui se retrouve alors dans le flot en entrée du programme, il sera lu par le scanf ou le getchar ce qui provoque un bug.
On l'élimine donc par un getchar();.

[snake264]

Je pense que ceci résoudra ton problème si j'ai bien compris ce que tu cherche à faire:

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
void introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
			if (i == 1)
			{
				saisie_repertoire(bla);
				lecture_repertoire(bla,NULL);
			}
			else if (i == 2)
			{
				credit();
			}
			else if (i == 3)
			{
				break;
			}
			else
			{
				printf("L'option choisie n'est pas valide, veuillez recommencer\n");
			}
	}while(!((i==1)||(i==2)||(i==3)));
}

[Neocid]

Merci pour cette aide.

j'ai finalement opté pour cette solution, c'est ptete pas très "jolie" mais "c'est moi qui l'ai fait" :p

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
36
 
int introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
		getchar();
 
	}while(! ((i==1)||(i==2)||(i==3)) || (isdigit(i)));
 
		if(i==1)
		{
			saisie_repertoire(bla);
			lecture_repertoire(bla,NULL);
		}
		else if(i==2)
		{
			credit();
		}
		else if(i==3)
		{
			return 0;
		}
 
 
}

[gl]

Je pense que ceci résoudra ton problème si j'ai bien compris ce que tu cherche à faire:

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
void introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
			if (i == 1)
			{
				saisie_repertoire(bla);
				lecture_repertoire(bla,NULL);
			}
			else if (i == 2)
			{
				credit();
			}
			else if (i == 3)
			{
				break;
			}
			else
			{
				printf("L'option choisie n'est pas valide, veuillez recommencer\n");
			}
	}while(!((i==1)||(i==2)||(i==3)));
}

Ça ne résout le problème qu'en partie, la fonction introduction() se comportera à priori comme attendue mais s'il y a une autre saisie utilisateur ailleurs dans le programme, celle-ci peut mal se passer puisqu'il reste des caractères dans le flux stdin (au minimum le \n, potentiellement n'importe quoi si l'utilisateur n'entre pas 1, 2 ou 3)