LABYRINTHE AVEC GESTION DES LISTES BY LIONCEAU
attreid
Messages postés
18
Date d'inscription
samedi 6 avril 2002
Statut
Membre
Dernière intervention
18 mai 2011
-
18 mai 2011 à 09:22
Actares1456 Messages postés 41 Date d'inscription samedi 13 août 2011 Statut Membre Dernière intervention 21 mai 2012 - 13 août 2011 à 21:58
Actares1456 Messages postés 41 Date d'inscription samedi 13 août 2011 Statut Membre Dernière intervention 21 mai 2012 - 13 août 2011 à 21:58
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/53169-labyrinthe-avec-gestion-des-listes-by-lionceau
https://codes-sources.commentcamarche.net/source/53169-labyrinthe-avec-gestion-des-listes-by-lionceau
13 août 2011 à 21:58
18 mai 2011 à 09:22
- 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 :-)