EXPERIENCES ALEATOIRES
jedinos
Messages postés
25
Date d'inscription
vendredi 12 février 2010
Statut
Membre
Dernière intervention
20 juin 2011
-
20 juin 2011 à 18:09
cczerty Messages postés 6 Date d'inscription mardi 22 mars 2011 Statut Membre Dernière intervention 19 mai 2014 - 21 juin 2011 à 08:37
cczerty Messages postés 6 Date d'inscription mardi 22 mars 2011 Statut Membre Dernière intervention 19 mai 2014 - 21 juin 2011 à 08:37
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/53277-experiences-aleatoires
https://codes-sources.commentcamarche.net/source/53277-experiences-aleatoires
21 juin 2011 à 08:37
- pour les tabulations, ça doit être l'espacement d'origine dans visual C++, après c'est sur que mon indentation n'arrange pas grand chose. Et d'ailleurs, j'ai un grose erreur d'indentation au switch, les case son en recul et plein d'autres choses, c'est certainement du à la copie car je n'ai pas ça sur mon code.
- J'ai codé ce programme un peu à l'arrache, au départ il ne devait comporter que le lancer de pile ou face, puis quand j'avais un peu de temps je l'améliorais un peu. D'ailleurs j'ai vraiment hésité a publier ce code sur CodeS-SourceS.
- Pour les variables, effectivement je ne les ai pas toutes initialisées, d'ailleurs, j'initialise la variable doublons en début de programme, mais pas à sa déclaration, je ne sais pas ce qui m'est passé par le tête à ce moment la.
- Quel tableau devrai-je mettre en liste chainées ?? Ceux avec le nom des cartes, je le garderais en tableau car j'accède directement a une case qui n'est pas forcément la première. Même chose pour celui des doublons, ça m'obligerais à faire un while en plus.
Merci de tes remarques qui étaient effectivement pertinentes.
20 juin 2011 à 18:09
C'est pas mal mais j'ai plusieurs remarques à faire, dont quelques une sont assez pertinentes, à mon avis :
- Des tabulations de 8 espaces, c'est une peu abusé et assez méchant pour ceux qui ont des petits écrans (ex : mes parents);
- Les fonctions, ça existe, pour un petit programme comme celui-là, ça va, et encore; prends l’habitude des fonctions (voir http://www.siteduzero.com/membres-294-214198.html#bio );
- Il y a trop de variables (leur nombre serait acceptable si tout ceci était enfermé dans des fonctions), certaines sont initialisées, d'autres non. En passant, mettre "int freq_lancer[6] = {0}" suffit ;) ;
- Tu aurais pu remplacer les tableaux par des liste chaînées (voir http://www.siteduzero.com/tutoriel-3-36245-les-listes-chainees.html , http://www.siteduzero.com/tutoriel-3-66283-les-listes-doublement-chainees-en-langage-c.html et http://www.siteduzero.com/tutoriel-3-69466-notions-de-structures-de-donnees-tableaux-et-listes-chainees.html#ss_part_4 ).
Dans l'ensemble, c'est quand même bien.