Discussion :

[AsPrO]

Bonjour,
Je m'interroge sur les différences qu'il y a à écrire le même code de 3 manières différentes. Esthétique ? Optimisation ? Bonne pratique ?

Donnez vos avis, sur la meilleur ...

Merci

Code 1

Code :

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

1
2
3
4
5
6
7
8
9
                foreach (Reservation reservation in order.Reservations)
                {
                    if (isInFilter(reservation))
                    {
                        row = table.NewRow();
                        FillRow(Config, row, reservation);
                        table.Rows.Add(row);
                    }
                }

Code 2

Code :

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

1
2
3
4
5
6
7
8
9
10
 
                order.Reservations.ForEach(delegate(Reservation reservation)
                {
                    if (isInFilter(reservation))
                    {
                        row = table.NewRow();
                        FillRow(Config, row, reservation);
                        table.Rows.Add(row);
                    }
                });

Code 3

Code :

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

1
2
3
4
5
6
7
8
9
10
 
                order.Reservations.ForEach(reservation =>
                {
                    if (isInFilter(reservation))
                    {
                        row = table.NewRow();
                        FillRow(Config, row, reservation);
                        table.Rows.Add(row);
                    }
                });

[StormimOn]

La première méthode est bien suffisante. L'utilisation d'une méthode anonyme (code 2) n'apporte strictement rien dans le cas présent. Et donc l'utilisation d'une expression lambda (code 3) non plus, car une expression lambda n'est qu'une manière simplifiée d'écrire une méthode anonyme.

Je m'interroge sur les différences qu'il y a à écrire le même code de 3 manières différentes. Esthétique ? Optimisation ? Bonne pratique ?

Dans l'exemple donné, les codes 2 et 3 relèvent plus du "t'as vu ! je sais utiliser des méthodes anonymes/expressions lambda !!" qu'autre chose. Avis perso.

[Bluedeep]

Dans l'exemple donné, les codes 2 et 3 relèvent plus du "t'as vu ! je sais utiliser des méthodes anonymes/expressions lambda !!" qu'autre chose. Avis perso.

+1

Rien d'autre à ajouter.

[AsPrO]

Merci pour vos réponses

@StormimOn : Tu dis que dans le cas présent la méthode anonyme (code 2) n'apporte rien. Dans quel cas une méthode anonyme pourrait avoir un avantage par rapport au code 1 ?

[Guulh]

Ca aurait peut être un avantage si le code essaie à l'avenir de tenir compte de la parallélisation, en utilisant les Parallel Extension.
La syntaxe est du style Parallel.For ou Parallel.ForEach, et ça nécessite donc une Action<T> en paramètre, qui peut du coup être un méthode anonyme.

[StormimOn]

Ca aurait pu être un avantage également avec les prédicats (délégation Predicate<T>) en profitant de la capture des variables des méthodes anonymes.