Constructeur de copie

Marsh Posté le 16-06-2003 à 09:54:38

voici le code que me tracasse :

Code :

//constructeur de copie
Tableau::Tableau(const Tableau &int):taille(int.taille)
{
ptr=new int[taille];
assert(ptr!=0);
++compteTableau;
for (int i=0;i<taille;i++)
ptr[i]=init.ptr[i];
}

Or ptr est une donnée membres (un pointeur), donc ptr[i]=int.prt[i] va modifier mon pointeur...
Je vois pas ou est la copie en fait..vu que j'ecrase le contenu du prt??

Reply

Marsh Posté le 16-06-2003 à 09:54:38

Reply

Marsh Posté le 16-06-2003 à 10:07:21

Citation :

Or ptr est une donnée membres (un pointeur), donc ptr[i]=int.prt[i] va modifier mon

Non vu que tu ne modifie pas le pointeur mais les données pointées par ce pointeur

Reply

Marsh Posté le 16-06-2003 à 10:12:22

oui excuse je modifie la donnée pointée mais donc aucune copie..j'ai pas deux donnée contenant la meme chose...
c'est ça que je comprend pas....
:sweat:

Reply

Marsh Posté le 16-06-2003 à 11:03:28

bin si.

le ptr=new int (taille);

crée le nouveau tableau...

puis le for copie les données...

Reply

Marsh Posté le 16-06-2003 à 11:13:09

C'est une copie légère que tu fais (shallow copy en anglais), pas une copie profonde (deep copy).

Tu dupliques le tableau, mais pas le contenu du tableau. Le tableau et sa copie pointeront sur les mêmes objets.

Petite nuance : ceci est vrai dans l'absolu, et c'est vrai à tous les coups en Java. En C++, cela dépend de ce que contient le tableau. S'il contient des pointeurs ou des références, ce sera le cas. S'il contient des atomiques ou des objets, alors l'affectation "ptr[i ]=init.ptr[i ];" fera elle-même une duplication implicite du i-ème élément du tableau et la copie sera quand même profonde.

Message édité par BifaceMcLeOD le 16-06-2003 à 11:14:53

Reply

Marsh Posté le 16-06-2003 à 11:50:55

grosmethos >> tu pourrais pas renommer le nom du tableau passé en paramètres ? passke int comme nom de variable, c'est super limpide pour le reste du code...

Reply

Marsh Posté le 16-06-2003 à 11:51:37

BJOne a écrit :

grosmethos >> tu pourrais pas renommer le nom du tableau passé en paramètres ? passke int comme nom de variable, c'est super limpide pour le reste du code...

nan il a du merder dans la recopie du code, regarde la suite, c'est init le nom de sa variable (sinon de toute facon ca compilerait pas )

Reply

Marsh Posté le 16-06-2003 à 11:53:35

ce serait pas plustôt ça que tu devrais avoir: :??:

Code :

Tableau::Tableau(const Tableau &init):taille(init.taille)
{
ptr=new int[taille];
assert(ptr!=0);
++compteTableau;
for (int i=0;i<taille;i++)
ptr[i]=init.ptr[i];
}

EDIT: joceeeeeeeeeeeeeeeeeee la balise de changement de couleur chie dans le [cpp][/cpp]

Message édité par bjone le 16-06-2003 à 11:54:46

Reply

Marsh Posté le 16-06-2003 à 11:53:55

chrisbk a écrit :

nan il a du merder dans la recopie du code, regarde la suite, c'est init le nom de sa variable (sinon de toute facon ca compilerait pas )

+1

on est d'accord, donc en principe y'a bien l'allocation du nouveau tableau, et la recopie.

Message édité par bjone le 16-06-2003 à 11:55:33

Reply

Marsh Posté le 16-06-2003 à 12:50:33

terriblement efficace le assert, aucune utilité puisque bad_alloc est lancé

moi je préfère

Code :

Tableau::Tableau(const Tableau &init)
: taille(init.taille), ptr(new int[taille])
{
++compteTableau;
std::copy(init.ptr, init.ptr+taille, ptr);
}

réutilisation maximum, meilleure protection contre les exceptions

Reply

Constructeur de copie

Sujets relatifs:

Leave a Replay