Discussion :

[Invité]

Alors je vous préviens, je débute donc c'est un programme console...Un jeu tout bête : le jeu génère un nombre aléatoirement qu'il faut trouver. Je l'avais à l'origine créé en BASIC et finalement je l'ai porté sur C++...Le jeu enregistre également les 5 meilleurs scores sur un fichier qui est dans le même dossier que l'exe.

Si vous pouviez me donner un note détaillée (une pour les fonctionnalités, une pour l'optimisation, etc) je vous en serais reconnaissant. Pour ma part je l'ai testé dans tous les sens et fait tester par des amis/famille et à priori il est très bien.
J'en ferais une version fenêtre...quand je saurais le faire.
J'ai l'impression d'avoir été trop zélé sur les commentaires, n'hésitez pas à me dire aussi ce qu'il en est (trop, pas assez, pas assez explicites...).

Merci d'avance.

[oodini]

Mets-nous directement le code.

[Trademark]

Salut,

Tu peux déjà aller lire ceci : http://www.developpez.net/forums/d14...-jeux-mystere/

Les commentaires sont aussi valables pour ton code (en grande partie).

En plus je dirais :

1. Utilise les const référence pour le passage de paramètre.
2. Généralise ton programme pour avoir un nombre quelconque de champion et record, tu auras moins de lignes de code.
3. Même si tu as déjà des fonctions, fait-en plus ! Une pour chaque fonctionnalité.

[Invité]

1. Merci, en fait je ne connaissais pas encore les références constantes quand je l'avais fait
2. "Généraliser mon programme" ? Je ne comprends pas. Et, je laisse volontairement 5 records, je ne veux pas en enregistrer plus je trouve ça inutile. Pour ce jeu précisément en tous cas.
3. Tu veux dire n'avoir que des appels de fonctions de main() ?

[Trademark]

Tu ne dois pas avoir var1, var2, ..., varN car on a ce qu'on appelle des tableaux pour regrouper des données de même type. Utilise vector par exemple.

Tes fonctions sont trop grandes, découpe mieux ton programme.

[Invité]

D'accord. Est-ce que je re-upload mon code quand j'aurais arrangé ça ?