LABYRINTHE AVEC GESTION DES LISTES BY LIONCEAU

Commentaire de attreid le 18/05/2011 09:22:04

J'ai jeté un oeil à ce que tu as fait. J'aurais quelques petites remarques que tu trouveras, je l'espère, constructives.

- Si tu programmes en C, nomme tes fichiers .c, pas .cpp. Si tu programmes en c++, nomme tes fichiers en .cpp et .hpp.
- N'utilise pas de termes français dans tes noms de variable ou de méthode, en règle générale, ça ne se fait pas en programmation. D'après le fichier .doc que tu as joint, c'est un problème de ton prof, on dirait. Ne prend pas ses mauvaises habitudes. Evite aussi des noms mixtes, comme "cellule_new". De plus, en C, on n'utilise pas le upper camel case - mais d'après le .doc, c'est aussi un problème qui vient de ton prof.
- Si les pipes existent sous Windows, jette un oeil à la fonction istty(). Elle te permet de déterminer si une sortie est dans un terminal ou non. Ca te permettra d'éviter des choses comme "ENVOI_VERS_STDOUT"
- Des méthodes qui renvoient 0 pour dire true, c'est un peu ... inhabituel. En informatique, on utilise généralement une valeur nulle pour dire  faux, et une valeur non-nulle pour dire vrai. De nouveau un problème du côté de ton prof. Ventre saint-gris, tu étudies où ??!
- Essaie d'être constant dans ton indentations. Par endroit, tu utilise un '\t', à d'autres ' ', et à d'autres une*, deux, trois ou quatre espaces. Personnellement, je préfère utiliser '\t', comme ça le lecteur peut choisir la largeur de l'indentation.
- Le code est portable, mais sous windows a ajouté des '\r' dans tes fichiers texte. Si tu ajoutes "case '\r': break;" dans le switch de la ligne 38 de GetLaby.c, alors ça tourne sans problème sous linux.
- Essaie d'entrer 'a' dans le menu principal.

Je dirais que les deux trucs les plus importants sont la langue, et l'extension. Le premier peut rendre la lecture du code pénible, et le second peut simplement empêcher sa compilation.

Je dirais que dans l'ensemble, c'est un bon code, et si, d'après ce que j'ai compris, le but est que tu apprennes à utiliser les listes, je penses qu'il est atteint :-)

*: oui, en typo, espace est un nom féminin :-)