Discussion :

[Sunsawe]

Salut à tous,

Je rencontre un problème qui m'intrigue.

J'ai une méthode qui renvoie qui produit un tableau de char et le retourne. elle est donc déclarée ainsi :

Code :

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

char * maMethode();

Je déclare un "char *" pour récupérer le contenu du tableau et l'utiliser. Seulement voilà au sortir de la méthode, tout est bon, le débugger indique bien les bonnes valeurs dans le tableau (une suite de chiffre). Deux lignes plus tard, alors que rien ne touche à ce "char *", le debugger indique que la valeur a changé et à la place j'ai des trucs bizarres. Je ne comprends pas d'où celà vient.

Visiblement, ce changement à une incidence puisque si je déclare en dur, la valeur du "premier" retour, le reste fonctionne normalement. Mais avec la nouvelle valeur, rien ne fonctionne.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
char * outText ={'\0'};
	outText= maMethode();
 
	int i = 0; //ici la valeur de outText est correcte, composée de chiffre
 
        int comp = a+3; //ici la valeur de outText est devenue ÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌHî
 
	while(outText[i]!='\0')
	{
               // faire des trucs
	}

Si quelqu'un peut m'expliquer ce qui se passe...

Merci

[Luc Hermitte]

Tu n'aurais pas renvoyé un pointeur vers un tableau statique local ? Ou un truc non alloué ?
Utilises les string plutôt.

[loufoque]

Tu ne gères manifestement pas la mémoire comme il faut.
Utiliser std::string est une bonne idée.

[galak63]

Et bien c'est tout simple, il doit déclaré un char * avec lequel il travaile dans "maMéthode()" et qu'il renvoit. Vu que la déclaration est faite dans "maMéthode()", à la sortie de "maMéthode()" cette variable et donc l'allocation qui en découle ne sont plus, donc le pointeur vers cette ancienne zone mémoire pointe "dans le vide" CQFD

Il faudrait plutot faire comme ceci :

Code :

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

1
2
3
4
 
void maMéthode( char * & maChaine ){
...
}

Code :

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

1
2
3
 
char [80] maChaine;
maMéthode( maChaine );

[Médinoc]

1. Le C++ déconseille les char* au profit des std::string.
2. Le & n'est pas nécessaire ici.
3. La taille du tableau n'est pas passée. Ce qui pourrait être intéressant, c'est une fonction template:

   Code :

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

   1
   2
   3
   4
   5
   6
   template< size_t N >
   void maMéthode(char (&maChaine)[N])
   {
      // Utiliser chaine, sachant que sa taille N est considérée 
      // comme une constante connue à la compilation.
   }

[Jean-Marc.Bourguet]

[*]Le & n'est pas nécessaire ici.

Passer un tableau a une fonction attendant un char *& ne devrait pas passer la compilation.

[Sunsawe]

Merci à tous pour vos réponses.

Et bien c'est tout simple, il doit déclaré un char * avec lequel il travaile dans "maMéthode()" et qu'il renvoit.

Presque Galak63, je ne sais pas si ça change quelque chose, mais c'est un tableau qui est déclaré à l'intérieur et qui est renvoyé. Mais ce que tu dis me parait pertinent.

Toute fois j'aurais surement du préciser un petit détail....
La fonction (que j'ai appelé maMethode) est en fait une déclaration liée avec du code en C.

J'ai tenté de mettre le & dans cette fonction en C, mais le compilateur n'a pas beaucoup aimé au niveau de la définition...

ça ressemblait à ça:

Code :

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

extern "C" void maMéthode( char * & maChaine );

avec la définition comme ça:

Code :

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

1
2
3
4
void maMéthode( char * & maChaine )
{
//code C
}

Ah, ok.
En lisant son post, j'ai cru qu'il voulait qu'il soit possible de passer un tableau...

Edit: Et après relecture, je le crois encore, en fait...

En fait, j'ai un tableau de char dans cette fonction, j'aimerai le récupérer. Quelque soit le moyen, mais en disposer sous forme de tableau de char ou quoi que ce soit qui me permette de bosser sur chaque char du tableau.

voilà