Discussion :

[globule71]

Bonjour,

J'ai affaire à un drôle de bug lorsque j'utilise la classe CString et surtout lorsque celle-ci fait appel à l'opérator+ visiblement.
Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
 
CString Query="UPDATE "+m_Tables+" SET ";		
for(int i=0;i<(int)m_pFX.m_TableauColonnes.m_nbElem;i++)
		{
			if(i==40)
				i=40;
			Query=Query+m_pFX.m_TableauColonnes.m_pTabString[i]+CString(" = '")+m_pFX.m_TableauValeurs.m_pTabString[i]+CString("',");
 
		}

Je m'explique : le if(i==40) me permet juste d'avoir un point d'arrêt à cet endroit car c'est à ce moment que la recopie ne se passe pas comme il faut.
Lorsque j'ajoute le dernier element, la variable Query qui avait accumulé toutes mes chaines repassent à la valeur "". Pourtant je ne fais que des concaténations...
Est ce que quelqu'un aurait une idée là dessus ?

[farscape]

salut,
attention en debug trace la visualisation de la CString est limitée a 256 caractéres (du moins avec vc6.0).
pour visualiser la chaine il vaut mieux faire un AfxMessageBox et ne pas inspecter la variable en mode trace..
Sinon la vérité est ailleurs, la CString fonctionne très bien sans bug connu .
je m'en suis déjà servi dans le même contexte que toi ( la génération d'une requete SQL) sans problème.
quel est la nature de l'objet m_pTabString,il ne faudrait pas faire * ?

[globule71]

Ah, ça je ne le savais pas... Mais est ce que ça explique le remplacement complet de ma chaine de caractère par une chaine vide à la dernière concaténation ? Par ce que si c'était limité à 256 caractères, j'arriverai normalement à suivre la trace des 256 premiers caractères et non pas une chaine vide ?

edit : m_pTabString[i] est un tableau de CString donc normalement pas de déférencement... Et puis ça marche pour les premières chaines alors je vois pas pourquoi ça ne marcherai pas pour la 40eme chaine

[farscape]

Ah, ça je ne le savais pas... Mais est ce que ça explique le remplacement complet de ma chaine de caractère par une chaine vide à la dernière concaténation ? Par ce que si c'était limité à 256 caractères, j'arriverai normalement à suivre la trace des 256 premiers caractères et non pas une chaine vide ?

non ça n'explique pas tout, mais tu n'as pas repondu à

quel est la nature de l'objet m_pTabString,il ne faudrait pas faire * ?

[globule71]

Desolé j'avais pas vu ta dernière question

[globule71]

Bon j'ai tenté d'arrêter la recopie dans la boucle du for juste avant la dernière chaine pour récupérer une chaine remplie... et j'ai esseyé de remplir avec le dernier element hors de la boucle : mais cette fois ci ça plante méchamment sur l'operator+ qui reçoit en paramètre un pointeur de la chaine à concaténer ainsi que.... un pointeur vide !
Visiblement à l'appel de l'opérator+ je perds carrément le pointeur du CString
Pour t'as question si t'as pas vu le edit du dessus j'ai bien un tableau de CString avec le m_pTabString

[farscape]

je peux voir la declaration ?
si tu es hors indices ça plante.
il vaut mieux utiliser un CArray<CString ,const char *> pour le tableau et utiliser la fonction GetCount() du CArray pour avoir la taille et iterer...

[globule71]

ouhla je suis mal barré si je dois refondre la classe de mon collègue...
Bon j'ai réussi à contourner le probleme une fois... Dans les requetes il y avait des espaces je les ai supprimés vu qu'on en a pas besoin pour les requètes sql du coup ça ne plante plus au même endroit mais plus loin dans le même type de boucle par contre là la technique du "je supprime les espaces " ne marche pas lol
Bon pour les déclarations je suis un peu pommé je vais voir comment il les déclare mais bon normalement j'avais pas trop à toucher à sa classe

[globule71]

Au passage ça ne plante pas sur une histoire d'indice mais bel et bien bcp plus tard lorsque j'ai besoin de cette chaine qui normalement doit être remplie mais se retrouve vide !
C'est vraiment lors de la concaténation que ça plante et je vérifie bien ce que je concatène : ça n'est pas vide du tout ! Mais ce qui est bizarre c'est que l'operator+ n'arrive pas à chopper les paramètres qui lui sont passés au bout d'un moment (surtout l'ancienne chaine ...)

[farscape]

si tu debordes au niveau de l'indice tu planteras l'operation + .
tant que l'on ne sera pas de quoi on parle au niveau de cette variable tableau on ne pourra rien dire de plus...

[globule71]

Voilà en creusant plus loin j'atteris ici :

Code :

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

1
2
3
4
5
6
7
8
	friend CStringT operator+( __in const CStringT& str1, __in const CStringT& str2 )
	{
		CStringT strResult( str1.GetManager() );
 
		Concatenate( strResult, str1, str1.GetLength(), str2, str2.GetLength() );
 
		return( strResult );
	}

A savoir : str1 et str2 sont bien remplie comme il faut mais lors de l'opération concatenate celle ci ne rempli pas strResult comme il faut (il devrait y avoir une chaine remplie et non une chaine vide ("")

[farscape]

je le redis encore la faute n'est pas sur CString mais de ton environnement, tes chaines ne doivent pas être valide.
vérifie la taille physique de ton tableau .