classe et constructeur

Question

Bonjour a tous



je vais essayer de me faire comprendre sans les codes si ce n'est pas possible je les posterais :)



en fait j'ai un probl&#232;me pour les d&#233;clarations de classe : 



si on a:



class X

{

    private int a;

    ...



    public:

        X( int ); // constructeur

        X( const X & );  // constructeur par recopie

};

X::X( int c  { a = c; }

X::X( const X &x) { a = x.a; }



puis dans un autre fichier:



#include"X.h"



class Y

{

    private X a1;

    private X a2;

    private int b;

    ...



    public: Y( X, X, int);

};



Y::Y(X x1, X x2, int t)

{

    a1 = new X(x1);

    a2 = new X(x2);

    b = t;

}





Si je comprends bien il n'est pas possible de d&#233;clarer 

    private X a1;


    private X a2;

 car il n'existe pas de constructeur sans argument pour X .. mais est
il possible de faire sans puisque je cr&#233;e les objets dans le
constructeurs de Y ??



Il  est possible que je ne sois pas clair du tout, je m'en excuse d'avance :p



Merci

vincemail · Answer

Non, je pense que tu dois obligatoirement mettre un constructeur par d&#233;faut, quitte &#224; ne rien mettre &#224; l'int&#233;rieur... ca ne coute rien de toute facon.

iam_myst · Answer

je sais pas je trouve pas ca tr&#232;s "propre" :'(

merci de ta r&#233;ponse je vais quand meme faire ca

iam_myst · Answer

c'est toujours moi :)



dans mon constructeur par recopie j'ai cette erreur : 



erreur: no match for &#145;operator=&#146; in &#145;((Arc*)this)->Arc::somsrc =
(((Sommet*)operator new(8u)),
(->Sommet::Sommet(((Sommet&)(& s1))),
))&#146;



// ce qu'il faut savoir : Arc est une classe ayant pour attribut deux Sommets ( une autre classe ) qui sont somcrc et somdest



je ne comprends vraiment pas cette erreur ( je suis d&#233;sol&#233;, je d&#233;bute )

vincemail · Answer

Tu n'as pas &#224; &#234;tre d&#233;sol&#233; :-) tu pourrais me donner la ligne de code ou tu appelles ce constructeur par recopie ainsi que le prototype de ce constructeur par recopie ? j'ai un peu de mal &#224; d&#233;crypter les messages d'erreur comme ca :)

vincemail · Answer

ah non j'avais pas vu que cette erreur &#233;tait DANS le constructeur de recopie... si tu pouvais mettre la ligne de code correspondant &#224; l'erreur ce serait bien

vincemail · Answer

En fait du devrait avoir un constructeur de recopie de la forme :

Arc::Arc(&Arc arc)
{
    this->somsrc=arc.somsrc;
    this->somdest=arc.somdest;
}

luhtor · Answer

Non, de la forme:

Arc::Arc(const &Arc arc)

{

    this->somsrc=arc.somsrc;

    this->somdest=arc.somdest;

}



Tjs utiliser "const" des que possible.



Je vois pas en quoi ca ne serait pas propre, toi tu veux cr&#233;er un objet sans l'initialis&#233;, c'est encore moins PROPRE !



Sinon dans ta classe X:

class X
 {
     private int a;
     ...
 
     public:
         X( int _val = 0); // <= Met une valeur par d&#233;faut, et ainsi, par besoin de constructeur par d&#233;faut, puisque celui la jouera ce role.
         X( const X & );  // constructeur par recopie
 };



Sinon autre solution, mets ton constructeur par d&#233;faut private dans la
classe X, et ajoute friend class Y dans la classe X. Comme ca, le
constructeur par d&#233;faut n'existe que pour la classe Y et pas pour le
programmeur hors de cette classe.

iam_myst · Answer

voici les deux classes : 



class Arc

{

   private: Sommet somsrc; // sommet source

   private: Sommet somdest; // sommet destination

   private: int poid; // poid de l'arc



   public: Arc( Sommet, Sommet , int );

   public: int getpoid(); 

   public: void setpoid( int );

   public: Sommet getsource(); 

   public: Sommet getdestination();

   public: void setsource( Sommet s );

   public: void setdestination( Sommet s );

}; 







Arc::Arc( Sommet s1, Sommet s2, int p )

{

   somsrc = new Sommet(s1);   /////////// c'est ici que ce situe l'erreur

   somdest = new Sommet (s2);

   poid = p;

}





// accesseur

string Arc::getsource() { return somsrc; }

string Arc::getdestination() { return somdest; }

int Arc::getpoid() { return poid; }



// modifieur

void Arc::setsource( Sommet s ) { somsrc = s; }

void Arc::setdestination( Sommet s ) { somdest = s; }

void Arc::setpoid( int p ) { poid = p; }



void Arc::affiche()

{

   somsrc.affiche();

   cout<<"-"->Sommet::Sommet(((Sommet&)(& s1))),
))&#146;

iam_myst · Answer

pour luthor : ce que je disais c'est que lorsque on d&#233;clare un objet ca
le cr&#233;e en meme temps ? je pensais que le new cr&#233;e un objet
initialement d&#233;clar&#233;

vincemail · Answer

Ca fait une erreur car tu ne peux pas faire de new sur un objet cr&#233;&#233; statiquement... Tu devrais soit surd&#233;finir l'op&#233;rateur "=", ou bien faire de l'allocation dynamique comme tu fais mais dans ce cas il faut mettre  Sommet* somsrc; au lieu de Sommet somsrc, comme ca tu cr&#233;es un pointeur (perso je prefere cette deuxi&#232;me solution). Apr&#232;s il ne faudra pas oublier de faire de delete dans le destructeur.

iam_myst · Answer

ok je vais faire cette deuxieme solution;



donc mes deux attributs sont d&#233;sormais des pointeurs, dans ce cas je n'ai plus besoin de constructeur par recopie non ?



Merci pour les r&#233;ponses en tout cas

vincemail · Answer

Ah ben si :-) Il faut que la classe Sommet puisse recopier s1 et s2...

