Przydatne przykłady C# 3.0

Wydaje mi się, że fakt istnienia LINQ i metod rozszerzających nie powinien służyć za usprawiedliwienie stylu kodowania “byleby zmieścić wszystko w jednym wyrażeniu”, za który przez lata programistów straszono, że zacytuję pewnego znanego wszystkim Zine’owego blogera, ucinaniem rączek. Niestety bardzo często do tego się to sprowadza (do usprawiedliwania, nie rozczłonkowania).
Przyjrzyjmy się, co właściwie robi ta “_czytelna_” linia:
_usersSrv.GetUsers().Where(u => u.Age == age).Select(u => new { Text = u.Name, Value = u.Id }).ToList().ToJSON();
1) przygotowuje zbiór danych:
_usersSrv.GetUsers()
2) deklaruje przekształcenia do wykonania na zbiorze – selekcję i projekcję:
.Where(u => u.Age == age).Select(u => new { Text = u.Name, Value = u.Id })
3) przekształca zbiór zgodnie z wcześniejszymi deklaracjami i zwraca wynik w postaci listy:
.ToList()
4) konwertuje listę par etykieta-wartość do łańcucha w notacji JSON:
.ToJSON()
Podumowując, w jednej linii dostajemy wyrażenie składające się z 5 wywołań metod i realizujące 4 zadania. Czy to oby na pewno dobry pomysł? Według mnie – taki sobie.
Osobiście rozbiłbym to wyrażenie na kilka mniejszych, mając na uwadze właśnie odpowiedzialności poszczególnych fragmentów:
1) zdefiniuj zbiór danych:
UserCollection users = _usersSrv.GetUsers();
2) zdefiniuj przekształcenia dla zbioru*:
var query =
  from u in users
  where u.Age == age
  select new { Text = u.Name, Value = u.Id };
3) wygeneruj dane w formacie JSON uwzlędniając zadeklarowane przekształcenia na zbiorze danych (konwersję do listy pominę, gdyż ja bym metodę ToJson zaimplementował jako rozszerzenie IEnumerable):
string formattedData = query.ToJson();
W wyniku dostajemy taki fragment kodu:
UserCollection users = _usersSrv.GetUsers();
var query =
  from u in users
  where u.Age == age
  select new { Text = u.Name, Value = u.Id };
string formattedData = query.ToJson();
który jest może nieco dłuższy, ale według mnie czytelniejszy i łatwiejszy w utrzymaniu. Po jednym zerknięciu na kod możemy powiedzieć, że w pierwszym wyrażeniu uzyskujemy jakiś zbiór, w drugim jakieś jego przekształcenia, a w trzecim jakiś wynik na ich podstawie, czyli możemy na bardzo ogólnym poziomie odtworzyć logikę. Gdy wszystko zapiszemy w jednej linii, po jednym zerknięciu możemy co najwyżej zakrzyknąć “D’oh!”.
P.S. Tak przy okazji, o tym, że zbyt intensywnie wykorzystując składnię LINQ można się zapędzić w kozi róg pisał dziś Ayende Rahien w poście o dużo mówiącym tytule “Unreadable LINQ”:
http://ayende.com/Blog/archive/2008/08/02/Unreadable-Linq.aspx
______
* Tak naprawdę nie byłoby większym złem połączenie punktów 1) i 2) w jeden, pod warunkiem, że wyrażenie pobierające zbiór danych jest trywialne (proste wywołanie metody lub właściwości).

Przykład 1 jest wg mnie zapisany nieelegancko. Jeżeli już bawimy się w takie skróty napisałbym
private bool AreAllTextboxesFilledNewWay()
{
return this.Controls.OfType().All(tb => tb.Text != string.Empty);
}

Zgodzę się z Olkiem, że takie upychanie w jednolinijkowce nie sprzyja czytelności. Przypominają mi się wszelkie wariacje na temat “:?”.
Jeśli zaś chodzi o ciekawe nowe cechy C# 3.0 to extension methods i lambdy są absolutnie genialne.
Podam mój ulubiony przykład. Typowym idiomem w aplikacjach desktopowych jest znane
delegate void FunHandler();
void Fun()
{
   if (control.InvokeRequired)
   {
       control.Invoke(Fun);
   }
   else
   {
       control.Text = “blah blah”;
   }
}
Aktualnie metodę Fun można uprościć do postaci:
void Fun()
{
   control.Invoke(() => control.Text = “blah blah”);
}
oczywiście nasz nowy Invoke może też zwracać wartość. Wszystko dzięki dwóm metodom rozszerzającym(?) o postaci:
static class ControlEx
   {
       public static TResult Invoke(this T controlToInvokeOn, Func code) where T : Control
       {
           if (controlToInvokeOn.InvokeRequired)
           {
               return (TResult)controlToInvokeOn.Invoke(code);
           }
           else
           {
               return (TResult)code();
           }
       }
       public delegate void Func();
       public static void Invoke(this Control controlToInvokeOn, Func code)
       {
           if (controlToInvokeOn.InvokeRequired)
           {
               controlToInvokeOn.Invoke(code);
           }
           else
           {
               code();
           }
       }
   }
Btw: Czy silnik zine.net.pl posiada obsługę tagów w komentarzach? Mam na myśli [code] ;).
Filip