Discussion :

[tetedeturcq]

Bonjour, j'ai une application c++ qui fonctionne mal, je pense que le probleme vient de la memoire car j'ai une fonction qui remplit un tableau et le renvoit en retour, mais je ne vois pas comment ensuite liberer la memoire car la variable est locale à la fonction.

voici ma fonction:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
ns__classedetab transATVAL2(ATListe * apTab)
{
	ns__classedetab tab;
 
	tab.tabcoparam.__size = apTab->Count() ;
	tab.tabcotype.__size = apTab->Count() ;
	tab.tablibparam.__size = apTab->Count() ;
 
	tab.tabcoparam.__ptr = new char*[apTab->Count()*sizeof(char*)];
	tab.tabcotype.__ptr = new char*[apTab->Count()*sizeof(char*)];
	tab.tablibparam.__ptr = new char*[apTab->Count()*sizeof(char*)];
 
 
long LigTab;// ligne du tableau
 
for (LigTab = 0 ; LigTab < apTab->Count() ; LigTab ++ )
{
 	//on recupere la 1ere colonne du tableau
 	AF_char * ltabligne1 = apTab->gString(LigTab,0);
 
 	//on recupere la 2ieme colonne du tableau
 	AF_char * ltabligne2 = apTab->gString(LigTab,1);
 
	//on recupere la 3ieme colonne du tableau
 	AF_char * ltabligne3 = apTab->gString(LigTab,2);
 
 	//recopie des noms de fonctions contenues dans la 1ere ligne
 	char * nomfonc = (char *) ltabligne1 ;
 	//recopie les cotype des parametres contenus dans la 2ieme ligne
	char * cotype = (char *) ltabligne2 ;
	//recopie les libelles des parametres contenus dans la 3ieme ligne
	char * lib = (char *) ltabligne3 ;
 
     tab.tabcoparam.__ptr[LigTab] = new char[20];
     strcpy(tab.tabcoparam.__ptr[LigTab],  nomfonc );
 
     tab.tabcotype.__ptr[LigTab] = new char[20];
     strcpy(tab.tabcotype.__ptr[LigTab],  cotype );
 
     tab.tablibparam.__ptr[LigTab] = new char[20];
     strcpy(tab.tablibparam.__ptr[LigTab],  lib );
  }
return tab;
}

le code est un peu particulier car c'est pour le faire communiquer avec une page web, mais sur le principe des allocations c'est pareil.
quelqu'un pourrait-il m'aider?
Si je ne suis pas assez claire dite le moi. Merci.

[yorke]

tu devrais utiliser [CODE*][/CODE*] ca serait plus lisible.

Pourquoi pas creer l'objet dynamiquement, et renvoyer le pointeur ?

D'ailleurs si ns__classedetab est une classe, un constructeur adapté pourrait s'occupé de toutes les allocations.
Genre:

Code :

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

1
2
3
4
5
6
7
8
 
ns__classedetab *transATVAL2(ATListe * apTab)
{
    ns__classedetab *tab=new ns__classedetab(apTab->Count() );
 
    ...
    return tab
}

et tu surcharge ton constructeur de ns__classedetab:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
ns__classedetab::ns__classedetab(long count)
{
    this->tabcoparam.__size = count;
    this->tabcotype.__size = count;
    this->tablibparam.__size = count;
 
    this->tabcoparam.__ptr = new char*[count*sizeof(char*)];
    this->tabcotype.__ptr = new char*[count*sizeof(char*)];
    this->tablibparam.__ptr = new char*[count*sizeof(char*)];
}

Et tu fais le destructeur adapté pour librere toute la memoire.

[tetedeturcq]

ok, et quand est ce que j'appelle le destructeur? apres l'appel de la fonction?

[yorke]

Ben tu detruis ton objet quand tu en a plus l'utilité.

C'est une solution possible, mais y'en a d'autre.

[befalimpertinent]

Le mieux est peut être de gérer l'allocation en dehors de la fonction :
tu alloues ton tableau, passe le pointeur à la fonction, et tu le désalloues quand tu n'en n'a plus besoin.
Tu peux aussi vérifier dans la fonction si le tableau passé en paramètre est NULL. Dans ce cas soit tu renvoie un code d'erreur soit tu le créé en interne dans ta fonction.

[yorke]

Oui je pense aussi que c'est mieux de le passer en parametre, plutot que alloué la memoire dans une fonction et la supprimer ailleurs.

[smashy]

l objet que tu retourne est une copie de ton objet local a la fonction "ns__classedetab tab". Cette remarque conduit aux questions suivantes:

- existe t il un constructeur pour la classe ns__classedetab ?

- l operateur de recopie est il definie ?

- existe il un destructeur ? si oui , est ce qu il ne detruirait pas par hazard les pointeurs qu'il contient ?

note: l idee de faire un constructeur pour ns__classedetab est plutot bonne, mais une bonne regle de programmation est que :

si un objet a besoin de constructeur / destructeur / operateur de copie il a besoin des 3.

[smashy]

ok, et quand est ce que j'appelle le destructeur? apres l'appel de la fonction?

le destructeur d un objet est appele implicitement :

- soit a la sortie du bloc auquel il appartient (une fonction est un bloc) s il est alloue sur la pile (c est le cas de ton objet)

- soit quand tu appelles delete si il est alloue dans le tas

[loufoque]

Utilise std::string et std::vector.