Problème de réallocation de la mémoire

Salut voila je suis entrain de bosser sur une petite application et je veut réallouer de la mémoire et ajouter un element avec la fonction new mais le problème c’est que chaque fois je réalloue la mémoire de mon objet tous les données disparaisses
Voici le code

Code:

---

1 2	Rendezvous *t; int n;

---

Code:

---

1 2 3 4 5 6 7 8

agenda::operator +(Rendezvous &s){ this->t=new Rendezvous[this->n+1]; this->t[this->n].p.anne=s.p.anne; this->t[this->n].p.jour=s.p.jour; this->t[this->n].p.mois=s.p.mois; this->t[this->n].s.copy(s.s); this->n++; }

---

Conseil : Pour avoir des conteneur de taille variable, utilise la STL. Je ne sais pas quel est le type de t, mais il gagnerait probablement à être un vector<Rendezvous>, ou une list<Rendezvous>, selon les opérations que tu veux lui faire.

En interne, ces conteneurs réalisent quand nécessaire la copie qui est le seul moyen de garder les données initiales lors d'une réallocation.

Citation:

---

Envoyé par JolyLoic

Conseil : Pour avoir des conteneur de taille variable, utilise la STL. Je ne sais pas quel est le type de t, mais il gagnerait probablement à être un vector<Rendezvous>, ou une list<Rendezvous>, selon les opérations que tu veux lui faire.

En interne, ces conteneurs réalisent quand nécessaire la copie qui est le seul moyen de garder les données initiales lors d'une réallocation.

---

T c’est un tableau dynamique d’objet


Je peux pas travailler avec stl ou vecteur car je suis suivi par un cahier de charge qui m’oblige a ne rien ajouter

Il faut recopier tes données à la main.
Et il faut aussi libérer les anciennes données.

Citation:

---

Envoyé par loufoque

Il faut recopier tes données à la main.
Et il faut aussi libérer les anciennes données.

---

je viens de le faire mais ca beug :beug:

Code:

---

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

agenda::operator +(Rendezvous &rv){   int i=0; Rendezvous *T=new Rendezvous[this->n+1]; for(i=0;i<this->n;i++) T[i]=this->t[i]; this->t=new Rendezvous[this->n+2]; for(i=0;i<this->n;i++) this->t[i]=T[i];   this->n=this->n+1; printf("**************\n"); this->t[this->n]=rv;   }

---

Si tu ne peux pas utiliser la STL, rien ne t'empeche de coder un équivalent de std::vector qui s'adapterait à tes besoins ? C'est loin d'être insurmontable.

Citation:

---

Envoyé par dot-_-net

je viens de le faire mais ca beug :beug:

Code:

---

1 2 3 4 5 6 7 8 9

agenda::operator +(Rendezvous &rv){   int i=0; Rendezvous *T=new Rendezvous[this->n+1]; for(i=0;i<this->n;i++) T[i]=this->t[i];     }

---

---

Salut,
J'usqu'à la tout va bien :D c'est ensuite que ça se gâte.

Tu dois libérer l'ancienne allocation mémoire et faire pointer this->t sur la nouvelle. Ensuite copie rv dans l'emplacement [n] et incrémente n.
Ca devrait marcher.

Sinon je ne comprend pas pourquoi tu ne peux pas utiliser la STL, sauf si c'est un exercice.
Autre remarque : l'opérateur += serait plus approprié ici car habituellement l'opérateur + ne modifie aucun de ses opérandes, ça peut rendre la lecture de ton code confuse.

a+
jmv

Citation:

---

Envoyé par jmv

Salut,
J'usqu'à la tout va bien :D c'est ensuite que ça se gâte.

Tu dois libérer l'ancienne allocation mémoire et faire pointer this->t sur la nouvelle. Ensuite copie rv dans l'emplacement [n] et incrémente n.
Ca devrait marcher.

Sinon je ne comprend pas pourquoi tu ne peux pas utiliser la STL, sauf si c'est un exercice.
Autre remarque : l'opérateur += serait plus approprié ici car habituellement l'opérateur + ne modifie aucun de ses opérandes, ça peut rendre la lecture de ton code confuse.

a+
jmv

---

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

agenda::operator +(Rendezvous &rv){   int i=0;   Rendezvous *T=new Rendezvous[this->n+1];   for(i=0;i<this->n;i++){ T[i].p.mois=this->t[i].p.mois; t[i].p.anne=this->t[i].p.anne; t[i].p.jour=this->t[i].p.jour; t[i].s.copy(this->t[i].s.str); } this->t=new Rendezvous[this->n+2]; for(i=0;i<this->n;i++){ this->t[i].p.mois=T[i].p.mois; this->t[i].p.anne=T[i].p.anne; this->t[i].p.jour=T[i].p.jour; this->t[i].s.copy(T[i].s.str); } this->n=this->n+1;   this->t[i].p.mois=rv.p.mois; this->t[i].p.anne=rv.p.anne; this->t[i].p.jour=rv.p.jour; this->t[i].s.copy(rv.s.str);       }

