JEU DE PENDU AVEC SDL, FMOD ET TTF
fabricelepro
Messages postés
21
Date d'inscription
vendredi 7 décembre 2012
Statut
Membre
Dernière intervention
3 janvier 2009
-
5 déc. 2008 à 19:11
Lemng Messages postés 31 Date d'inscription mardi 3 juillet 2007 Statut Membre Dernière intervention 4 mai 2009 - 13 déc. 2008 à 13:27
Lemng Messages postés 31 Date d'inscription mardi 3 juillet 2007 Statut Membre Dernière intervention 4 mai 2009 - 13 déc. 2008 à 13:27
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/48596-jeu-de-pendu-avec-sdl-fmod-et-ttf
https://codes-sources.commentcamarche.net/source/48596-jeu-de-pendu-avec-sdl-fmod-et-ttf
13 déc. 2008 à 13:27
D'autres part si pour changer la liste de mots, on doit modifier le code source cela ne donne pas quelque chose de très souple, je t'encourage à stocker MAX_POSITION dans un fichier(dans un autre fichier ou par exemple dans la première ligne du dico)
11 déc. 2008 à 21:36
Rien qu'en mettant ton pseudo jsui fatigué :p
Bon j'ai quelques remarques:
- ton code est super lisible et très bien commenté, ca c'est es deux plus de ta source
- tes variables et fonctions sont bien nommées aussi (dans le prototype des fonction le nom des variables n'est pas nécessaire, le type suffit)
Quelques remarques sinon :
- dans ta fonction creerMotSecret, cette fonction pourrait retourner le mot, plutot que de prendre en paramètre de sortie.
- tu fais une lecture dna sle fichier mais si le fichier dictionnaire est court et ne contient pas assez d'éléments qu'est-ce qu'il se passe?
- dans ChoixDeLAccueil et dans d'autre, pourquoi "continuer" n'est pas un booléen? plutôt que de mettre un entier a 1 ou 0.
- ton default continuer=1 ne sert a rien caril est déjà a 1
# quitterJeu = jouer(ecran)
# if (quitterJeu == 1)
remplace par
# continuer = jouer(ecran)
avec jouer() qui retourne un booléen.
- dans jouer(), cette fonction pourrait retourner un boolée,
- perdu et gagne devraient aussi etre des booléen
- la variable emplacement trouvé devrait être un tableau dynamique d'entier, tableau du nombre de cas
- dans recupererLettre ton SQL_QUIT ne devrait pas retourner 0 mais null
- ta boucle devrait etre while (1), car tu ne peux jamais quitter la boucle hormis par un return donc ta variable continuer est inutile
- enfin dans verification, au lieu de
# victoire = 0
utilise return false;
En gros tu pourrais privilégier des booléen, eviter des tableaux contenant trop de cases (emplacementTrouve qui devrait avoir le meme nombre de cases que le mot).
Voila ce que je peux dire de beau sur cette source, mais en tout cas continue comme ca
++
5 déc. 2008 à 19:11
merci pour le code j'avais bien besoin d'un comme ceci