Discussion :

[thelpi]

salut,

je suis confronté à des interrogations dans mon projet c# concernant la convertion en int, je ne sais pas vraiment quel méthode utilisé et comment.

je m'explique:

je crée des requète, du type

Code :

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

string chSqlCount = "select count(*) from table1, table2 where ..."

je crée ensuite ma commande

Code :

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

1
2
IDbCommand cmd = maConnection.CreateCommand();
cmd.CommandText = chSqlCount;

et, en attendant de trouver mieux, je récupère le résultat comme ceci :

Code :

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

1
2
int nbRow;
try { nbRow = Convert.ToInt32(cmd.ExecuteScalar()); } catch { nbRow = 0; }

sachant :
_que la chaîne de la requète est créer dans une autre classe, et que je ne pourrais jamais être sur à 100% quel soit d'une syntaxe correct

_que les performances de l'application sont primordiales et que j'ai l'impression que mon code est plutôt lent (mais peut ètre est-ce du à mes requètes plus qu'au try/catch)

Que me conseilleriez vous pour améliorer tout ça? (si c'est possible) les try/catch sont t'ils indispensable?

merci d'avance

[ArianeV]

Il me semble que si tu passes par un datareader (donc comme si ta requête renvoyait plusieurs lignes qu'il faut lire séquentiellement), tu peux appeler une méthode du genre ReadAsInt (sur le datareader) ou quelque chose comme ca, qui s'occupe de la conversion à ta place.

Conceptuellement, c'est plus propre : c'est dans la couche DB que tu vas faire la conversion, puisque c'est la que tu lis le résultat. C'est peut être aussi plus performant (même si ce n'est pas évident).

Par contre tu ne peux pas t'affranchir du try / catch.

J'espère que ça pourra t'aider.

Bonne journée

[Graffito]

mon code est plutôt lent : mais peut ètre est-ce du à mes requètes plus qu'au try/catch)

l'exécution du catch est effectivement lente.
Pour une conversion de string en int, utiliser : i:=int.TryParse(MyString);

[thelpi]

merci pour vos réponses,

Graffito, malheureusement ce n'est pas toujours (voir quasiment jamais) du string que je doit parser je crois qu'il n'y a pas d'équivalent à TryParse pour les autres types, à moins de faire :

Code :

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

int.TryParse(monObjet.ToString());

mais la je me demande si je gagne beaucoup en performance de plus je ne sais pas si le toString lève parfois des exceptions sur certains types particuliers...

je vais tester la méthode d'ArianeV pour le moment et voir si les perfs sont meilleures.

[Graffito]

je me demande si je gagne beaucoup en performance

Tu gagneras beaucoup, si tu passe souvent dans le catch (j'en ai fait l'expérience).