Discussion :

[vonemya]

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 :

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

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 :

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

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

qualqu'un a une idée?

Merci

[Médinoc]

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

[vonemya]

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

Donc je fais autrement

Code :

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

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 :

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

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

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!

[Médinoc]

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

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

[vonemya]

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 :

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

1
2
3
4
5
6
7
char * NomFichier = "tagada";
        CString maChaine;
        maChaine.Format( "youppiii");
	NomFichier = CStringToChar (maChaine);        
	AfxMessageBox(NomFichier);
        maChaine.ReleaseBuffer();

[*]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!

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

Code :

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

1
2
3
4
 
         CString maChaine ;
        char * NomFichier = maChaine.GetBuffer(0);       
	AfxMessageBox(NomFichier);

ne marche pas!

[mchk0123]

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

Code :

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

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 :

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

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édinoc]

Code :

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

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

Ceci devrait marcher.