Lemng
Messages postés31Date d'inscriptionmardi 3 juillet 2007StatutMembreDernière intervention 4 mai 2009 13 déc. 2008 à 13:27
Salut, je n'ai fait que survoler ton code, pas le lire attentivement et j'ai été rapidement impressionné par la longueur de ta source, tout ça dans un seul fichier, c'est peut-être beaucoup, je pense que tu aurait du séparer prototypes et définitions et placer touts ces élément dans des fichiers en les regroupant par thèmes.
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)
cs_petifa
Messages postés215Date d'inscriptiondimanche 20 février 2005StatutMembreDernière intervention10 mars 2014 11 déc. 2008 à 21:36
Slt bad_dark_spirit.
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
++
fabricelepro
Messages postés21Date d'inscriptionvendredi 7 décembre 2012StatutMembreDernière intervention 3 janvier 2009 5 déc. 2008 à 19:11
salut
merci pour le code j'avais bien besoin d'un comme ceci
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