[debutant] Premier programme en C++, qu'en pensez-vous?
Le
Beware
Bonjour,
J'ai depuis une grosse semaine commencé à apprendre le C++. Je
l'apprends de manière autonome (ce qui n'est pas totalement une
excuse). Pour ce fait, je suis les tutos pour C++ du site du zero.
J'ai donc utilisé leur exemple de (trés trés) petit RPG, mais qui me
permet de manipuler certains concept de base en C++.
Pour en revenir donc à ce message, je voudrais demander aux
connaisseurs qui peuvent et surtout qui veulent si il pouvait jeter un
oeil à mon code pour me dire tout ce qu'il ne va pas et que par
conséquent je devrais améliorer (ou carrément changer :) )
Merci à eux.
L'ensemble des fichiers sont disponibles à cette adresse :
http://beware007.free.fr/Projet_C++/rpg/
Au revoir
PS : Je précise que j'ai codé sous C::B et sous Linux.
J'ai depuis une grosse semaine commencé à apprendre le C++. Je
l'apprends de manière autonome (ce qui n'est pas totalement une
excuse). Pour ce fait, je suis les tutos pour C++ du site du zero.
J'ai donc utilisé leur exemple de (trés trés) petit RPG, mais qui me
permet de manipuler certains concept de base en C++.
Pour en revenir donc à ce message, je voudrais demander aux
connaisseurs qui peuvent et surtout qui veulent si il pouvait jeter un
oeil à mon code pour me dire tout ce qu'il ne va pas et que par
conséquent je devrais améliorer (ou carrément changer :) )
Merci à eux.
L'ensemble des fichiers sont disponibles à cette adresse :
http://beware007.free.fr/Projet_C++/rpg/
Au revoir
PS : Je précise que j'ai codé sous C::B et sous Linux.
Poser une question
Le constructeur de copie et l'opérateur d'affectation de Combat (ceux
générés par défaut), ne sont pas bons.
Plusieurs solutions :
- rend Combat non-copyable
- n'utilise pas de pointeurs comme membres (puisque de toutes façons,
tu utilises des pointeurs pour rien)
- redéfinis les.
Pareil pour Personnage.
La manière dont tu gères la durée de vie de l'argument dans
Personnage::setArme(Arme*) est aussi plutôt une mauvaise idée, et
assez dangereux finalement, si tu veux mon avis.
Tu ferais mieux de virer tous ces pointeurs...
Je ne connais pas de bon tutoriel c++ mais il y a aussi celui de
devellopez qui semble assez fourni (bien qu'il prenne aussi l'approche
C pour enseigner le C++).
http://cpp.developpez.com/cours/
Du point de vue programmation pure:
- utilises des const ou des enums plutôt que des defines
- tu fais un delete dans le destructeur de Personnage alors que
m_arme n'a pas été positionné.
- tu fais des new non nécessaire (par exemple combat1 dans main.
--
Michael
Non, il ne peut y en avoir par définition.
C'est nul (et c'est un euphémisme) et ce site est malsain (cf tests cmcm et
autres)
Alors arrête de suite.
C'est du niveau 12 ans et rempli de conneries (c'est écrit par des
étudiants, si, si (!!!)... qui n'ont donc aucune expérience
professionnelle et mélangent tout (QT, C, C++, etc.. )
Lis un bon bouquin (Deitel ou autre) si tu veux apprendre sérieusement.
Il n'y a pas d'autres méthodes...
N'importe quoi. On peut très bien apprendre sérieusement d'un nombre
infini de méthodes.