[C++] Question de débutant et solution simple

Question de débutant et solution simple [C++] - C++ - Programmation

Marsh Posté le 19-11-2006 à 13:28:53    

Bonjour,
 
je suis débutant en programmation c++. Le c ca va mais pas le c++. Je dois faire dans le cadre de mes études un programme et je débute ici avec l'encodage de pièce dont les infos sont contenues dans une classe. Je voudrais pouvoir ensuite les afficher à l'écran mais je cale par rapport au valeur de retour à donner.
 
Je vous joins le code ce sera plus parlant. Si quelqu'un pouvait m'indiquer quel paramètre donner à ma fonction encodage pour que ma fonction affiche fonctionne correctement.
 
Merci d'avance :)
 
fichier main
 

Code :
  1. #include <iostream>
  2. using namespace std;
  3. #include "PieceMusee.h"
  5. int main ()
  6. {
  7.         int value=0;
  8.         class PieceMusee pPiece;
  10.         do
  11.         {
  12.                 printf("\033[2J" );
  13.                 system ("clear" );
  14.                 cout << "\n\n\n";
  15.                 cout << "\n\n\t\t\t\t\t1- REDIGER MESSAGE";
  16.                 cout << "\n\n\t\t\t\t\t2- LIRE MESSAGE";
  17.                 cout << "\n\n\t\t\t\t\t3- RELEVER MESSAGE";
  18.                 cout << "\n\n\t\t\t\t\t4- ENVOYER MESSAGE";
  19.                 cout << "\n\n\t\t\t\t\t5- ENCODER UNE ADRESSE E-MAIL";
  20.                 cout << "\n\n\t\t\t\t\t0- EXIT";
  21.                 cout << "\n\n\t\t\t\t\tVOTRE CHOIX : ";
  22.                 cin >> value;
  24.                 switch (value)
  25.                 {
  26.                         case 1: pPiece.EncodePiece();
  27.                                 break;
  29.                         case 2: pPiece.AffichePiece();
  30.                                 break;
  32.                         case 3:
  33.                                 break;
  35.                         case 4:
  36.                                 break;
  38.                         case 5:
  39.                                 break;
  41.                         default: break;
  42.                 }
  43.         }
  44.         while (value!=0);
  45. }


 
fichier PieceMusee.cpp
 

Code :
  1. #include <iostream>
  2. using namespace std;
  3. #include "PieceMusee.h"
  5. PieceMusee::PieceMusee (void)
  7. {
  8.         identifiant = "aucun";
  9.         categorie = "aucun";
  10.         description = "aucun";
  11.         dateentree = "aucun";
  12.         estimation = 0;
  13.         expose = 0;
  14.         prete = 0;
  16. }
  19. PieceMusee::PieceMusee(string id, string cat, string descript, string dat, double est, bool exp, bool pre)
  21. {
  22.         identifiant = id;
  23.         categorie = cat;
  24.         description = descript;
  25.         dateentree = dat;
  26.         estimation = est;
  27.         expose = exp;
  28.         prete = pre;
  30. }
  33. PieceMusee::PieceMusee(PieceMusee& Piece)
  35. {
  36.         identifiant = Piece.identifiant;
  37.         categorie = Piece.categorie;
  38.         description = Piece.description;
  39.         dateentree = Piece.dateentree;
  40.         estimation = Piece.estimation;
  41.         expose = Piece.expose;
  42.         prete = Piece.prete;
  44. }
  47. PieceMusee::~PieceMusee ()
  48. {
  49.         /*if (identifiant)
  50.         {
  51.                 delete identifiant;
  52.         }
  53.         if (categorie)
  54.         {
  55.                 delete categorie;
  56.         }
  57.         if (description)
  58.         {
  59.                 delete description;
  60.         }
  61.         if (dateentree)
  62.         {
  63.                 delete dateentree;
  64.         }
  65.         if (estimation)
  66.         {
  67.                 delete estimation;
  68.         }
  69.         if (expose)
  70.         {
  71.                 delete expose;
  72.         }
  73.         if (prete)
  74.         {
  75.                 delete prete;
  76.         }*/
  78. }
  80. //  ==================================
  81. // | METHODE POUR AFFICHER UNE PIECE  |
  82. //  ==================================
  84. void PieceMusee::AffichePiece()
  85. {
  86.         class PieceMusee *pPiece;
  87.         printf("\033[2J" );
  88.         cout << "\n\t\t\t\tIDENTIFIANT : " << pPiece->identifiant;
  89.         cout << "\n\t\t\t\tCATEGORIE : " << pPiece->categorie;
  90.         cout << "\n\t\t\t\tDESCRIPTION : " << pPiece->description;
  91.         cout << "\n\t\t\t\tDATE ENTREE : " << pPiece->dateentree;
  92.         cout << "\n\t\t\t\tESTIMATION : " << pPiece->estimation;
  93.         cout << "\n\t\t\t\tEXPOSE : " << pPiece->expose;
  94.         cout << "\n\t\t\t\tPRETE : " << pPiece->prete;
  95. //        sleep (2);
  96. }
  98. //  ================================
  99. // | METHODE POUR SAISIR UNE PIECE  |
  100. //  ================================
  102. void PieceMusee::EncodePiece()
  103. {
  104.         // --------------------------------------
  105.         // Allocation dynamique d'un nouvel objet
  106.         // --------------------------------------
  107.         class PieceMusee * pPiece;
  108.         pPiece=new PieceMusee;
  110.         // ---------------------
  111.         // Effacement de l'écran
  112.         // ---------------------
  113.         printf("\033[2J" );
  115.         // ------------------
  116.         // Entrée des données
  117.         // ------------------
  118.         cout << "\n\t\t\t\tIDENTIFIANT : " << endl;
  119.         cin >> pPiece->identifiant;
  121.         cout << "\n\t\t\t\tCATEGORIE : " << endl;
  122.         cin >> pPiece->categorie;
  123.         cout <<"\n\tSAISIE: " << pPiece->categorie << endl;
  125.         cout << "\n\t\t\t\tDESCRIPTION :" << endl;
  126.         cin >> pPiece->description;
  128.         cout << "\n\t\t\t\tDATE ENTREE : " << endl;
  129.         cin >> pPiece->dateentree;
  131.         cout << "\n\t\t\t\tESTIMATION : " << endl;
  132.         cin >> pPiece->estimation;
  134.         cout << "\n\t\t\t\tEXPOSE : " << endl;
  135.         cin >> pPiece->expose;
  137.         cout << "\n\t\t\t\tPRETE : " << endl;
  138.         cin >> pPiece->prete;
  140.         return pPiece();
  141. }


 
