Discussion :

[Preez]

Bonjour à tou(te)s!
J'ai pas mal avancé dans mon bouquin de C, et j'ai décidé de réaliser un petit jeu du pendu, sans m'aider du bouquin, pour voir ce qui n'allait pas trop. J'attends donc de vous des critiques constructives, des indications sur les mauvaises habitudes que j'ai pu adopter etc.
pendu.c
Je n'arrive pas à configurer GCC optimalement (même tout court, tout compte fait!), je n'obtiens aucun warnings, rien du tout...
Pourquoi le if de ma while principale (ouch!) s'éxecute deux fois??
Merci à vous, et encore plus si vous prenez le temps de jeter un petit coup d'oeil sur ce bout de code!
Je n'arrive pas encore à indenter correctement, j'ai fais ce que j'ai pu!

[granquet]

ça pars mal

Code :

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

1
2
/* Declaration des variables et tableaux */
char motSecret[10] = "EXACTEMENT";

et les globales, çaymal.
compte les lettres... rajoute une place pour le 0 final... et ça fait combien? (moi j'ai 11)

Code :

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

if(gagne = 1)

manque un = je suppose?

j'ai pas tout lut ... corrige ça, vire tes globales ... et utilise autre chose que scanf ... ça "devrais" marcher.

pour l'indentation, y'as un petit utilitaire balaize qui s'appelle indent, et qui te fait toute l'indentation

[Faiche]

Les variables globales c'est pas bien, essaye de toujours laisser tes variables internes à tes fonctions.

Code :

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

1
2
 
*ptr_gagne = &boolmotSecret[0];

Pas de type, et ne sert a rien il me semble.

break dans un if c'est dur la.

essaye plutot de faire qqch comme

Code :

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

1
2
3
4
5
6
7
8
 
int fgagne()
{
   for(i = 0; i < 10; i++)
       if(boolmotSecret[i] == 0)
           return 0; 
   return 1;
}

En C, FALSE vaut 0 et TRUE != FALSE, donc TRUE peut valoir n'importe quoi sauf 0.

Donc plutot que de tester 200 fois si c'est bien égal à 1 ou pas, considère que tout ce qui n'est pas faux (== 0) est vrai.

[Gnux]

Les variables globales c'est pas bien, essaye de toujours laisser tes variables internes à tes fonctions.

Code :

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

1
2
 
*ptr_gagne = &boolmotSecret[0];

Pas de type, et ne sert a rien il me semble.

C'est une question d'initialisation de pointeur non? Regarde: http://emmanuel-delahaye.developpez.....htm#pointeurs

[Faiche]

C'est une question d'initialisation de pointeur non? Regarde: http://emmanuel-delahaye.developpez.....htm#pointeurs

Je voulais juste dire que dans son code, ce pointeur n'est pas déclaré, juste initialisé, et qu'il n'est utilisé nulle part.

[Gnux]

Je voulais juste dire que dans son code, ce pointeur n'est pas déclaré, juste initialisé, et qu'il n'est utilisé nulle part.

pardon j'ai regardé ça un peu vite mais effectivement. Il est inutile...
Preez: j'utilise une version standard de gcc:

Code :

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

gcc (GCC) 4.1.0 (SUSE Linux)

Tu utilises quelle implémentation sur quel système?
Reposte ton code une fois que tu auras fait les modifs

[Preez]

Merci Gnux pour les warnings! Merci à tous pour vos réponses aussi rapides.
Gnux, pour gcc j'ai -Wall -pedantic et -o, tout comme toi, mais je n'ai toujours aucun warnings. Je n'ai pas -ansi...
J'ai compris pour la fgagne() et TRUE & FALSE, je vais essayer d'arranger ça.
Pour cette histoire de pointeur, j'étais parti sur une autre idée au départ de l'affaire, et j'ai oublié de supprimer ensuite, je le fais tout de suite!

Edit: voilà j'ai suivi toutes vos indications et compris mes erreurs.
Sauf sur un point. Les variables globales, pourquoi "ce n'est pas bien" ou "c'est mal"? C'est quand même le corps du jeu (le motSecret, le caractere etc.).
Mon corps d'affichage et de saisie du caractere s'execute encore et toujours deux fois,

Code :

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

1
2
3
4
 
while(1) /*Boucle principale*/
	{
	    if(!fgagne()) /*Corps du jeu,  (si l'utilisateur n'a pas gagné)*/

Ca ne viendrait pas du while(1)? Pourtant, le programme est en attente lors du scanf, et ne devrait donc s'executer qu'une fois après la pression sur entrée..

[Gnux]

Sauf sur un point. Les variables globales, pourquoi "ce n'est pas bien" ou "c'est mal"? C'est quand même le corps du jeu (le motSecret, le caractere etc.).

http://emmanuel-delahaye.developpez....s.htm#globales pas mal de choses qui te permettront de comprendre ça.

[Emmanuel Delahaye]

Merci Gnux pour les warnings! Merci à tous pour vos réponses aussi rapides.
Gnux, pour gcc j'ai -Wall -pedantic et -o, tout comme toi, mais je n'ai toujours aucun warnings. Je n'ai pas -ansi...

Essaye plutôt ceci :

-Wall -Wextra -O2

Edit: voilà j'ai suivi toutes vos indications et compris mes erreurs.
Sauf sur un point. Les variables globales, pourquoi "ce n'est pas bien" ou "c'est mal"? C'est quand même le corps du jeu (le motSecret, le caractere etc.).

• Parce qu'on ne sait pas comment ni quand ni qui accède à cette variable. Ca rend le code intestable et incompréhensible. On a aucune certitude sur le code.
• En plus, ça crée une dépendance, ce qui le rend non modulaire et non réutilisable.
• Pour finir, l'instance étant unique, ça le rend impropre à la récursion et à l'utilisation dans des threads.

Par défaut, pas de globales, sauf si la raison est clairement établie (pas d'autre choix possible).

Mon corps d'affichage et de saisie du caractere s'execute encore et toujours deux fois,

Cesse une bonne fois pour toutes d'utiliser scanf() (ou apprends à le faire correctement). Les alternatives à base de fgets() ont été maintes fois évoquées...

[Preez]

D'accord! C'est retenu!

manque un = je suppose?

Oui! Et c'est pas faute de me l'avoir dit pourtant, je suis tombé sur beaucoup de mise en garde là-dessus mais bon... Ca deviendra un automatisme plus tard!
Mon bloc d'affichage du mot caché et de lecture du caractère s'execute toujours deux fois.
Pour le Scanf, j'étais avec getchar au début, mais je dois pas m'en servir à merveille, ça fonctionnait pas.

[Emmanuel Delahaye]

pour l'indentation, y'as un petit utilitaire balaize qui s'appelle indent, et qui te fait toute l'indentation

Ou A-Style qui est intégré à Code::Blocks...

[Preez]

Comme compiler j'utilise mingw32-gcc sous Code::Blocks.
Merci Emmanuel pour A-Style, je vais essayer!

Edit: j'ai répondu juste avant Gnux.

[Gnux]

Bonjour à tou(te)s!
Je n'arrive pas à configurer GCC optimalement (même tout court, tout compte fait!), je n'obtiens aucun warnings, rien du tout...
Pourquoi le if de ma while principale (ouch!) s'éxecute deux fois??
Merci à vous, et encore plus si vous prenez le temps de jeter un petit coup d'oeil sur ce bout de code!
Je n'arrive pas encore à indenter correctement, j'ai fais ce que j'ai pu!

Voilà pour les warnings:

Code :

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

1
2
3
4
5
6
7
gcc -Wall -ansi -pedantic /home/gnux/pendu.c -o pendu
/home/anne/pendu.c:9: warning: data definition has no type or storage class
/home/anne/pendu.c:9: warning: type defaults to 'int' in declaration of 'ptr_gagne'
/home/anne/pendu.c: In function 'main'
/home/anne/pendu.c:36: warning: suggest parentheses around assignment used as truth value
/home/anne/pendu.c: In function 'lectureCaractere'
/home/anne/pendu.c:67: warning: ISO C90 forbids mixed declarations and code

Pour l'indentation, c'est une blague ou c'est vrai? Aujourd'hui tout les éditeurs digne de ce nom propose un mode permettant une indentation correcte. Sinon il y a le programme GNU Indent qui peut le faire pour toi.
Dans la fonction fgagne() j'utiliserais plutot la structure switch...case