to linq or not to linq

Version imprimable

Salut,
je me pose une question pas forcément fondamentale mais bon...
Parmi les deux bouts de codes ci-dessous qui font la même chose, lequel est le plus performant selon vous.
J'aime bien la syntaxe linq mais j'ai l'impression que le foreach est plus rapide car l'itération s'arrête dès que l'élement est trouvé...

Code:

1
2
3
            string subjId = (from x in subjIdCollection
                             where x.ToLower().Contains(pattern)
                             select x).FirstOrDefault();

Code:

1
2
3
4
5
6
7
8
9
10
 
            string subjId = string.Empty;
            foreach (string s in subjIdCollection)
            {
                if (s.ToLower().Contains(pattern))
                {
                    subjId = s;
                    break;
                }
            }

Merci de vos conseils

11/11/2010, 13h05
PetitCastor13

Bonjour,

Réécris ton code LINQ de cette façon :

Code:

subjIdCollection.FirstOrDefault(x => x.ToLower().Contains(pattern));

Ainsi, ton itération s'arrête dès que tu as trouvé l'élément cherché.
11/11/2010, 13h08
StormimOn

Ce doit se valoir niveau performance (utilise un Stopwatch si tu veux voir le temps que ça prend), car la requête Linq va arrêter de parcourir les données au premier élément qui correspond.

Il ne faut pas tomber dans le "piège" de la micro-optimisation, qui est une grosse perte de temps en général. On ne programme pas en assembleur après tout ^^
11/11/2010, 13h36
girouxv

J'ai déjà fait des tests: foreach a toujours gagné.

En gros linq c'est une couche qui te permet de sauver des lignes de codes, mais tu n'es jamais certain à 100% de ce que fera linq en arrière.

Si les performances sont importantes dans ton développement fait des boucles foreach.

Beaucoup de sites sur le net le démontre, si tu as plusieurs millions d'objets dans ta requête les millisecondes peuvent se transformer en secondes.
11/11/2010, 14h04
julien.63
Merci pour vos réponses.

Donc les deux bouts de code ci-dessous ne génère pas le même IL?

Code:

subjIdCollection.FirstOrDefault(x => x.ToLower().Contains(pattern));
Code:

1 2 3 string subjId = (from x in subjIdCollection where x.ToLower().Contains(pattern) select x).FirstOrDefault();
Citation:

Envoyé par StormimOn

Il ne faut pas tomber dans le "piège" de la micro-optimisation

ouais, je suis tout à fait d'accord. C'est plus par curiosité et pour mieux comprendre comment marche linq. J'ai bien conscience que ça va pas changer mon résultat final;)
11/11/2010, 14h04
girouxv

Citation:

Il ne faut pas tomber dans le "piège" de la micro-optimisation, qui est une grosse perte de temps en général. On ne programme pas en assembleur après tout ^^

Ne le prends pas mal mais je ne t'engagerais pas dans mon entreprise.

La micro-optimisation est ce qui compte le plus, je programme des logiciels pour des dessinateurs, je suis le meilleur programmeur pas parce que je programme rapidement, pas parce que je fais le plus beau code. Mais parce que l'intégrité des données et la rapidité d'exécution de mes applications surpasse toujours celles des autres programmeurs.

Il m'arrive souvent de refaire 3 fois le même bout de code de 3 manières différentes juste pour comparer les performances. Le calcule est simple 60 dessinateurs qui sauvent 1 minutes dans la journée économise 1 heure en productivité dans l'entreprise par jour. C'est facile pour mon augmentation de salaire à la fin de l'année.

J'ai déjà repris une génération d'un rapport dans du SQL qui prenait 15 minutes et je l'ai ramené à moins d'une minute et j'ai des exemples à la tonne de code bien beau comme ça. Des dataset typés en c# etc.

Tout ce qui ressemble à la génération de code par un logiciel par exemple Linq ne pourra jamais être aussi performant qu'un bout de code bien pensé et bien testé.

Donc, oui la micro-optimisation compte dans une application, sauf bien sûr si c'est pour une application chez toi que tu fais pour le plaisir.
11/11/2010, 14h09
champomy62

salut,
Un sujet comme le tient avait été traité sur ce forum et je me souviens que le foreach été toujours gagnant !
11/11/2010, 14h40
StormimOn

Citation:

Envoyé par girouxv

Donc, oui la micro-optimisation compte dans une application, sauf bien sûr si c'est pour une application chez toi que tu fais pour le plaisir.

Je parlais en général bien évidemment. Gagner 1 seconde sur un traitement qui dure 1 minute, et qui ne sert que 2-3 fois par jour, n'a absolument aucun intérêt par exemple.
Et je disais bien de ne pas tomber dans ce "piège", c'est à dire faire de la micro-optimisation qui n'apporte absolument rien. Cela ne veut pas dire qu'il ne faut pas en faire, il faut juste ne pas le faire pour rien.
11/11/2010, 15h06
champomy62

pourquoi c# et pas c++ pour un logiciel de dessinateur ? tu as de bonnes performances ?
11/11/2010, 15h40
julien.63

Citation:

je suis le meilleur programmeur

Félicitations !

Citation:

Envoyé par champomy62

salut,
Un sujet comme le tient avait été traité sur ce forum et je me souviens que le foreach été toujours gagnant !

Oui, j'avoue ne pas avoir vraiment cherché avant... :?
11/11/2010, 15h42
champomy62

ce n'était pas une critique juste que si tu cherches des informations en plus tu sais ou cherché ;)

un lien :

http://blogs.msdn.com/b/mitsufu/arch...-e-ou-non.aspx
11/11/2010, 21h14
girouxv

Citation:

pourquoi c# et pas c++ pour un logiciel de dessinateur ? tu as de bonnes performances ?

C'est de la gestion de données pour les dessinateurs dans des bd et fichiers. Non pas dans des images.

Citation:

Félicitations !

Je ne suis pas vraiment ce qu'on pourrait appeler la personne la plus modeste sur terre :oops:
11/11/2010, 21h17
champomy62

Ok