[resolu]tableau d'objet dans une classe (et ca marche pas bien sur)

tableau d'objet dans une classe (et ca marche pas bien sur) [resolu] - C++ - Programmation

Marsh Posté le 05-11-2010 à 16:27:10    

Salut,
 
je débute et j'ai deja des petits problemes.
 
resume:
j'ai une classe voiture et une classe parking.
Je veux mettre des voitures dans le parking. (mais le gardien est pas trop cool, il veut pas  :bounce: )
 
j'ai donc cree dans parking.hpp un attribut:

Code :
  1. voiture _park


 
et dans parking.ccp:

Code :
  1. _park = new voiture[inCapa];


 mais il veut pas...
 
je vous met les codes:  
(la classe voiture marche bien, pas d'erreur de compilation ni d'execution.)
 
voiture.hpp

Code :
  1. #ifndef _V_HPP
  2. #define _V_HPP
  4. #include <iostream>
  5. #include <string>
  7. class voiture
  8. {
  9.   public:
  10.     voiture(std::string inMarque, std::string inCouleur,int inPu,int intinRm,int inConsu,int inConseu,int inCapres,int inContres,int inECO2);
  11.     void remplir_bibi() {if(getContres() < getCapres()) setContres(getCapres());}
  12.     void ontheroad(int inDist, std::string inType);
  14.     std::string getMarque() {return Marque;}
  15.     std::string getCouleur() {return Couleur;}
  16.     static int getnbrvoiture() {return compteur;}
  17.     int getContres() {return Contres;}
  18.     int getPu()  {return Pu;}
  19.     int getRm()  {return Rm;}
  20.     int getConsu()  {return Consu;}
  21.     int getConseu() {return Conseu;}
  22.     int getCapres() {return Capres;}
  23.     int getECO2() {return ECO2;}
  25.     void setMarque(std::string inMarque)  {Marque  = inMarque;}
  26.     void setCouleur(std::string inCouleur)  {Couleur = inCouleur;}
  27.     void setContres(int inContres) {Contres = inContres;}
  28.     void setConseu(int inConseu)  {Conseu = inConseu;}
  29.     void setCapres(int inCapres)  {Capres = inCapres;}
  30.     void setPu(int inPu) {Pu = inPu;}
  31.     void setRm(int inRm) {Rm = inRm;}
  32.     void setConsu(int inConsu) {Consu = inConsu;}
  33.     void setECO2(int inECO2) {ECO2 = inECO2;}
  35.   private:
  36.     static int compteur;
  37.     std::string Marque;
  38.     std::string Couleur;
  39.     int Contres; // contenu reservoir
  40.     int Pu;
  41.     int Rm;
  42.     int Consu;
  43.     int Conseu;
  44.     int Capres;
  45.     int ECO2;
  46. };
  48. #endif


 
parking.hpp

Code :
  1. #ifndef _PA
  2. #define _PA
  4. #include <iostream>
  5. #include <string>
  7. #include "voiture.hpp"
  9. class Parking
  10. {
  11.   public:
  12.     Parking(int inCapa);
  13.     ~Parking();
  14.  
  15.   void getContPark();//liste contenu du parc
  16.   void addvoit(voiture  &invoit);
  18.   private:
  19.     voiture * _park;
  20.     int _capacite;
  21.     int _nbrvoit;
  22. };
  24. #endif


 
parking.cpp

Code :
  1. #include "parking.hpp"
  3. Parking::Parking(int inCapa)
  4. {
  5.   _capacite = inCapa;
  6.   _nbrvoit  = 0;
  7.   _park = new voiture[inCapa];
  8. }
  10. Parking::~Parking()
  11. {
  12.   delete[] _park;
  13. }
  15. void Parking::getContPark()//liste contenu du parc
  16. {
  17.   int i = 0;
  18.   for (; i < _nbrvoit; i++)
  19.   {
  20.     std::cout << "voiture " << i << ": "<< _park[i].getMarque() <<" "<<_park[i].getCouleur()<<std::endl;
  21. }
  22. }
  24. void Parking::addvoit(voiture  &invoit)
  25. {
  26.   _park[_nbrvoit -1] = invoit;
  27.   _nbrvoit++;
  28. }


 
main.cpp

Code :
  1. #include "voiture.hpp"
  2. #include "parking.hpp"
  4. int voiture::compteur = 0;
  5. int main(void)
  6. {
  8.   voiture lamienne("Mustang", "nuit",400,2500,10,7,65,65,10);
  9.   voiture ladfienne("Mustang", "nuit",400,2500,10,7,65,65,10);
  10.   lamienne.ontheroad(30, "ville" );
  12.   std::cout << voiture::getnbrvoiture() << std::endl;
  13.  
  14.   Parking brio(10);
  15.  
  16.   return 0;
  17. }


 
voila
merci d'avance
 
ps: l'erreur doit etre ridicule, c'est notre premier TP de C++...


Message édité par karlakir le 07-11-2010 à 14:02:44

---------------
Si vous avez rien à faire, cliquez ici:
Reply

Marsh Posté le 05-11-2010 à 16:27:10   

Reply

Marsh Posté le 05-11-2010 à 18:55:08    

Citation :

_park[_nbrvoit -1] = invoit;
_nbrvoit++;


C'est pas bon:
Au départ, _nbrvoit est à 0
Quand tu ajoutes ta première voiture, tu fais donc: _park[ -1] :whistle:  
Si tu fais  

Citation :

_park[_nbrvoit] = invoit;
_nbrvoit++;


Ça devrait être mieux.
A+,
 


---------------
There's more than what can be linked! --  Le capitaine qui ne veut pas obéir à la carte finira par obéir aux récifs. -- No jab ? No job ! -- (╯°□°)╯︵ ┻━┻
Reply

Marsh Posté le 06-11-2010 à 00:34:45    

C'est déjà un problème que j'avais pas vu, merci  :)  
 
il y a encore une derniere erreur à la compilation:
 
la ligne:

Code :
  1. _park = new voiture[inCapa];

semble lui casser les pieds
je crée bien un tableau contenant inCapa objet voiture, je ne devrais pas avoir à préciser autre chose si?
 

Code :
  1. g++   -Wall -ansi -pedantic -Wextra -c parking.cpp -o parking.o
  2. parking.cpp: In constructor ‘Parking::Parking(int)’:
  3. parking.cpp:7: error: no matching function for call to ‘voiture::voiture()’
  4. voiture.hpp:10: note: candidates are: voiture::voiture(std::string, std::string, int, int, int, int, int, int, int)
  5. voiture.hpp:8: note:                 voiture::voiture(const voiture& )
  6. make: *** [parking.o] Erreur 1


Reply

Marsh Posté le 06-11-2010 à 09:55:40    

C'est parce que tu n'as pas défini de constructeur sans paramètres pour la classe Voiture. A partir du moment où tu définis un constructeur, le compilateur ne génère pas de constructeur par défaut.
Pense à vérifier s'il y a des places libres avant d'ajouter une voiture, sinon tu vas dépasser la capacité du tableau.


Message édité par el muchacho le 06-11-2010 à 09:56:53

---------------
Les aéroports où il fait bon attendre, voila un topic qu'il est bien
Reply

Marsh Posté le 07-11-2010 à 14:03:17    

c'est bon ca marche,  
 
merci pour votre aide et vos réponses!!


---------------
Si vous avez rien à faire, cliquez ici:
Reply

Sujets relatifs:

Leave a Replay

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