Discussion :

[JeanNoel53]

bonjour,
j'ai un test qui échoue:
le valeur à tester est:

Code :

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

pWord->wsWord = {"deactivates"}

quand Je rentre dans la méthode wcsxcmp j'obtiens:

Code :

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

1
2
un = 0x09422984 "the"
et deux = 0x01042DA4 "deactivates"

je sort de la méthode avec -1
le code sort donc avec b0 = false

Code :

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

1
2
3
4
					 b0 = CStrConv::wcsxcmp( (wchar_t*)pWord->wsWord.c_str(), L"deactivates") != -1;
					 b0 = CStrConv::wcsxcmp( (wchar_t*)pWord->wsWord.c_str(), L"deactivates") != -1;
					 b0 = CStrConv::wcsxcmp( (wchar_t*)pWord->wsWord.c_str(), L"deactivates") != -1;
					 b0 = CStrConv::wcsxcmp( (wchar_t*)pWord->wsWord.c_str(), L"deactivates") != -1;

qui m'aidera sera

[Médinoc]

Qu'est censé faire wcsxcmp()? Peut-on voir sa documentation (ou à défaut, son code)?

[JeanNoel53]

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
	int CStrConv::wcsxcmp( wchar_t* un, wchar_t* deux)
	{
		int i = 0;
		if( wcslen( un) == 0 || wcslen( deux) == 0)
			return( - 1);
		while( un[i] != 0 && deux[i] != 0)
		{
			if( (un[i] == 0 && deux[i] != 0) || (un[i] != 0 && deux[i] == 0))
				return( -1);
			if( un[i] != deux[i])
				return( -1);
			i++;
		}
		return( 0);
	}

[Médinoc]

OK, voici ce que je comprends de la fonction en question:

Code C++ :

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
int CStrConv::wcsxcmp(wchar_t * un, wchar_t * deux) //Ces pointeurs devraient être const
	int i = 0;
	//Retourne -1 si une des deux chaînes est vide
	if( wcslen(un) == 0 || wcslen(deux) == 0 )
		return( - 1);
 
	while( un[i] != L'\0' && deux[i] != L'\0' )
	{
		//Si l'une des deux chaînes se termine mais pas les deux, retourne -1
		//Sauf qu'on n'arrivera JAMAIS à ce test dans une condition où il peut être vrai, à cause de la condition du while!
		if( (un[i] == L'\0' && deux[i] != L'\0') || (un[i] != L'\0' && deux[i] == L'\0'))
			return( -1);
		//S'il y a la moindre différence (ce qui inclut complètement le précédent test), retourne -1
		if( un[i] != deux[i])
			return( -1);
		i++;
	}
	return( 0);
}

Si j'ai bien compris, ce code retourne zéro si les chaînes sont identiques ou si une des deux chaînes est préfixe de l'autre, et retourne -1 sinon.

Dans ce cas:

1. À quoi t'attends-tu? Avec ça, il est parfaitement normal que wcsxcmp("the", "deactivates") retourne -1.
2. Ce code est une façon plus longue, plus maladroite et mal constée de dire:
   

   Code C++ :

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

   1
   2
   3
   4
   5
   6
   7
   8
   9
   10
   11
   int CStrConv::wcsxcmp(wchar_t const * un, wchar_t const* deux)
   {
   	size_t length1 = wcslen(un);
   	size_t length2 = wcslen(deux);
   	size_t minLength = (length1 < length2) ? length1 : length2;
   	if(minLength == 0)
   		return -1;
   	if(wcsncmp(un, deux, minLength) != 0)
   		return -1;
   	return 0;
   }

   Récupère cette version-là de la fonction, et en bonus, tu pourras supprimer ces affreux casts en (wchar_t*).

Edit: Ah, apparemment ton problème c'est que pWord->wsWord se transforme mystérieusement de "deactivates" en "the"? Ben dans ce cas, tu dois avoir un bug, mais ce n'est pas ici qu'il est. Es-tu sûr que ton code n'est pas multi-thread, pour commencer?

[JeanNoel53]

C'est bien le passage de paramètre qui est en cause
je ne sais pas comment savoir si je suis multi thread
mais quand je test je met un point d'arrêt avant la méthode, ce qui devrait arrêter tous les thread
J'ai modifié l'appel de la méthode

Code :

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

b0 = CStrConv::wcsxcmp( ( wchar_t const*)pWord->wsWord.c_str(), L"deactivates") != -1;

et ça n'a rien changé
J'ai aussi essayé sans succès

Code :

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

 b0 = CStrConv::wcsxcmp( ( wchar_t const*)pWord->wsLemma.c_str(), L"deactivates") != -1;

finalement J'ai eu recours a la fonction de la librairie std, ce qui a marché

Code :

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

b0 = wcscmp( ( wchar_t const*)pWord->wsLemma.c_str(), L"deactivates") != -1;

[Médinoc]

Attention, les valeurs de retour de wcscmp() ne sont pas cantonnées à 0 et -1!
En fait, selon la norme elle pourrait retourner n'importe quelle valeur qui tient dans un int, la seule valeur particulière étant zéro.

PS: Qu'est-ce que c'est que ce bordel de ( wchar_t const*)pWord->wsLemma.c_str()!! wstring::c_str() retourne déjà un wchar_t const *!

[JeanNoel53]

la question de fond est que se passe t-il quand vcscmp() rencontre deux mots différents

[Médinoc]

Ne sais-tu pas lire la doc?

Quand wcscmp() rencontre deux mots différents, elle retourne une valeur différente de zéro.