iam_myst · Answer

hum :/



puisqu' il s'agit d'un pointeur je ne peux pas juste faire dans le
constructeur d'arc quelque chose comme ca : *somsrc = &s1 ?

vincemail · Answer

Ah oui, tu peux faire *somsrc=s1 mais dans ce cas tu ne cr&#233;es pas d'objet ind&#233;pendant de s1, toute modification de s1 entrainera une modification de ton arc. Tout d&#233;pend de ce que tu veux faire, c'est &#224; toi de voir.

iam_myst · Answer

oui ceci me convient parfaitement



Merci de tes conseils ca fait plaisir, si j'ai un autre probl&#232;me j'h&#233;siterais pas a revenir ;)



Merci

vincemail · Answer

U're welcome :)

luhtor · Answer

Deja, pk écris tu ca: public: Arc( Sommet, Sommet , int ); public: int getpoid(); ... et non ca: public: Arc( Sommet, Sommet , int ); int getpoid(); ... Ensuite, c'est important de bien comprendre le constructeur. Pour ca, mets un std::cout << "Constructeur de MACHIN" << std::endl; ou tu remplaces machin par ta classe, tu vas voir l'ordre d'appel au constructeur. Quand tu créés un objet Y, les constructeurs de a1 et a2 (dans ton cas tout en haut du post) sont appelés avant meme d'éxécuter la première ligne de code du constructeur Y ! ima_myst => Sinon, ca: *somsrc = &s1 (sommet) a gauche et (sommet*) a droite, donc type incompatible. vincemail => Surement pas. Faut pas confondre *somsrc=s1 et somsrc = &s1. Avec la première solution, comme ca deja été dit, faut surdéfinir l'opérateur = de la classe sommet. Mais avec la deuxième solution (dangereuse), la oui l'objet somsrc est lié a s1, donc si s1 est détruit, somsrc ne pointe plus sur quelque chose de valide.

vincemail · Answer

Ah vi oups :) 

iam_myst => Tu peux effectivement faire *sommet=s1 si la classe Sommet ne contient pas de pointeurs, cela provoquera une copie de chaque membre, par contre si Sommet contient des pointeurs, il vaut mieux surd&#233;fnir l'op&#233;rateur "=" pour ne pas faire deux pointeurs sur le meme objet...

luthor => Merci, j'espere que je n'ai pas dit de b&#234;tise cette fois-ci

iam_myst · Answer

hum ;)



si le sommet de arc point vers le meme objet que celui de sommet , c'est bien ( j'essaye de faire une repr&#233;sentation de graphe )



Si j'ai bien compris, je laisse tout comme c'est, sauf que dans mon constructeur d' Arc je passe par r&#233;f&#233;rence ?

vincemail · Answer

Ouep, voil&#224;

iam_myst · Answer

ok, je suis content , ca marche merci pour vos r&#233;ponses !

luhtor · Answer

Je pense qu'il est tjs mieux de cr&#233;er manuellement l'op&#233;rateur "=" que de laisser faire un qcq m&#233;canisme automatique.

iam_myst · Answer

ouais , mais ca copierais l'objet, or dans mon cas ce n'est pas avantageux je pense, parce que les objets  sont les memes

iam_myst · Answer

ouais , mais ca copierait l'objet, or dans mon cas ce n'est pas avantageux je pense, parce que les objets  sont les memes

Classe et constructeur

24 réponses

Votre réponse

Discussions similaires