fichier PieceMusee.h
 

Code :
  1. //****************************************************//
  2. //                CLASSE PIECEMUSEE                   //
  3. //               SIMONIS ARNAUD 2229                  //
  4. //****************************************************//
  6. #include <iostream>
  8. class PieceMusee
  9. {
  10.         public :
  11.                         string identifiant;
  12.                         string categorie;
  13.                         string description;
  14.                         string dateentree;
  15.                         double estimation;
  16.                         bool expose;
  17.                         bool prete;
  19.                         //constructeur par defaut
  20.                         PieceMusee(void);
  21.                         PieceMusee(string id, string cat, string descript, string dat, double est, bool exp, bool pre);
  22.                         PieceMusee(PieceMusee& Piece);
  23.                         ~PieceMusee ();
  25.                         void EncodePiece();
  26.                         void AffichePiece();
  29. };


 
merci encore pour votre aide  :jap:

Reply

Marsh Posté le 19-11-2006 à 13:28:53   

Reply

Marsh Posté le 19-11-2006 à 14:03:47    

et ou est le problème concraitement?  
retourner un void à ta méthode AffichePièce() ne pose pas de problème...  
 
Par contre, ce qui m'embète vraiment dans ton code, c'est que tous est en public... pas bien... pas besoin de faire une classe dans ce cas la...
 
toutes les variables doivent être en "private" et tous le reste en "public" sinon ça me semble bon...
 
De plus, normalement, on créer des assesseurs pour accéder aux variables membres privées (GET/SET) pour pour setter ou getter les valeurs depuis l'extérieur...  
 
Enfin voila déjà pas mal de remarque qui vont te permettre d'avoir une classe digne de ce nom...  
 
Autre gros problème  
class PieceMusee * pPiece;        
pPiece=new PieceMusee;
 
Tu es dingue ou quoi?
 
Tu n'as pas compris la notion d'objet à mon avis.  
Je t'explique, tu as foutu ça dans ton constructeur. Donc dans ton objet PieceMusee, tu créer un nouvel objet qui PieceMusee. Pour accéder à l'objet courant, rien ne vaux le THIS
 
 
 
PS: tu ne ferais pas tes études à l'inpres par hasard?


Message édité par moi23372 le 19-11-2006 à 14:23:54
Reply

Marsh Posté le 19-11-2006 à 14:49:38    

merci pour les réponses,
 
si je fais mes études à l'inpres et toi???
 
Pour les variables publiques je sais bien mais je mettrai tout cela en private par après je ne fais que débuter ici. Je souhaite simplement pouvoir encoder mes pièces et pour les relires après même si je vais balancer ca sur fichier après.
 
Je vais un peu voir ce que tu m'as indiqué mais je suis une daube en c++ mais faut dire que je déteste la programmation et surtout le c++.
 
Merci de me dire quoi :)


Message édité par nolimites le 19-11-2006 à 17:20:53
Reply

Marsh Posté le 19-11-2006 à 18:40:08    

moi j'ai fais mes études à l'inpres, suis sorti l'an passé...  
 
Comment ça se fait que tu fais des études d'informatique si tu détestes la programmation, ce n'est pas vraiment compatible.  
 
De plus, sache que le C++ n'est qu'une base pour les autres languages tel que C# (.NET) et JAVA par exemple. Donc si sincèrement tu veux continuer la dedans, essaye de bien comprendre les bases du C++ (la programmation OO en général) qui te seront indispensables pour accéder à d'autres languages bien plus intéressant que le CPP

Reply

Marsh Posté le 19-11-2006 à 18:46:28    

ben moi je voulais faire du réseau et la ya trop de prog.
 
On est de la mm année mais je suis doublant donc vala et je te connais bien de nom ;)
 
Ben le pire c que java et c# ca va ;)
 
Mais moi je veux du cisco et reseau :)

Reply

Marsh Posté le 19-11-2006 à 19:33:18    

oui mais dans ce métier faut savoir faire un peu de tout...  
 
ça me rassure si java et c# ça te vas. Essaye de tenir le coup en C++ alors, après tout une fois les bases aquises de la POO, tu peux aller partout...  
 
Bonne continuation

Reply

Marsh Posté le 07-12-2006 à 15:31:12    

c bizarre tes mix entre printf et cout
pareil entre \n et endl
à trop mélanger le C et le C++, ça finit en joyeux bordel.
Essaie aussi de fournir un code plus propre au début plutôt que de dire "je changerais plus tard" parceque plus tard, tu ne le feras pas ou mal. Surtout le jour où tu manipulera de centaines voir des milliers de lignes. Les centaines de lignes ça arrive plus vite qu'on le croit.
Je ne vois pas non plus le problème avec ta valeur de retour. Il est normal qu'elle soit void pour une méthode d'affichage.
 
Bon courage

Reply

Marsh Posté le 07-12-2006 à 23:29:26    

mmh, si je comprends bien tu cherches à faire un catalogue de 'pieces de collections' presente dans un musée.
 
Ta démarche n'est pas correcte, tu as un objet "piece de musée" auquel tu demandes:
- initialiste ton état
- affiche toi
- reprends les valeurs d'une autre piece
- CREER UNE PIECE
 
on ne demande pas a une piece de creer une autre piece. Tu as besoin d'une fonction statique indépendante pour faire ca. Ca s'appelle une methode de classe. (static function()). Donc ton header doit etre:
 

Code :
  1. #ifndef __PIECEMUSEE_H__
  2. #define __PIECEMUSEE_H__
  4. #include <iostream>
  6. /**
  7. * Une classe representant une piece de musee
  8. */
  9. class PieceMusee {
  10. private:
  11.                        string identifiant;
  12.                        string categorie;
  13.                        string description;
  14.                        string dateentree;
  15.                        double estimation;
  16.                        bool expose;
  17.                        bool prete;
  18.        public :
  19.                        //constructeur par defaut
  20.                        PieceMusee();
  21.                        PieceMusee(string id, string cat, string descript, string dat, double est, bool exp, bool pre);
  22.                        PieceMusee(const PieceMusee& Piece);
  23.                        virtual ~PieceMusee();
  27.                        static PieceMusee* EncodePiece();
  28.                        void AffichePiece();
  29. }
  30. #endif


 
j'ai mis virtual devant le destructeur, si ca t'échappe c'est pas grave (c'est pour resoudre des problemes de cast de classe dérivées).
 
J'ai mis une fonction static. Cela veut dire que cette fonction n'est pas attachée à un objet instancié, mais à la classe globalement. C'est une fonction globale quelque part..
Tu pourras ensuite l'utiliser comme ceci:

Code :
  1. void mafonction() {
  2.   PieceMusee *p=PieceMusee::EncodePiece();
  3.   p->AffichePiece();
  4.   ...
  5.  
  6.   delete p;
  7. }


 
Pour t'aider, penses d'abord a faire ton header. Le reste vient apres. ll faut juste bien comprendre le header, d'autant que je trouve qu'en c++ séparer les 2 permet d'avoir quelque chose de plus clair qu'en java.
 
 
/nicolas
--
http://www.valdeloire-consulting.com
 

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed