Discussion :

[WELCOMSMAIL]

comment savoir directement la variable qui a cree une exception dans le code en lisant le message d'erreur?
je pense que ce bout de code qui m'a cree une exception. et le programme crash directement.
en faisant un debug j'ai trouvé que les dernieres instructions exécuté sont ce bout de code.

est ce qu'il y a un autre moyen de liberer la memoire allouée pour la variable ch?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
char *ch;
WCHAR *pStr2;
 
for (int i =1; i<1000;i++)
{
ch =new char[string(OLE2T(pStr2)).length() + 1];
memset(ch, '\0', string(OLE2T(pStr2)).length() + 1);
memcpy(ch, string(OLE2T(pStr2)).begin(), string(OLE2T(pStr2)).length());
printf("%s\n",ch);
delete(ch);
}

[Médinoc]

J'ai des doutes là-dessus:

Code :

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

memcpy(ch, string(OLE2T(pStr2)).begin(), string(OLE2T(pStr2)).length());

Tu crées deux strings différentes, là, non?

Ne serait-il pas plus prudent de créer une fois pour toute la string dans ton itération?

De plus, OLE2T n'est pas la bonne macro pour les types que tu utilises.
Si tu veux vraiment convertir de WCHAR à char, tu dois utiliser W2A...

[WELCOMSMAIL]

merci pour ta rapide reponse.
le probleme c'est que ptr2 change pour chaque iteration (bon la boucle for est mal placée mais ptr2 est lue à chaque iteration.
a ton avis quelle est la meilleur façon de convertir wchar en char

[Médinoc]

Généralement, j'utilise directement les fonctions de conversion de l'API Win32 : WideCharToMultiByte() notamment...

Ou bien si on veut utiliser du C++ standard, un std::ostringstream dans lequel on insère une std::wstring, ça marche...
Ou encore, toujours en C++ standard, il y a quelque part une classe template qui contient des fonctions narrow() et widen()...

PS: J'ai dit une fois pour toutes dans ton itération, pas une fois pour toutes dans ta boucle...

[WELCOMSMAIL]

je te remercie pour ton aide.
j'ai trouvé cette fonction "wcstombs" qui m'as resolu le probleme de crash.

Code :

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

1
2
3
4
5
6
 
size_t pStr2_size;	
char *ch           ;
pStr2_size = wcslen(pStr2) + 1;
ch =new char[pStr2_size];
wcstombs(ch, pStr2, pStr2_size);

[Médinoc]

ERREUR.

Tu ne dois pas utiliser directement la taille de la chaîne d'origine pour la chaîne de destination.
Tu dois appeler deux fois wcstombs() (la première fois avec NULL en destination), et utiliser comme taille le résultat du premier appel.

[WELCOMSMAIL]

C'est à dire? comment ça devient cet exemple?
car j'ai mis ça

Code :

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

1
2
3
4
5
6
 
pStr2_size=wcstombs(NULL, pStr2, pStr2_size);
 
ch =new char[pStr2_size];
 
wcstombs(ch, pStr2, pStr2_size);

et les chaines sont tronquées( le debut c bon et le reste de la chaine des caracteres inconnus(illisibles)

[Médinoc]

Tu oubliais le zéro terminal ET tu passais un pointeur non-initialisé...

Un truc de ce genre devrait marcher :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <cassert>
#include <cstdlib>
 
//...
 
{
	//Taille du buffer: Valeur retournée par wcstombs + zéro terminal.
	size_t cchLengthA = wcstombs(NULL, pStr2, 0);
	size_t cchSizeA = cchLengthA+1;
 
	//Allocation
	char *sDest = new char[cchSizeA];
 
	//Conversion
	int res = wcstombs(sDest, pStr2, cchSizeA);
	if(res != cchLengthA)
	{
		//Erreur.
		std::cout << "Erreur de conversion" << std::endl;
	}
	else
	{
		//Conversion terminée.
		std::cout << "Converti : \"" << sDest << "\"" << std::endl;
 
		//Vérification avec une assertion :
		//La condition devrait TOUJOURS être vraie.
		assert(sDest[cchLengthA] == '\0');
	}
}

Glossaire: cch = compte de caractères (par opposition à cb = compte d'octets)
Length (longueur) = taille de la chaîne, sans zéro terminal.
Size (taille) = taille de la chaîne avec zéro terminal.

[WELCOMSMAIL]

ok ça marche
mais quelle est la difference entre utiliser :

Code :

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

1
2
 
pStr2_size = wcslen(pStr2) + 1;

et utiliser :

Code :

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

1
2
3
 
size_t cchLengthA = wcstombs(NULL, pStr2, 0);
size_t cchSizeA = cchLengthA+1;

[Médinoc]

La différence :
C'est garanti marcher, même avec une page de code double-byte.

Avec ton code, si UN SEUL des caractères doit se coder sur deux octets (caractères orientaux, notamment) dans la page de code courante, c'est le dépassement de buffer garanti.
Microsoft nous propose un moyen de faire marcher à coup sûr, pourquoi s'en priver?

Dernière minute : De plus, wcstombs est aussi disponible sur les OS unixoïdes, et beaucoup de Linux sont en UTF-8 de nos jours (et en UTF-8, BEAUCOUP de caractères prennent deux octets, à commencer par nos accents...).

[WELCOMSMAIL]

Je vous remercie enormement pour toutes vos explications.