BONNE FETE
cptpingu
Messages postés
3837
Date d'inscription
dimanche 12 décembre 2004
Statut
Modérateur
Dernière intervention
28 mars 2023
-
20 juin 2011 à 17:22
xeo56 Messages postés 92 Date d'inscription mardi 23 novembre 2010 Statut Membre Dernière intervention 15 janvier 2014 - 20 juin 2011 à 18:00
xeo56 Messages postés 92 Date d'inscription mardi 23 novembre 2010 Statut Membre Dernière intervention 15 janvier 2014 - 20 juin 2011 à 18:00
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/53305-bonne-fete
https://codes-sources.commentcamarche.net/source/53305-bonne-fete
20 juin 2011 à 18:00
20 juin 2011 à 17:51
20 juin 2011 à 17:47
20 juin 2011 à 17:22
Quelques remarques:
- sexe<"F", devrait être sexe == "F".
- Évite la commande system, tu perds en portabilité pour des fonctionnalités franchement inutiles
- Si tu veux quitter ton programme, fait le proprement, mais surement pas avec un system("exit") (ici un return aurait été suffisant)
- Il manque un return 0 à la fin de ton main.
- Ne mélange pas C et C++. Soit tu fais du C, soit tu fais C++.
- En C++ on évite les char[] et les char* au profit de std::string.
- Pourquoi faire un Sleep ? Un programme a vocation à donner un résultat rapidement, je ne vois pas l'intérête de le freiner volontairement.
- Évite les "using namespace", voir: http://0217021.free.fr/portfolio/axel.berardino/articles/bon-usage-using-namespace
- Un seul std::cout suffit dans la plupart des cas.
Ex:
std::cout << "salut\n";
std::cout << " toi\n";
Devrait être:
std::cout << "salut\n"
<< " toi\n";
- Répétition de code: Une fonction aurait été plus pratique que de copier coller le coder et changer quelques petits trucs.
- Suite de if: Quand on a plein de "if" à la suite, on utilise généralement un "switch".
Ne pas confondre le forum, ou tu peux poster un code avec comme sujet "Critiquez mon code pour que je m'améliore" et le dépot de source qui sert à montrer de manière pédagogique une technique, ou présenter un programme original (un minimum fourni en taille de code).
A mon avis, ce code aurait plus sa place dans le forum.
Je ne pense pas qu'un admin gardera ta source.