Discussion :

[abelman]

Bonjour,

Depuis 5 ans j'ai ce bout de code qui fonctionne sans souci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
// fonction
CString ToSQLString(CString sql)
{
	sql.Replace("'", "''");
	sql.Format("'%s'", sql);
	return sql;
}
 
//et dans le code appellant
CString s = ToSQLString(CTime::GetCurrentTime().Format("%H%M%S"))

jusqu'à present j'avais toujours
s = '165517' // par exemple.

Depuis aujourd'hui, la même fonction renvoit des fois
s = ''''''''

Quelqu'un a une explication?

[farscape]

salut,
ton projet n'est pas en UNICODE des fois ?

[Médinoc]

Je ne suis pas certain que ça compilerait si c'était le cas.

Non, ça ressemble plus à un changement dans l'implémentation de la fonction CString::Replace()...

[abelman]

salut,
ton projet n'est pas en UNICODE des fois ?

Non. Jeu de caractère non défini

Médinoc, sql.Replace retourne bien ce qu'il me faut.
C'est sql.Format qui 'foire'. J'ai comme l'impression que faire sql.format("%s",sql) ne lui plait pas.

Quand je passe par une étape intermédiare comme ci dessous, ça marche très bien.

Exemple.

Code :

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

1
2
3
4
5
6
7
8
 
CString ToSQLString(CString sql)
{
	CString s;
	sql.Replace("'", "''");
	s.Format("'%s'", sql);
	return s;
}

Quelqu'un peu essayer chez lui les deux versions de ToSqlString avec le paramètre CTime::GetCurrentTime().Format("%H%M%S") ?

Merci

[farscape]

alors essaye plutôt ceci:

Code :

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

1
2
s.Format("'%s'",static_cast<const char *>( sql));

[abelman]

ça marche aussi ... tout comme ça

Code :

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

1
2
3
4
5
6
7
CString ToSQLString(CString sql)
{
	CString s;
	sql.Replace("'", "''");
	s.Format("'%s'", sql);
	return s;
}

C'est pareil.

[Médinoc]

farscape: Ton code force la CString à forker, c'est ça ?

En tout cas, je pense que le sql.Format("'%s'", sql) est un comportement aussi indéfini que sprintf(str, "%s", str) ---> Un truc à bannir, quoi...

[abelman]

sprintf(str, "%s", str) ---> Un truc à bannir, quoi...

C effectivement l'équivalent en C.

Pourquoi c'est indéfini ce truc? C'est ça que j'aimerai comprendre.
5 ans que l'appli tourne et boum d'un coup ça plante à cet endroit.

[farscape]

c'est indefini à mon avis ,parce que format ou sprintf utilise une signature argument variable .
comment dans ce contexte le compilo peut-il faire pour choisir l'operateur aproprié ?
si je mets const char *, c'est pour forcer l'appel de l'operateur de la CString...

ps:j'ai vu beaucoup de gens ne pas mettre le const char * dans ce contexte avec vc6 ,j'ai toujours pensé que c'était une erreur à ne pas faire ,et je me suis toujours demandé comment ça pouvait fonctionner ....

[Médinoc]

Avec le sprintf(), tu lis et tu écris au même endroit, pire, tu INSERES des caractères. Ce qui me surprend, ce n'est pas que ça ne marche plus, mais que ça ait marché avant...

Je suppose que CString::Format() fait plus ou moins la même chose si son buffer est assez grand pour qu'il n'ait pas à le réallouer...

[farscape]

la difference avec format ,c'est qu'il y a une pre analyse des arguments pour estimer la longueur de la chaine à allouer.

[abelman]

Ok Merci ... et à l'année prochaine