Discussion :

[dot-_-net]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
	Rendezvous *t;
	int n;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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++;
}

[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.

[dot-_-net]

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

[loufoque]

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

[dot-_-net]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
 
}

[NewbiZ]

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.

[jmv]

je viens de le faire mais ca beug :beug:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 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

[dot-_-net]

Salut,
J'usqu'à la tout va bien 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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[dot-_-net]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[jmv]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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]

[jmv]

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

[Mendahor]

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

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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);

[dot-_-net]

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

[jmv]

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.

[loufoque]

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

[dot-_-net]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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??

[jmv]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

FILE *f;

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