Pourquoi la valeur change sans intervention?

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:

---

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:

---

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

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

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

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:

---

1 2 3 4

void maMéthode( char * & maChaine ){ ... }

---

Code:

---

1 2 3

char [80] maChaine; maMéthode( maChaine );

---

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:

   ---

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

   ---

Citation:

---

Envoyé par Médinoc

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

---

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

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

Merci à tous pour vos réponses.

Citation:

---

Envoyé par galak63

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....:oops:
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... :aie:

ça ressemblait à ça:

Code:

---

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

---

avec la définition comme ça:

Code:

---

1 2 3 4

void maMéthode( char * & maChaine ) { //code C }

---

Citation:

---

Envoyé par Médinoc

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à

Citation:

---

Envoyé par Sunsawe

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

---

L'hypothese de Luc devient de plus en plus probable...

Citation:

---

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... :aie:

---

Normal. Comme de toute facon le compilateur C++ n'aurait pas aime non plus l'appel, ca n'a pas d'importance.

Code:

---

1 2 3 4 5 6 7 8 9 10

extern "C" void maMéthode( char * maChaine );   void maMéthode( char * maChaine ) { //code C }   char table[UneTailleAdequate];   maMethode(table);

---

Devrait passer mais il y a le probleme de trouver UneTailleAdequate. La solution C++ c'est d'utiliser des std::string. Avec une interface compatible avec le C, differentes solutions sont possibles, la plus complete est de definir l'equivalent de std::string; alternativement passer aussi la taille allouee et retourner une erreur si maMethode a besoin de plus, a charge a l'appelant de faire des reallocations.

Mea culpa, le " *& " aurait été nécessaire si il voulait changer la valeur du pointeur a l'intérieur de sa méthode ;) Mais normalement a la compil ca devrait passer

Citation:

---

Envoyé par galak63

Mea culpa, le " *& " aurait été nécessaire si il voulait changer la valeur du pointeur a l'intérieur de sa méthode ;) Mais normalement a la compil ca devrait passer

---

Faire:

Code:

---

1 2 3 4

void f(char*&); ... char table[SIZE]; f(table);

---

ne doit pas passer a la compilation. table est un tableau qui est converti implicitement en un pointeur vers son premier element, donc un temporaire qui ne peut pas etre binde a une reference non constante.

Alors peux tu me dire la manière à adopter si tu veux changer la valeur de ton pointeur ?

Citation:

---

Envoyé par galak63

Alors peux tu me dire la manière à adopter si tu veux changer la valeur de ton pointeur ?

---

Ta methode est correcte pour modifier un pointeur. Le probleme, c'est que table n'est pas un pointeur mais un tableau.

Alors je précise ma question : comment fait-on pour allouer ce tableau dans la fonction

Citation:

---

Envoyé par galak63

Alors je précise ma question : comment fait-on pour allouer ce tableau dans la fonction

---

Code:

---

1 2 3 4

void f(char*&); ... char* foo = NULL; f(foo);

---

Quel est le probleme?

A part naturellement que vector est vraissemblablement une meilleure solution si on n'est pas restreint par la compatibilite avec le C.

Donc j'avais un peu raison aussi :D :yaisse2: :yaisse2:

Merci à tous,

J'ai utilisé la méthode proposée par Jean-Marc.Bourguet et Galak63 qui nécessite une taille adéquate de tableau.

La taille étant parfaitement connu, celà ne pose pas de soucis, même si je suis curieux de savoir comment on peut s'en sortir, si justement, cette taille n'est pas connu...

Tout fonctionne normalement maintenant.