Discussion :

[snicolay]

Bonjour,
j'ai récupéré le code suivant (C++) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
typedef struct 
{
	unsigned char		*cData;			/* partie data du message */
	unsigned short		nDataLength;		/* longueur de la partie data */
}  t_Message;
 
myfunction(t_Message message)
{ 
	CString	stStrData((char*)message.cData, message.nDataLength);
	/* etc. */
}

Et je dois le transformer en C => j'ai écrit ceci pour récupérer la chaîne de caractère passée en paramètre:

Code :

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

1
2
3
4
5
6
7
8
9
myfunction(t_Message message)
{ 
	char	*stStrData;
 
	stStrData= malloc(message.nDataLength+1);
	memcpy(stStrData, (char*)message.cData, message.nDataLength);
	stStrData[message.nDataLength] = 0;
	/* etc. */
}

=> Cela est-il correct ? N'y a-t-il pas plus simple ?
Merci

[Christophe]

Pour commencer quand tu as du code, il faut le mettre dans la balise code (le dièse dans l’éditeur de message).

Ensuite, je ne fais pas beaucoup de C++, mais je peux te dire que ta méthode n'est pas bonne.

En regardant la doc de cstring, j'ai vu qu'il y a une méthode c_str qui retourne le contenu du cstring en chaine équivalente à une chaine en C. Il est précisé aussi que c'est un pointeur sur le contenu interne du cstring et qu'il ne faut pas y toucher, il te faut donc la recopier. Tu fais ton malloc en récupérant la taille de ta chaine via la méthode size (en y ajoutant 1 pour le \0 de fin car je sais pas si size en tient compte), puis tu copie ta chaine via strcpy.

Code :

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

1
2
3
4
5
6
7
 
char *temp=objet_cstring.c_str();
char *chaine=malloc(objet_cstring.size()); // size retourne un size_t qui est normalement égal à un int
strcpy(chaine,temp);
...
...
free(chaine); // ne pas oublier de libérer la mémoire du malloc une fois plus necessaire.

Tu peux remplacer l'utilisation du temp par :

Code :

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

1
2
 
strcpy(chaine,objet_cstring.c_str());

[white_tentacle]

Si je comprends bien, tu réécris du code C++ en C ?

Ça m’a l’air correct, sauf que tu ne testes pas le retour de malloc (oui, personne ne le fait, c’est d’ailleurs pour ça qu’il ne faut pas faire du C).

[bacelar]

Pour @chrtophe, CString est une classe de gestion de chaines de caractère des MFC bien antérieur à la normalisation de la STL.

[snicolay]

Salut,
merci beaucoup pour vos réponses, oui, je réécris du code C++ en C...

@chrtophe, je ne comprends pas ta méthode: il n'y a pas de notion d'objet en C => d'une part CString est inconnu en C, donc objet_cstring.c_str() n'est pas une syntaxe reconnue.
D'autre part, en lisant la doc de CString, j'y lis:

CString( LPCTSTR lpch, int nLength );
lpch: A pointer to an array of characters of length nLength, not null-terminated.

Aussi, je ne suis pas sûr qu'il soit possible de faire un strcpy sur une chaîne non terminée.

Je vais donc me fier à l'avis de @white_tentacle et me dire que ça devrait fonctionner avec ce que j'ai écrit.
Merci

[Christophe]

Pour @chrtophe, CString est une classe de gestion de chaines de caractère des MFC bien antérieur à la normalisation de la STL.

Il n'y avait pas de précision sur l'utilisation de MFC.

@chrtophe, je ne comprends pas ta méthode: il n'y a pas de notion d'objet en C => d'une part CString est inconnu en C, donc objet_cstring.c_str() n'est pas une syntaxe reconnue.

Effectivement, j'avais pas bien compris, oublies ce que j'ai dit. Je pensais que tu convertissais un CString en char*

Par contre je connais pas ton niveau en C/C++ donc penses qu'en C les chaines de caractères n'existent pas, c'est traité comme un tableau de char terminé par \0 (ou un pointeur c). Quand tu fais le malloc, penses à ajouter la taille de ta chaine +1 (pour le \0), et je t'es dit une grosse connerie en plus, car le CString ne contient pas de \0 (du moins celui de la STL, pour la versin MFC je sais pas), donc strcpy va segfaulter, il faut remplacer par strncpy avec la taille de la chaine. L'usage de memcpy est aussi valide.

[snicolay]

Ok, merci

[Médinoc]

le CString ne contient pas de \0 (du moins celui de la STL, pour la versin MFC je sais pas)

La chaîne retournée par std::string::c_str() est bel et bien terminée par un \0, et c'est le cas aussi pour les équivalents dans CString.