Dupliquer un objet

Bonjour

Sur un code pré-existant, je dois appliquer des unit tests avec cppUnits.
Une classe me laisse dubitatif :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

class Spectrum: public std::vector<Dot> { Spectrum(); public: void addPoint(const Dot& point); Spectrum& clone(); }   void Spectrum::addPoint(const Dot& point) { push_back(point); }   Spectrum& Spectrum::clone() { return *this; }

---

Et voici ce que je veux évaluer dans l'un de mes tests unitaires (NB: on suppose que pour le Spectrum s, j'ai appelé le constructeur et celui-ci lui met déjà un point de coordonnées (1,2):

Code:

---

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

void SpectrumTest::setUp(){ s = Spectrum(); s.addPoint(Dot(1,2)); }   void SpectrumTest::cloneTest(){ s.addPoint(Dot(1, 2)); s.addPoint(Dot(3, 4)); s.addPoint(Dot(5, 6)); s.addPoint(Dot(7, 1)); Spectrum s1 = s.clone(); CPPUNIT_ASSERT_EQUAL(s.size(), s1.size()); std::vector<Dot>::iterator it = s.begin(); std::vector<Dot>::iterator it1 = s1.begin(); while(it != s.end()) { Dot d = *it; Dot d1 = *it1; CPPUNIT_ASSERT_EQUAL(d.getX(), d1.getX()); CPPUNIT_ASSERT_EQUAL(d.getY(), d1.getY());*/ it++; } }

---

Dans cette méthode, effectivement, le premier vecteur reçoit bien les objets Dot (ie des points). Mais, à l'appel de la méthode clone(), le second vecteur reçoit bien 5 points mais TOUS avec les coordonnées (1,2) :koi:

A quoi est dû ce phénomène et comment le solutionner ?

@++

C'est normal, l'itérateur sur ton deuxième vector n'est jamais incrémenté (it1), du coup il pointe toujours sur le premier élément, soit le point de coordonnées (1,2).

Oups ! C'est juste :oops:
Merci à toi bolhrak
Décidément, les lundis ne me réussissent jamais.

@++

Citation:

---

Envoyé par GLDavid

Oups ! C'est juste :oops:
Merci à toi bolhrak
Décidément, les lundis ne me réussissent jamais.

@++

---

Héhé t'inquiètes pas tu n'es pas le seul ;)

PS: C'est normal que clone() ne crée pas réellement de copie comme son nom le laisse supposer ?

Citation:

---

Envoyé par Médinoc

PS: C'est normal que clone() ne crée pas réellement de copie comme son nom le laisse supposer ?

---

Comme il hérite de std::vector - :vomi: -, c'est operator= qui fait le clonage, en fait...

Comme j'ai dit en préambule, je ne fais que tester le code. Mais apparemment, faire hériter de std::vector = pas bien ?

@++

GLDavid, qui s'excuse d'avance car il code de préférence en Java-perl.

Citation:

---

Envoyé par GLDavid

Comme j'ai dit en préambule, je ne fais que tester le code. Mais apparemment, faire hériter de std::vector = pas bien ?

---

En général oui, il peut y avoir des cas où c'est bien, mais là, apparemment, c'est typiquement normalement une "implémentation en terme de" - donc une variable membre std::vector - plutôt qu'un "est un" - héritage public -

Je rigole Miles :mrgreen: car ce que tu ne sais pas, c'est que le code que je dois tester à déjà été initialement implémenté par votre serviteur en ... Java ou le vecteur était une donnée membre :mrgreen: . Puis, le client a demandé une version librairie C++ et voici le changement dont je n'étais pas responsable. Qui avait raison ? Bibi !!!

@++

Tu m'étonnes... Pour tous les langages, il y a des gens qui font n'importe quoi sans réfléchir aux implications :(