[c++] constructeur par copie et par parametre

Salut 'a vous,

je me demande c'est quoi la difference entre

Code:

---

1 2 3

chaine(char *ch2);// parametre chaine(char & ch2);// constructuer par copie

---

pour pouvoir dire que le premier est constructueur par parametre et le deuxieme par copie. finalement pourquoi il ya une difference entre char * et char & puisuqe les deux renvoi une addresse
voic mon code [CODE]

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26

#ifndf CHAINE #define CHAINE #include <iostream> #include <cstring> using namespace std;   class chaine { char *ch_; int longueur_; int lmax_; static int quantite_;   public: chaine(int n=100); chaine(char *ch2);// parametre chaine(char & ch2);// constructuer par copie ~chaine(); void affiche(){cout<<ch_<<endl;} static int combien(){return quantite_;}   }; #endif /CODE]

---

Si tu parles du constructeur de copie alors non la bonne forme est :

Code:

---

1 2	chaine(const chaine&);

---

Citation:

---

Envoyé par Goten

Si tu parles du constructeur de copie alors non la bonne forme est :

Code:

---

1 2	chaine(const chaine&);

---

---

tu as raison je viens de voir mon erreur hehe

cependant c'est quioi la difference char & et char*

Salut,
char* est ... un pointeur sur caractères (autrement dit, même si le raccourcis est un peu rapide, l'équivalent à une chaine de caractères "C style"), alors que char & est, tout simplement, une référence sur un caractère unique ;)

Ceci dit, pourquoi diable te faire ch:aie: à créer ta propre classe de type "chaine de caractères", alors qu'il existe la classe string, disponible (comme tout ce qui est issu du standard) dans l'espace de noms std, par simple inclusion du fichier d'en-tête <string> :question:

merci pour ta reponse je comprends mieux.
en fait je debute et j'ai trouve l'idee de cree une calss string bonne pour deux raisons
familiarise avec la surcharge d'operateur et manipuler destchaines de caratere et des pointeur
il n'y a pas mieux
meme si c'est casse pied je l'avoue:)

Je comprend l'attrait "didactique" de réinventer la roue, mais, le gros problème est qu'il est souvent synonyme de prise de mauvaises habitudes...

Celle de manipuler des pointeurs "à tout va", sans penser aux possibilités fournies par le standard en est une, et il y a surement des exercices plus corrects à envisager pour s'habituer à leur utilisation ;)

Si le seul but est de t'habituer à manipuler des pointeurs, tu devrais, par exemple, envisager un petit projet où tu mettrais en oeuvre les méthodes qui nécessitent vraiment leur utilisation (héritage, polymorphisme et autres en font plus ou moins partie ;))

D'un autre coté, tu devrais très rapidement prendre l'habitude de travailler à coup de std::string, std::vector et autres conteneurs, car une mauvaise habitude est d'autant plus difficile à perdre au profit d'une bonne que la mauvaise aura eu le temps de s'installer ;)

salut,
merci pour tes conseils.
j'ai un progblem avec le resutat ca compile bien ca affiche le resultat
mais ca affiche un tres grand ... ou
merci

Citation:

---

bash-3.2$ g++ main.cpp chaine.cpp chaine.h -o chain
bash-3.2$ ./chain
delem alikomm hahhaha
*** glibc detected *** ./chain: double free or corruption (out): 0xbfdd5d68 ***
======= Backtrace: =========
/lib/libc.so.6[0x2b5ac1]
/lib/libc.so.6(cfree+0x90)[0x2b90f0]
/usr/lib/libstdc++.so.6(_ZdlPv+0x21)[0x1df111]
/usr/lib/libstdc++.so.6(_ZdaPv+0x1d)[0x1df16d]
./chain(__gxx_personality_v0+0x2ff)[0x804899f]
./chain(__gxx_personality_v0+0x1d6)[0x8048876]
/lib/libc.so.6(__libc_start_main+0xe0)[0x262390]
./chain(__gxx_personality_v0+0x41)[0x80486e1]
======= Memory map: ========
00110000-0012b000 r-xp 00000000 08:03 648970 /lib/ld-2.7.so
0012b000-0012c000 r--p 0001a000 08:03 648970 /lib/ld-2.7.so
0012c000-0012d000 rw-p 0001b000 08:03 648970 /lib/ld-2.7.so
0012d000-0012e000 r-xp 0012d000 00:00 0 [vdso]
0012e000-0020c000 r-xp 00000000 08:03 111844 /usr/lib/libstdc++.so.6.0.8
0020c000-0020f000 r--p 000dd000 08:03 111844 /usr/lib/libstdc++.so.6.0.8
0020f000-00211000 rw-p 000e0000 08:03 111844 /usr/lib/libstdc++.so.6.0.8
00211000-00217000 rw-p 00211000 00:00 0
00217000-0023e000 r-xp 00000000 08:03 645918 /lib/libm-2.7.so
0023e000-0023f000 r--p 00026000 08:03 645918 /lib/libm-2.7.so
0023f000-00240000 rw-p 00027000 08:03 645918 /lib/libm-2.7.so
00240000-0024b000 r-xp 00000000 08:03 648978 /lib/libgcc_s-4.1.2-20070925.so.1
0024b000-0024c000 rw-p 0000a000 08:03 648978 /lib/libgcc_s-4.1.2-20070925.so.1
0024c000-0039f000 r-xp 00000000 08:03 648853 /lib/libc-2.7.so
0039f000-003a1000 r--p 00153000 08:03 648853 /lib/libc-2.7.so
003a1000-003a2000 rw-p 00155000 08:03 648853 /lib/libc-2.7.so
003a2000-003a5000 rw-p 003a2000 00:00 0
08048000-0804a000 r-xp 00000000 00:19 136314967 /usagers/hamaa/Desktop/temp/string/chain
0804a000-0804b000 rw-p 00001000 00:19 136314967 /usagers/hamaa/Desktop/temp/string/chain
0897e000-0899f000 rw-p 0897e000 00:00 0 [heap]
b7e00000-b7e21000 rw-p b7e00000 00:00 0
b7e21000-b7f00000 ---p b7e21000 00:00 0
b7faf000-b7fb1000 rw-p b7faf000 00:00 0
b7fd6000-b7fd7000 rw-p b7fd6000 00:00 0
bfdc2000-bfdd7000 rw-p bffeb000 00:00 0 [stack]
delem alikomm hahhahaAbandon

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11