---

ca beug pas mais quand je reallou la memoire je perd mes données:(

Citation:

---

Envoyé par dot-_-net

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

agenda::operator +(Rendezvous &rv){ int i=0; Rendezvous *T=new Rendezvous[this->n+1]; for(i=0;i<this->n;i++){ T[i].p.mois=this->t[i].p.mois; t[i].p.anne=this->t[i].p.anne; t[i].p.jour=this->t[i].p.jour; t[i].s.copy(this->t[i].s.str); } this->t=new Rendezvous[this->n+2]; for(i=0;i<this->n;i++){ this->t[i].p.mois=T[i].p.mois; this->t[i].p.anne=T[i].p.anne; this->t[i].p.jour=T[i].p.jour; this->t[i].s.copy(T[i].s.str); } this->n=this->n+1; this->t[i].p.mois=rv.p.mois; this->t[i].p.anne=rv.p.anne; this->t[i].p.jour=rv.p.jour; this->t[i].s.copy(rv.s.str); }

---

ca beug pas mais quand je reallou la memoire je perd mes données:(

---

Lol ça fait 4 heurs que je cherche un beug et ce beug vient d’une faute d’inattention
probléme :resolu::aie:

Bien que personnellement je ne ferai pas comme ça, je te propose ce code qui est le plus proche possible du tiens, essaye ça, j'ai pas vérifier

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

agenda::operator +(Rendezvous &rv){   int i=0;   // allocation de la mémoire Rendezvous *T=new Rendezvous[this->n+1];   // on copie les données for(i=0;i<this->n;i++) { T[i].p.mois=this->t[i].p.mois; T[i].p.anne=this->t[i].p.anne; T[i].p.jour=this->t[i].p.jour; T[i].s.copy(this->t[i].s.str); }   // on ajoute rv this->t[i].p.mois=rv.p.mois; this->t[i].p.anne=rv.p.anne; this->t[i].p.jour=rv.p.jour; this->t[i].s.copy(rv.s.str);   // on détruit l'ancienne alloc mémoire et on fait pointer t sur la nouvelle delete this->t; this->t = T;   // on incrémente n this->n=this->n+1; }

---

[edit]j'ai corrigé t par T[/edit]

Pas si résolu que ça, tu as 2 new sans un seul delete.

Edit : désolé, j'ai pas été assez rapide, tu t'en es déjà rendu compte :)

Citation:

---

Envoyé par dot-_-net

[CODE]agenda::operator +(Rendezvous &rv){

int i=0;

Rendezvous *T=new Rendezvous[this->n+1];

for(i=0;i<this->n;i++){
T[i].p.mois=this->t[i].p.mois;
t[i].p.anne=this->t[i].p.anne;
t[i].p.jour=this->t[i].p.jour;
t[i].s.copy(this->t[i].s.str);
}

---

Il y a une erreur de majuscules là : t[i] et this->t[i] sont les mêmes éléments. essaie avec :

Code:

---

1 2 3 4 5 6

for(i=0;i<this->n;i++){ T[i].p.mois=this->t[i].p.mois; T[i].p.anne=this->t[i].p.anne; T[i].p.jour=this->t[i].p.jour; T[i].s.copy(this->t[i].s.str);

---

Citation:

---

Envoyé par jmv

Pas si résolu que ça, tu as 2 new sans un seul delete.

---

D’après mes connaissance je pense que new écrase la mémoire allouer et alloue une autre donc pas besoin a faire des deletes

Citation:

---

Envoyé par dot-_-net

D’après mes connaissance je pense que new écrase la mémoire allouer et alloue une autre donc pas besoin a faire des deletes

---

Non, new ne fait qu'allouer un nouvel espace mémoire.
En règle générale : à chaque new doit correspondes un delete.

Et oublie pas de faire en sorte que ton code soit exception-safe

Code:

---

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

int agenda::load(){ FILE *f; int i=0; f=fopen("agenda.dat","rb"); while(!feof(f)){ this->t=(Rendezvous *) realloc(this->t,i+1); fread(&this->n,sizeof(n),1,f); fread(&this->t[i].s,sizeof(this->t[i].s.str),1,f); fread(&this->t[i].p.jour,sizeof(int),1,f); fread (&this->t[i].p.mois,sizeof(int),1,f); fread(&this->t[i].p.anne,sizeof(int),1,f); i++; } fclose(f); return 0; }

---

pour quoi ca beug ici??:(

Citation:

---

Envoyé par dot-_-net

Code:

---

FILE *f;

---

---

Argh!!! ne me dis pas que tu ne peux pas utiliser les flux non plus :marteau: