manipulation des objets c++

Bonjour,
Mon problème est le suivant :
J'ai une classe type A :
Victor<X>,double y ......
Je charge ma structure à partir d'un fichier et je fait un traitement la dessus , avant de lancer le traitement je copie l'ancient structure ( A *old=newstructure) pour la garder
et je manipule newstructure si le traitement est pas bien je reviens à l'ancien structure(newstructure=old) sinon je relance le traitement sur la newstructure ainsi de suite.(boucle de 1000 fois par exemple)
ça se compile mais au niveau de l'excution la boucle tourne 2 ou 3 fois et me génére ça :Exception en point flottant
quelqu'un a une solution, je serai reconnaissant

Bonjour,

Citation:

---

Envoyé par zougagh

...je copie l'ancient structure ( A *old=newstructure) pour la garder

---

visiblement vous ne faites pas de copie, une copie se serait A old=*newstructure ou A *old=new A(newstructure) en supposant que l'operateur= ou le copie constructeur sont corrects

Merci de votre réponse,
oui j'ai dans mon fichier.h A *old,A *newstructure;
et dans mon fichier .cpp je fais old=newstructure
Il me paraît que j'ai un probleme dans le copiage vu que c'est des pointeurs, du coup qu'on je fais l'affectation c'est comme je change rien mais je sais pas comment faire.

8O vous êtes sure d'avoir bien lu toute ma réponse précédente ?

oui toujours ça marche pas j'ai rectifié

Citation:

---

Envoyé par zougagh

oui toujours ça marche pas j'ai rectifié

---

La boule de cristale c'est facturé à l'entrée et payé d'avance.

Un code minimal serait pas de refus, on est là pour aider mais on est pas devins non plus :cry:

c'est un code complexe et long mais j'ai encadré le probleme me faut juste une solution :
quand je fais ça old=newstructure (ont même contenu)
j'effectue des changement sur newstructure , ca se produit aussi pour old du coup moi je veux garder l'ancien contenu old
je sais pas si c'est clear

:roll: bon, alors je recommence, si A est le type de la chose, faire soit

Code:

---

1 2	A * old = new A(*newstructure);

---

soit

Code:

---

1 2	A old(*newstructure);

---

et bien vérifier que le copie constructeur fait ce qu'il faut par défaut, sinon le définir

Salut,
Es-tu sur que cela est lié à tes copies et non à une erreur dans ton calcul (comme une division par zéro par exemple) ?

Merci oui c'est lié au copiage , j'ai pas un constructeur de copie , du coup quand je fais l'affectation le contenu de 2 objet ça reste le même avant et après le traitement exemple :
A *a; A*b (avec a contient 2)
je fais b=a;
je change le contenu de a devient par exemple 3
quand je consulte b je trouve 3 sachant que moi je veux garder 2 .
Donc il pointe sur la me chose , du coup faut que je fasse un constructeur de copie si oui comment sachant que mon type A est composé de 4 victor et 4 valeur de type double et un autre type.
Merci encore une fois

Citation:

---

Envoyé par zougagh

faut que je fasse un constructeur de copie si oui comment sachant que mon type A est composé de 4 victor et 4 valeur de type double et un autre type.

---

si les structures/classes copiées ne contiennent pas de pointeur quel que soit le niveau auxquelles elles se trouvent, alors le copie constructeur produit automatiquement pas le compilateur conviendra parfaitement

Merci ,Ma classe est exactement comme suit :

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

class A {   public:   char* DataFile;   int nbt;   int nbse;   int nbf; int nbg;   vector<B *> ser;   vector<C *> flux;   vector<T *> ter; vector<G *> groupes;   Cellule* cell;   public:   Probleme(char* df);   virtual ~Probleme();   void saisirDonnees();   };

---

est-ce aussi nécessaire de surcharger l'opérateur=
Merci

il y a donc bien des pointeurs

la question est alors de savoir ce qui doit se passer lorsque vous dupliquez une instance, les objets pointés sont-ils partagés ou doivent-t-il eux même être dupliqués ?

s'il faut dupliquer au moins l'un d'entre eux, alors il faut définir l'opérateur= et le copie constructeur afin de ne pas voir de mauvaise surprise.

ne pas oublier un destructeur adéquat libérant tout ce qu'il faut, mais pas plus en cas de partage de données ;)

je veux faire un déplicata de structure au debut et après chaque traitement relancé sur une structure je regarde si j'ai de bonne resultat je garde la structure et je relance le même traitemant sinon je reviens à la structure juste avant ainsi de suite.

Citation:

---

Envoyé par zougagh

faire un déplicata

---

ce n'est pas assez précis, relisez ma réponse précédente, et donc par exemple la duplication d'une instance implique-t-elle celle de son DataFile etc ?

non le premier constructeur c'est pour charger le fichier pour remplir la structure après on la modifie avec le traitement voila maintenant comment faire pour le constructure de copie et l'opérateur = en présence de cette structure avec les victor voila ce qui m'embête pour l'instant.

Par exemple je vais faire ça :
A::A(const A& a): nbt(p.nbt),nbs(p.nbs) .......{}
Mais je fais comment pour les victeurs

tout dépend de la réponse à la question déjà posée : les objets pointés sont-ils partagés ou doivent-t-il eux même être dupliqués ?

on ne peux pas répondre à cette question à votre place, et tant que vous n'y répondrez pas il sera impossible d'avancer :aie:

non sont pas partagés

alors il faut aussi les dupliquer, et donc par exemple

Code:

---

1 2 3 4 5 6 7 8 9 10

A::A(const A& a) : nbt(p.nbt),nbs(p.nbs) .......{ ser.reserve(a.ser.size()); // optimize insertion   for (vector<B *>::const_iterator itb = a.ser.begin(); itb != a.ser.end(); itb++) ser.push_back(new B(**itb));   ... }

---

plus d'information sur les vecteurs -> la faq / les tutoriels

Salut,

Ce qui serait intéressant de savoir, c'est si les classes B, C, G et T, respectivement utilisée avec des tableaux pour les membres cer, flux, groupe et ter, ont des comportement polymorphes.

En effet, si ces différents objets ne sont pas polymorphes, et si (cela semble être le cas, si j'ai bien compris :D) la durée de vie de ces objets est gérée par ta classe A, il n'y a, a priori, aucune raison pour les manipuler sous la forme de pointeurs.

En effet, si tu supprimes déjà le recours à des pointeurs pour ces objets, tu rend la copie d'une instance de type A beaucoup plus facile, car la classe vector est parfaitement copiable et assignable.

La deuxième chose qu'il me semblerait bon d'évaluer, c'est d'essayer de déterminer si la classe A, et surtout, la fonction saisirDonnees, n'en font pas un peu trop.

En effet, lorsque je vois qu'une classe dispose, en dehors des formes de Coplien, d'une seule fonction et de... 10 membres, je ne peux m'empêcher de penser que la classe, et très certainement la fonction, ont beaucoup trop de responsabilités :aie:

Enfin, il semblerait intéressant d'évaluer correctement si la classe a effectivement sémantique de valeur.

En effet, cette classe a tout ce qu'il faut pour être utilisée de manière polymorphe, à commencer par un destructeur virtuel, et en continuant par la seule fonction qui ne fasse pas partie des forme de Coplien (saisirDonnees, toujours) qui est, également, virtuelle.

D'après mon expérience, les objets polymorphes ont rarement (jamais :question:) une sémantique de valeur et très souvent une sémantique d'entité, ce qui devrait nous inciter à les rendre non copiables et non assignables. Au mieux, on pourrait envisager de rendre les objets ayant sémantique d'entité clônables ;)

Enfin, bref, tout cela pour dire qu'il serait intéressant de savoir l'utilisation que tu prévois pour ta classe A (et pour toutes les classes associées: B, C, G et T) , car, soit, elles ont sémantiques de valeur, et il est donc vraiment important de réfléchir au meilleur moyen de les rendre copiables et assignables (entre autres, en "virant" tous les pointeurs ;)), soit elles ont sémantique d'entité, et il faut revoir en profondeur la manière dont tu les utilises, car il n'y a sans doute pas lieu de permettre leur copie et leur affectation ;)

sinon svp comment faire pour surcharger l'opérateur = pour ma structure
merci de votre réponse

@koala1 : j'aurai besoin d'ytiliser les pointeurs pour cette structure