nclude "chaine.h"   int main() {   chaine c(10); chaine a("delem alikomm hahhaha"); a.affiche(); cout<< a.tostring(); return 0; }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28

#ifndef CHAINE #define CHAINE #include <iostream> #include <cstring> using namespace std;   class chaine { char *ch_; int longueur_; int lmax_; //static int quantite_;   public: chaine(int n=100); chaine(char *ch2);// parametre chaine(chaine & ch2);// constructuer par copie int size(){ return strlen(ch_);} ~chaine(); char* tostring(){return ch_;}; void affiche(){cout<<ch_<<endl;} //static int combien(){return quantite_;}   }; #endif

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59

#include "chaine.h"   chaine:: chaine(int n) { if(n>0) { ch_= new char[n+1] ; longueur_ =n ; // quantite_++; } else { ch_= new char[101]; longueur_ = 100; // quantite_++; }   lmax_=100;     } chaine::chaine(char *ch2) { if(ch2) { if (strlen(ch2)-1>lmax_) { ch_ = new char[lmax_]; ch_=strcpy(ch_,ch2); } else { ch_=strncpy(ch_,ch2,100); }     } else {   }   }// parametre   chaine::chaine(chaine & ch2) { ch_= strcpy(new char[ch2.size()+1],ch2.tostring()); longueur_ =strlen(ch_) ;   } // constructuer par copie chaine::~chaine() { if(ch_) delete [] ch_; // quantite_--; }

---

c'est cette instruction qui me fait defaut

Code:

---

chaine a("delem alikomm hahhaha");

---

merci pour votre aide

Ben faut investir dans un debuggeur !
J'ai juste copié collé ton code, lancé et boum, il s'est planté ici :

Code:

---

1 2 3 4

else { ch_=strncpy(ch_,ch2,100); }

---

Sur l'appel de

Code:

---

chaine a("delem alikomm hahhaha");

---

Juste deux remarques :
(1) Les nombres magiques c'est mal. Pourquoi 100 ? Si tu ne connais pas la taille de la chaine alors la convention est plutôt de construire une chaine vide.
(2) Vouloir rattraper une erreur grave sans prévenir l'utilisateur c'est mal. Si l'utilisateur a donné un nombre négatif pour le constructeur de la chaine c'est qu'il a une grave erreur de logique dans son programme. Autant que ça plante directement plutôt qu'allouer une chaine de 100 caractères silencieusement.

Citation:

---

Envoyé par Arzar

Ben faut investir dans un debuggeur !
J'ai juste copié collé ton code, lancé et boum, il s'est planté ici :

Code:

---

1 2 3 4

else { ch_=strncpy(ch_,ch2,100); }

---

Sur l'appel de

Code:

---

chaine a("delem alikomm hahhaha");

---

Juste deux remarques :
(1) Les nombres magiques c'est mal. Pourquoi 100 ? Si tu ne connais pas la taille de la chaine alors la convention est plutôt de construire une chaine vide.
(2) Vouloir rattraper une erreur grave sans prévenir l'utilisateur c'est mal. Si l'utilisateur a donné un nombre négatif pour le constructeur de la chaine c'est qu'il a une grave erreur de logique dans son programme. Autant que ça plante directement plutôt qu'allouer une chaine de 100 caractères silencieusement.

---

Merci j'ai vu l'erreure copy sans allocation de memoire
merci je vais tester