Problème de conversion de CString en char *

Salut,

J'ai un problème de conversion d' un CString en char *. J'ai fait des recherche sur le net et voilà le code que j'utilise:

Code:

---

1 2 3 4 5 6 7 8 9 10

//convertir une CString en un char * char * CStringToChar(CString chaineAConvertir) { AfxMessageBox(chaineAConvertir); char * szPtr = NULL; szPtr = chaineAConvertir.GetBuffer(2); AfxMessageBox(szPtr); chaineAConvertir.ReleaseBuffer(); return szPtr; }

---

Avec les message box je vérifie que j'ai la bonne chaine, notamment szPtr. Mais quand je récupere la valeur retournée par cette fonction en faisant

Code:

---

1 2 3 4

char * NomFichier = "tagada"; NomFichier = CStringToChar ("youppiii"); AfxMessageBox(NomFichier);

---

L'ordre de l'affichage est le suivant "youppiii", "youppiii" (correspondant au AfxMessageBox à l'interieur de CStringToChar) et puis le 3ème AfxMessageBox affiche une suite de caractères bizaaaaares 8O

qualqu'un a une idée?

Merci

Tu retournes un pointeur vers un buffer que tu viens de releaser ET de détruire...

Si tu tiens à convertir une CString en char*, utilise l'allocation dynamique (malloc(strlen())+strcpy(), ou _strdup()).

PS: Pourquoi chercher à faire une telle conversion? Es-tu sûr que tu ne veux pas simplement un const char * ?
La conversion d'une CString en LPCTSTR est implicite, mais pose les mêmes problèmes de portée (le pointeur n'est garanti valide que tant que la CString n'est absolument pas modifiée).

Citation:

---

Envoyé par Médinoc

Tu retournes un pointeur vers un buffer que tu viens de releaser ET de détruire...

---

Donc je fais autrement

Code:

---

1 2 3 4 5 6 7 8 9 10

char * CStringToChar(CString chaineAConvertir) { AfxMessageBox(chaineAConvertir); char * szPtr = NULL; szPtr = chaineAConvertir.GetBuffer(2); AfxMessageBox(szPtr); //je ne détruis pas le buffer //chaineAConvertir.ReleaseBuffer(); return szPtr; }

---

et puis je fais l'appel suivant:

Code:

---

1 2 3 4 5 6 7

char * NomFichier = "tagada"; CString maChaine; maChaine.Format( "youppiii"); NomFichier = CStringToChar (maChaine); AfxMessageBox(NomFichier); maChaine.ReleaseBuffer();

---

Mais ça marche toujours pas, pourtant je fais l'affichage avant qu je release le buffer, je comprends pas 8O

Citation:

---

Envoyé par Médinoc

PS: Pourquoi chercher à faire une telle conversion? Es-tu sûr que tu ne veux pas simplement un const char * ?
La conversion d'une CString en LPCTSTR est implicite, mais pose les mêmes problèmes de portée (le pointeur n'est garanti valide que tant que la CString n'est absolument pas modifiée).

---

justement y a des traitements sur la CString, sonc elle est modifiée!

Ta fonction bosse sur une copie de la chaîne, et cette copie est détruite lors du return...

Citation:

---

justement y a des traitements sur la CString, sonc elle est modifiée!

---

• AfxMessageBox() ne modifie pas la chaîne et peut se contenter d'un simple LPCTSTR.
• Si tu récupères un char* pour modifier la chaîne, alors inutile de passer par une fonction intermédiaire: Utilise directement GetBuffer()...

Citation:

---

Envoyé par Médinoc

Ta fonction bosse sur une copie de la chaîne, et cette copie est détruite lors du return...

---

oué mais je récupère sa valeur dans la variable nomFichier

Code:

---

1 2 3 4 5 6 7

char * NomFichier = "tagada"; CString maChaine; maChaine.Format( "youppiii"); NomFichier = CStringToChar (maChaine); AfxMessageBox(NomFichier); maChaine.ReleaseBuffer();

---

Citation:

---

Envoyé par Médinoc

[*]AfxMessageBox() ne modifie pas la chaîne et peut se contenter d'un simple LPCTSTR.

---

c'est pas ça le traitement, c'était juste pour m'assurer de la valeur de la variable, c'est tout!

Citation:

---

Envoyé par Médinoc

Si tu récupères un char* pour modifier la chaîne, alors inutile de passer par une fonction intermédiaire: Utilise directement GetBuffer()...

---

Meme ce qui suit ne marche pas :aie:

Code:

---

1 2 3 4

CString maChaine ; char * NomFichier = maChaine.GetBuffer(0); AfxMessageBox(NomFichier);

---

ne marche pas!

Je ne connais pas précisément l'utilisation de la classe CString, mais le code suivant :

Code:

---

1 2 3 4 5 6 7 8 9

char * CStringToChar(CString chaineAConvertir) { AfxMessageBox(chaineAConvertir); char * szPtr = NULL; szPtr = chaineAConvertir.GetBuffer(2); AfxMessageBox(szPtr); //je ne détruis pas le buffer //chaineAConvertir.ReleaseBuffer(); return szPtr; }

---

m'interpelle.

Tu passe l'argument chaineAConvertir par valeur au lieu d'un passage par adresse ou référence. Du coup la valeur szPtr renvoyée peut ne plus exister lorsque la fonction se termine. Essaye plutôt ceci :

Code:

---

1 2 3 4 5 6 7 8 9

char * CStringToChar(CString &chaineAConvertir) { AfxMessageBox(chaineAConvertir); char * szPtr = NULL; szPtr = chaineAConvertir.GetBuffer(2); AfxMessageBox(szPtr); //je ne détruis pas le buffer //chaineAConvertir.ReleaseBuffer(); return szPtr; }

---

Code:

---

1 2 3

CString maChaine = "foo" ; char * monCharEtoile = maChaine.GetBuffer(0); AfxMessageBox(monCharEtoile);

---

Ceci devrait marcher.