Passer un tableau dynamique en paramètre

Question

Salut tlm! J'aimerais savoir si je peux ou non passer en param&#232;tre un tableau dynamique  dont je n'ai pas encore allou&#233; de m&#233;moire(allocation &#224; l'int&#233;rieur de la fonction). En gros, voila ce que je veux faire :

//variables globales


CString *tab;
CString *temp;
CString machaine = "test";
int nombre_element = 0;


void SetDynaArray(int &nb_element, CString tab[], CString temp[], CString elt)
{
 if (nb_element) 
 {
  tab= new CString[nb_element+1];
  for (int i=0; i<nb_element; i++)
   tab[i] = temp[i];
  temp = tab;     
 }  
 else  temp = new CString;
 {     
  temp[nb_element++] = elt;
 }  
}


Appel de la m&#233;thode :
    SetDynaArray(nombre_element, tab, temp, machaine);

Y a-t-il des erreurs et surtout comment puis-je r&#233;gler mon probleme de tableau dynamique?

Merci

Galmiza · Answer

Salut,

BYTE *pByte;
AllocateMemoryForpByte(&pByte);
delete [] pByte;

avec:
void AllocateMemoryForpByte(BYTE **ppData)
{
 *ppData = new BYTE[256];
}

cs_AlexMAN · Answer

Je ne vois aucun probleme a faire ca. Par contre, si les variables que tu passes en param sont 'globales', il n'y a aucun interet a faire ceci, puis ce que tu pourras acceder a celles-ci a l'interieur de la fonction que tu appelles. C'est ca ton probleme d'allocation dynamique ?

+2(p - n)

bathof · Answer

Salut,

tu peux passer sans probl&#232;mes un tableau dynamique en param&#232;tre, mais il y a
des choses &#224; savoir.

Quand tu cr&#233;es un tableau dynamique tu cr&#233;es en fait une zone m&#233;moire pour ton tableau dans la m&#233;moire vive qui est point&#233;e par un pointeur du m&#234;me type, donc CString* tableau, n'est pas un tableau mais un pointeur sur l'adresse de la premi&#232;re case m&#233;moire de ta zone allou&#233;.

Alors pour passer un tableau dynamique en param&#232;tre tu passes un pointeur sur
la zone m&#233;moire concern&#233;e.

Quand tu fais  : tableau = new CString [10];  tu cr&#233;es la zone m&#233;moire pour ton 
tableau de 10 de type CString et "new" vas te retourn&#233; l'adresse au pointeur tableau.

Attention n'oublie pas de supprimer ta zone m&#233;moire quand tu ne l'utilise plus, car
si tu ne d&#233;truits pas les tableaux syst&#233;matiquement la m&#233;moire vive vas satur&#233;e.
Ex : delete [] tableau;

Ex programme :

void creat(char* tab)
{
 tab = new char[11];
}

void main()
{
 char* tableau = NULL;

creat(tableau);

delete [] tableau;
}

++ et bonne chance.

BunoCS · Answer

Moi, je passerais les pointeurs, comme ceci:

void SetDynaArray(int &nb_element, CString *tab, CString *temp, CString elt)

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

xterminhate · Answer

Bathof, si ton explication est juste, tu es malheureusement l'auteur d'une belle fuite de mémoire avec ton code donné comme exemple.

Le prototype correct de ta fonction est "void( char * & tab )". Tu vois pourquoi ? Je sais qu'on ne peut pas modifier son post et c'est bien dommage.

Ca rejoint un topic déjà existant et tres récent. Le passage par référence rend l'écriture plus sympathique. La solution proposée par Galmiza est ok si on tient à faire un passage par pointeur (du pointeur [sic!]).

xterminhate · Answer

Bruno, je crains que tu sois confronté à des fuites....

bathof · Answer

Je crois que j'ai fais just car :

char* tab;

tab = &tab;

donc j'init le poiteur tab de la fonction par l'adresse de tableau de la
fonction.

char* tab = tableau;

est egale a

char* tab = &tableau;

Mais je retiens, merci et a +

bathof · Answer

A oui d&#233;sol&#233;, tu as raison, j'ai fais une gourde. Merci et a plus.

Correction :

