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...