Discussion :

[boubz013]

Bonsoir, j'ai un probleme dans mon code, j'essaie de chercher cette erreur mais je m'embrouille avec les conditions dans une boucle voici 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
char mot_secret[6]="livre";
char lettre=0;
int comptant=5;
int i;
 
while(comptant>0)
{
	printf("Entrer la lettre:");
	scanf("%c",&lettre);
	getchar();
 
	for (i=0;i<6;i++)
	{
		if (mot_secret[i]==lettre)
		printf("C'est la %de lettre\n",i+1);
		else  // c'est ici quand j'entre la mauvaise lettre, il repete le printf() 6x c'est normal mais je sais pas comment eviter cela de plus, au lieu d'enlever -1 au comptant et bien ça quitte boucle c'est embettant
		printf("Aucune lettre, vous perdez un tour\n");
		comptant--;
	}
 
}

Voila je vous remercie de votre aide !!

[Obsidian]

Bonsoir,

Lorsque tu écris :

Code C :

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

1
2
3
4
5
		if (mot_secret[i]==lettre)
		printf("C'est la %de lettre\n",i+1);
		else
		printf("Aucune lettre, vous perdez un tour\n");
		comptant--;

Ton programme exécute printf("C'est la lettre"); si ta condition est vraie et printf("Aucune lettre"); si elle est fausse. L'instruction « comptant-- » est donc en dehors de ton test if(), et est donc exécuté dans tous les cas. Si tu veux que « printf("Aucune lettre"); » et « comptant-- » tous les deux soumis à ton else, il faut les mettre dans un bloc. Autrement dit, il manque des accolades :

Code C :

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

1
2
3
4
5
6
if (mot_secret[i]==lettre) printf("C'est la %de lettre\n",i+1);
else 
{
    printf("Aucune lettre, vous perdez un tour\n");
    comptant--;
}

au lieu d'enlever -1 au comptant et bien ça quitte boucle c'est embettant.

En fait, il ne quitte pas la boucle tout de suite, mais il fait six fois « -1 » parce qu'il fait six tours de boucle for();. Et comme « comptant » n'est initialisé qu'à 5 au départ, tu tombes en dessous de zéro et le jeu s'arrête.

[boubz013]

aa bha oui !! -1 ça le quitte ok cependant malgré que j'ai mis les accolades le code bug.

J'avais pensé à mettre for (i=0;i<1;i++) pour que ca evite d'envoyer 6 printf() à la fois je sais pas si c'est judicieux car aprés lorsque j'entre la lettre i de livre et bien il ne la detecte pas il affiche le printf() de else ...


PS: Voila mais je vois pas du tout comment le corriger ... si tu pourai m'eclaicir merci

=>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
printf("Entrer la lettre:");
	scanf("%c",&lettre);
	getchar();
 
	for (i=0;i<sizeof(mot_secret);i++)
	{
		if (mot_secret[i]==lettre){
		printf("C'est la %de lettre\n",i+1);}
	}
	if (!mot_secret[i]==lettre)
    {
        printf("Aucune lettre, vous perdez un tour\n");
		comptant--;
    }

[jeroman]

Code :

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

if ( !mot_secret[i] == lettre )

1) Ouille ! la priorité des opérateurs. ! a une priorité supérieure à ==.
2) Le résultat de l'expression !qqchose vaut 0 ou 1.

Attention également à ne pas confondre sizeof(tableau) et strlen(tableau). Dans le premier cas, c'est la taille du tableau (en bytes), alors que dans le second cas, c'est la longueur de la chaîne, ce qui n'est absolument pas pareil.
Dans ton exemple, ton tableau fait non pas 5 mais 6 bytes (le \0 final est compté), et la chaîne qu'il contient fait 5 bytes.

Ensuite, ton code doit-il trouver toutes les occurrences ou seulement la première ?
1) S'il doit trouver seulement la première, dès qu'il la trouve, tu peux sortir de la boucle une fois le message affiché. Par conséquent, une fois sortis de boucle, on sait que i ne peut pas être égal à strlen(mot_secret). Tu peux t'en servir pour savoir si une occurrence a été trouvée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
for ( i=0 ; i < strlen(mot_secret) ; i++ )
{
	if ( mot_secret[i] == lettre ) {
		printf("C'est la %de lettre\n",i+1);
		break; /* 1ère occurrence trouvée, pas besoin d'aller plus loin */
	}
}
if ( i == strlen(mot_secret) ) /* si la condition est vraie, c'est qu'on a forcément parcouru toute la chaine sans trouver la lettre */
{
	printf("Aucune lettre, vous perdez un tour\n");
	comptant--;
}

2) Si tu dois trouver toutes les occurrences, tu peux te servir d'une variable que tu initialiseras avant la boucle, et dont tu changeras la valeur dès la première occurrence trouvée. Une fois que la boucle est terminée, il te suffira alors de vérifier sa valeur. Si sa valeur n'a pas changé, c'est qu'aucune occurrence n'a été trouvée.

[boubz013]

a d'accord j'ai compris cependant je n'ai pas trés bien saisie le if ( i == strlen(mot_secret) ) .. ça te derange pas de me l'expliquer en details

[Obsidian]

a d'accord j'ai compris cependant je n'ai pas trés bien saisie le if ( i == strlen(mot_secret) ) .. ça te derange pas de me l'expliquer en details

C'est écrit dans son commentaire associé. Relis bien.

Code C :

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

/* si la condition est vraie, c'est qu'on a forcément parcouru toute la chaine sans trouver la lettre */

C'est moins une question de programmation que de conception, à ce stade : il s'agit de décrémenter « comptant » seulement si la lettre n'a pas été trouvée dans le mot-mystère. Et ça, tu ne peux le savoir que si tu as parcouru toute ta chaîne. Étant donné que, pour une longueur l, ta boucle court de 0 à l-1, si i vaut l, c'est que ta boucle est allée jusqu'au bout.

C'est un moyen de savoir, donc, si ta boucle précédente s'est terminée parce qu'elle est arrivée à son terme, ou si on a fait un break.

[boubz013]

a sayer enfin je viens de comprendre merci