void creat(char** tab){
 *tab = new char[10];
}

int main()
{
 char* tableau;

 creat(&tableau);
}

xterminhate · Answer

J'ai rien compris. Pour t'en convraince, tu remarqueras que dans ton code, apres l'appel de creat(), tableau est tjs égale à 0. Il n'est pas modifié dans le contexte appelant....

xterminhate · Answer

Amen! ;-)

bathof · Answer

Comme normalement je fais :

void creat(char*& tab)
{
 tab =new char[12];
}

int main()
{
 char* tableau;

 creat(tableau);

 delete [] tableau;
}

Alors je suis me suis tromp&#233;.

++ et merci, lol.

magic_Nono · Answer

beh vu que tu n'utilise pas la val de retour,

renvoi simplement le ptr de ton tableau avec une condition d'utilisation:

d&#233;sallouer absoluement la val de retour...



sinon, la soluce est celle du dbl ptr....

___________________________________________________________
Magicalement
Nono

xterminhate · Answer

... un std::vector resoud d'un coup tous ces problemes de mémoire.

freepouy18 · Answer

En fait entre temps j'avais trouv&#233; une sol qui se rapproche de celle de magic_Nono, voila ce que j'ai fai :



//**********Methode************************
CString *SetDynaArray(int &nb_element, CString elt)
{


  CString *tab;
 CString *temp;


  if (nb_element) 
  {
  temp= new CString[nb_element+1];
    for (int i=0; i<nb_element; i++)
      temp[i] = tab[i];
    tab = temp;     
  }  
  else  tab = new CString;
  {     
    tab[nb_element++] = elt;
  }  


 return tab;
}



//******Attributs de ma classe******
CString *Montab;


 


//********Appel *****************
Montab = SetDynaArray(nb_elt,"MonElt");


 



//********Dans mon Destructeur de Classe******
delete [] Montab;



Ai-je oubli&#233; de desallouer de la mem, je pense aux variables locales &#224; ma fonction "tab" et "temp"?Si oui, comment faire???

xterminhate · Answer

C'est n'importe quoi le code que tu nous as donn&#233;. Sans parler de la classe globale dont tu nous as pas montr&#233; le code encore. Je crains le pire.

D&#233;j&#224; un pointeur sur un CString, je trouve ca louche, compte tenu ce que tu en fais.

Pr&#233;cise ce que tu veux faire et on t'aide...

Cordialement,
Xterminhate.

freepouy18 · Answer

En fait j'ai une classe qui me permet de faire un traitement de donn&#233;es texte. Je parcours un document texte et j'ai un test qui si il est valid&#233; appel cette fonction. Ici, ma fonction permet de remplir un tableau de CString avec des donn&#233;es texte situ&#233;es &#224; une certaine position. Ce tableau est un attributs de ma classe puisqu'il va me servir dans une autre classe.

freepouy18 · Answer

je sais pas si j'ai &#233;t&#233; clair ou si cela suffit a me faire comprendre mais bon j'attends......

magic_Nono · Answer

a d&#233;faut d'oublier de la d&#233;sallouer, tu ne l'a pas allou&#233; du tt ce qui est pire...



sinon, 

comme le dit

Xterminhate

pr&#233;cise ce que tu veux....

mais c'est vrai que vu la maniere dont c'est partit, on est pas sur que
tu arrive qq part et encore moins pr&#232;s de ce que tu t'&#233;tais fix&#233; comme
objectif



Magicalement, all&#233; du courage, petit scarab&#233;
Nono

freepouy18 · Answer

mais si je l'ai allouer, et quand je fais :  tab = new CString     et     temp= new CString[nb_element+1]
Ce que je veux c'est juste savoir si je dois d&#233;sallouer la meme des variables locales tab et temp et comment?

Ma fonction renvoi bien un tableau de CString avec les donn&#233;es voulues.Elle sont stock&#233;es  dans mon tableau d&#233;clar&#233; en global dont je d&#233;salloue la mem dans le destructeur de classe. Maintenant, encore une fois,  je veux juste savoir comment d&#233;sallouer la mem des variables locales tab et temp?

Merci

Passer un tableau dynamique en paramètre

19 réponses

Votre réponse