Discussion :

[snopims]

Bonjour,
je travaille sur un projet que l'on vient de passer du framework 2.0 à 3.5.
Et donc je découvre linq, ma question est la suivante, avant pour filtrer une liste nous faisions :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
List<Fruit> liste= ChargerListe();
List<Fruit> listefiltre= new List<Fruit>();
 
foreach (Fruit fruit in liste)
{
    if (monfiltre(fruit))
    {
        listefiltre.Add(fruit);
    }
}

maintenant avec linq, je fais :

Code :

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

1
2
3
4
5
 
List<Fruit> liste= ChargerListe();
List<Fruit> listefiltre= (from fruit in liste
where monfiltre(fruit)
select fruit).toList<Fruit>();

Ma question est la suivante, est ce 'cosmétique' ou est ce plus performant ?
Par avance merci

[Philippe Vialatte]

Salut,

Ma question est la suivante, est ce 'cosmétique' ou est ce plus performant ?
Par avance merci

tout depends de ce pour quoi tu vas l'utiliser...

De toute façon, à moins que tu ne travailles sur gde trés gros ensembles de résultat, la différence, si elle existe, sera probablement infime

Après, si tu veux *vraiment* un gain en perf, il faudrait mettre monfiltre(fruit) avant

La meilleur règle, à mon avis, c'est d'écrire le code le plus clair possible, et de l'optimiser si besoin

[cybermaxs]

La meilleur règle, à mon avis, c'est d'écrire le code le plus clair possible, et de l'optimiser si besoin

Tout à fait. Une des nombreuses idées derrière Linq est d'apporter de le concision dans l'écriture de ce genre de requête.
Main concis ne veut pas forcément dire clarté. J'ai souvent été confronté à des requêtes Linq complexes dont la compréhension et le debug est extrêmement difficile. Il vaut mieux essayer de découper en plus petites requêtes.

[chrisdot]

@cybermaxs: Je pense que c'est bien plus que ça. Ca regroupe déjà tout un tas d'algos de tris, de selection, jointures, etc... que tu te tapais à la main avant...
Là c'est du code mature & réutilisable.

Ensuite pr le côté perf, n'oublions pas que tant qu'on a pas balayé la requête, celle ci reste au stade de IEnumerable... Celle ci sera lancée qu'au moment ou on fera un ToList(), un foreach dessus ou autre...
Ensuite la clarté ça restera au développeur de l'assurer (comme toujours).. En plus tu as 2 manières de l'écrire... Pour ma part je préfère la version plus "method-style" que "sql-style"...

[cybermaxs]

@cybermaxs: Je pense que c'est bien plus que ça. Ca regroupe déjà tout un tas d'algos de tris, de selection, jointures, etc... que tu te tapais à la main avant...
Là c'est du code mature & réutilisable.

Oui, c'est pour ça que j'avais précisé Une des nombreuses idées.

Ensuite pr le côté perf, n'oublions pas que tant qu'on a pas balayé la requête, celle ci reste au stade de IEnumerable... Celle ci sera lancée qu'au moment ou on fera un ToList(), un foreach dessus ou autre...

Disons simplement que si tu n'es pas dans une application très calculatoire ou avec beaucoup de données, ça n'est pas le peine de se soucier des perfs.

Ensuite la clarté ça restera au développeur de l'assurer (comme toujours).. En plus tu as 2 manières de l'écrire... Pour ma part je préfère la version plus "method-style" que "sql-style"...

idem

[Marsupilami_00]

Bonjour,

La méthode Linq est plus rapide que le Foreach. Par ailleurs Linq donne plus de choix sur la façon de traiter une liste.