Discussion :

[ThistleDub]

Bonjour à tous,

j'aurais voulu savoir la façon la plus performante de valoriser un objet métier à partir du résultat d'une requête (datatable).

J'ai par exemple un objet User avec les propriétés Nom et Prenom. Je voudrais remplir une List<User> (ou dico) à partir du résultat de ma requête sans faire un foreach sur les Rows de ma DataTable.

Merci pour votre aide !

[ostenhard]

Bonjour,

Des bonnes pratiques, je ne sais pas vraiment, mais en tous cas, éviter le for each je ne vois (à moins qu'en jouant sur les mots, tu acceptes le for int) !

Personnellement, je créé une fonction "mapper" par type d'objet à mapper et j'ai une méthode générique qui à partir d'un DataReader ou une DataTable bouclant sur chaque Row appelle la méthode "mapper" créant chaque occurrence d'objet (en mappant tous les champs) !

Je ne sais si c'est la meilleure façon, mais elle est plutôt générique je trouve !

[ThistleDub]

Ok en fait je me demandais s'il n'y avait pas de méthode plus performante.
C'est comme cela que je fais aussi mais je trouvais cela un peu lourd.

[ostenhard]

Lourd en code ? ou lourd en perf ?

En code, la méthode générique doit tenir en 10 lignes voir 20.
Les fonctions mapper autant de lignes que de champs !

[ThistleDub]

Non lourd en Perfs !
Si tu regardes ma question de base :

j'aurais voulu savoir la façon la plus performante

Maintenant s'il n'y a pas d'autre façon de faire...

Merci de ta réponse en tout cas !

[ostenhard]

Ok, enfin tu sais des fois la plus performante ne prend pas le sens que tu utilises là.

As-tu déjà essayé de ne pas utiliser une datatable mais directement un datareader ?

Sinon tu sembles dire que c'est lent, peut-on avoir une idée de la volumétrie ?
Comment effectues-tu le mappage de la datatable et ton objet (exemple de code) ?
As-tu réalisé des relevés de perfs pour identifier les éventuels goulots d'étranglement ?

[Bluedeep]

Bonjour

j'aurais voulu savoir la façon la plus performante de valoriser un objet métier à partir du résultat d'une requête (datatable).

J'ai par exemple un objet User avec les propriétés Nom et Prenom. Je voudrais remplir une List<User> (ou dico) à partir du résultat de ma requête sans faire un foreach sur les Rows de ma DataTable.

Dans ce cas, je ne comprends pas pourquoi tu veux passer par une DataTable. En effet, si l'objectif est d'être le plus performant possible, le plus silmple c'est de passer directement du reader à la List; on ne voit pas bien l'interêt ici de passer par l'étape DataTable.

[ThistleDub]

As-tu déjà essayé de ne pas utiliser une datatable mais directement un datareader ?
on ne voit pas bien l'interêt ici de passer par l'étape DataTable.

Oui biensûr, l'intitulé de ma question ne reflète effectivement pas ce que je fais effectivement dans mon code sur ce point là, car je passe par un reader.
Effectivement aucun intérêt.

Sinon tu sembles dire que c'est lent, peut-on avoir une idée de la volumétrie ?

Pour le moment en base Test, je n'ai que 3600 lignes dans ma table, donc le traitement prend moins d'une seconde. Mais à terme, avec 200 000 ou 300 000 lignes je demande à voir.
Il faut que je fasse des tests dans ce sens.

As-tu réalisé des relevés de perfs pour identifier les éventuels goulots d'étranglement ?

Non j'avoue que je ne suis pas allé jusque là autrement qu'en mettant des traces horodatées.
Si des outils sont performants dans ce domaine, je suis preneur !

[Bluedeep]

Si des outils sont performants dans ce domaine, je suis preneur !

Déjà mesurer le temps d'exécution de la requête sur la base elle même. Si ta base c'estdu Sql Server, tu as le profiler qui fait cela très bien.

A partir de cette base, il est possible de discuter optim du code; mais si déjà ta requêt est trop longue car index mal tunés, etc .... il faut commencer par là (Dans Sql Server tu as par exemple la possibilité de créer des champs non clefs dans les index, possibilité largement sous exploitée dans le cas de requêtes fréquentes)

[ThistleDub]

Oui effectivement nous passons toutes les requêtes au profiler.
Et bien merci pour tous ces éclaircissements.
Je clôture !