Discussion :

[boubz013]

Bonjour, J'essaie de réaliser le pendu, mais je m'oppose à un problème de position, je pense i et j

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
char mot[10];
char mot_secret[10];
char car;
int longu=0, i, j=0, compteur=10;
 
printf("Entrer le mot a deviner:");
scanf("%10s",mot);
system("cls");
 
longu=strlen(mot);
 
for (i=0;i<longu;i++) 
{
	mot[i]=mot_secret[j]; 
	mot_secret[j]='*'; 
	j++;
 
}
mot_secret[j]='\0'; 
 
 
while (compteur>0 && strcmp(mot,mot_secret)!= 0)
{
	printf("Entrer un caractere:");
	getchar();
	scanf("%c",&car);
 
	for (i=0;i<longu;i++)
	{
		if (mot[i]==car) 
		{
			mot_secret[j]=car;
		}
 
	}
	compteur--;
	printf("%s\n",mot_secret); 
}
 
printf("Bravo.\n");

Merci de votre attention

[Invité(e)]

Bonjour,

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 le mot a deviner:");
scanf("%10s",mot);
system("cls");
 
longu=strlen(mot);
 
for (i=0;i<longu;i++) 
{
	mot[i]=mot_secret[j]; 
	mot_secret[j]='*'; 
	j++;
 
}
mot_secret[j]='\0';

Il y a un problème ici : tu demandes à l'utilisateur d'entrer un mot dans la variable 'mot', mais tu remplis 'mot' avec le contenu de 'mot_secret' qui n'est pas initialisé.

On peut écrire simplement :

Code :

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

1
2
3
4
    for (i = 0; i < longu; i++) {
        mot_secret[i] = '*';
    }
    mot_secret[i] = '\0';

et plus loin

Code :

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

1
2
3
4
5
        for (i = 0; i < longu; i++) {
            if (mot[i] == car) {
                mot_secret[i] = car;
            }
        }

Du coup, on a plus besoin de la variable j.

[boubz013]

ah bha oui i ça suffisait !! merci

Cependant j'ai une derniere question si je voulais aprés que l'utilisateur ai entré son mot et bien de mettre dans mot_secret la premiere lettre que contient mot par exemple

salut
->
s****

j'avais pensé à genre mot_secret[0]=*(mot); enfin c'est n'importe quoi

[Invité(e)]

Alors, pour ça, c'est simple :

Code :

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

mot_secret[0] = mot[0];

Mais on aurait aussi pu écrire

Code :

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

*mot_secret = *mot;

et oui, si p est un pointeur sur une valeur, *p permet d'accéder à la valeur en question. *mot permet donc d'accéder à la première lettre de mot. (de même, *(mot+N) est équivalent à mot[N]);

Donc ce que tu as écrit

Code :

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

mot_secret[0] = *(mot)

est correct, mais n'est pas la solution la plus lisible.

[boubz013]

a oui d'accord je vois mieux, désolé de t'avoir dérangé pour si peu..

Bonne soirée

[ok.Idriss]

Salut.

N'oublies pas de mettre en si c'est le cas ...

Sinon, une petite idée d'amélioration pour t'entraîner : Actuellement, tu dois rentrer le mot qu'un deuxième joueur doit essayer de deviner. Tu peut essayer de faire une liste de mots sur un fichier type texte et essayer de faire en sorte que le programme choisisse un mot dans cette liste de manière aléatoire.

Cordialement,
Idriss