JOUR DE NAISSANCE

Commentaire de fredg19 le 05/12/2011 12:55:58

En réponse à votre message Buno, je trouve mon code parfait comme il l'est.

Commentaire de CptPingu le 05/12/2011 16:28:02 administrateur CS

Bonjour.

Très sympathique comme programme. J'ai tout de même quelques petites remarques.

Remarques techniques:
- C'est un grand classique :), mais évite les "using namespace", voir: http://0217021.free.fr/portfolio/axel.berardino/articles/bon-usage-using-namespace
- Évite aussi les "system", tu perds en portabilité pour une fonctionnalité très limitée.
- Tu ne vérifies pas les erreurs d'entrées, ce qui fait qu'on peut entrer dans une boucle infinie si on tapes quelque chose dont ton programme ne s'attend pas. Ex: "Inscrivez le jour de votre naissance" => 01/01/1900 => boucle infinie :(. Pour gérer ça correctement, utilise la technique décrite sur cette page du forum: http://www.cppfrance.com/forum/sujet-VERIFICATION-ENTREE-CIN_1544419.aspx
- Pense à bien activer tout les flags de warning quand tu codes, en compilant j'ai trouvé pas mal de soucis potentiels, je te les copie colle:
jour.cpp: In function 'void EnregistrerInfoJourNaissance(std::vector<TypeDateNaissance>)':
jour.cpp:488:44: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
jour.cpp: In function 'bool RecupererInfoJourNaissance(std::vector<TypeDateNaissance>&)':
jour.cpp:505:46: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
jour.cpp: In function 'void AfficherDernieresRecherches(const std::vector<TypeDateNaissance>&)':
jour.cpp:523:45: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
jour.cpp: In function 'bool ValiderDateNaissance(TypeDateNaissance&, int&)':
jour.cpp:238:2: warning: control reaches end of non-void function [-Wreturn-type]

En gros, tu mélanges entiers signés et non signés, et dans l'une des fonctions, il manque un return.
- Il y a quelques petites inélégances:
  * Par exemple, un else n'est pas nécessaire après un return (puisque le return est débranchant).
  * Utilise directement return dans tes cases, lorsque tu n'as qu'une valeur à retourner.
Ex:
int DeterminerCodeMoisNaissance(int& p_mois_naissance)
{
  switch(p_mois_naissance)
  {
    case 1: return 1;
    case 2: return 4;
   // etc...
  }
}
  * Inutile de passer un type pod (int, float, char, etc...) en référence si tu ne le modifies pas. Donc int& => int (En plus sizeof(int&) >= sizeof(int)).
  * De même, si tu passes une structure en référence (cette fois la référence est justifiée) mais que tu ne la modifies pas, alors il faut ajouter un "const" devant.
  * Un std::vector devrait normalement être passé en référence (sinon bonjour les copies dans "EnregistrerInfoJourNaissance").
  * Il manque un return 0 à la fin de ton main.

Remarques conceptuelles:
- Pourquoi demander à l'utilisateur s'il veut "commencer" ? Autant commencer directement. S'il veut quitter il le peut. En revanche, à l'usage, c'est assez lourd de devoir dire à chaque fois "oui je veux utilise le programme, sinon je ne l'aurais pas lancé".
- Ne pourrais-tu pas demander la date entière à l'utilisateur ? Tu détectes ensuite le jour, mois et année et s'il celui-ci n'a pas tout entrée, tu lui demandes les informations restantes. Ça serait plus agréable.
- Si l'idée du dessus est peu compliqué à mettre en place, précise au moins le format attendu dans la question (Ex: Jour (DD), Mois, (MM), Année (YYYY)).
- Pour le format du fichier, une ligne par entrée aurait été plus pratique, non ?

Remarques facultatives:
- Si tu utilises vraiment Dev-C++, attention ce projet est abandonné depuis des années, et le compilateur embarqué avec est trèèès vieux. Met à jour le compilateur embarqué ou utilise un IDE plus récent (Visual studio express, Qt Creator, Code::Blocks, ...).
- Essaie de coder en anglais (tu auras des noms plus courts et ça évitera le mélange anglais/français entre ton code et la STL par exemple). Si tu cherches à travailler professionnellement en informatique, tu verras que l'anglais est la langue à utiliser.
- Commente ton code. Par exemple avec doxygen. Voir: http://0217021.free.fr/portfolio/axel.berardino/articles/ecrire-de-bons-commentaires
- Utilise des "assert" dès que tu le peux. Ce sont de véritables barrières de sécurité pour ton code.

En espérant que ces remarques constructives te soient utiles.

Commentaire de fredg19 le 06/12/2011 17:41:51

tout à fais exact Pasokoniidesuka, j'ai appris les bases de C++ dans les salles de cours

Commentaire de CptPingu le 24/01/2012 10:19:19 administrateur CS

>> je crois que ça ne marche qu'avec les chiffres et les nombres, pas avec les caractères
Je ne comprends pas ta question. Un caractère est un nombre, donc pas de souci avec ça. Si tu parles d'une chaîne de caractère, le problème ne se pose pas non plus.