Nazywanie zmiennych w lambdach #2 (x versuss t,p,pi…)

Posted on

Podoba mi się konwencja aby nazyw zmiennych w lambdach nazywać ‘x’, jeśli potrzeba kolejnej ‘y’, a jeśli kolejnej (nie zdażyło mi się, pewnie mało widziałem, młody jestem ;)) to ‘z’. Poniżej na przykładach. Gdy jednak uważasz że trzeba je nazwać bardziej opisowo to odsyłam do Nazywanie zmiennych w lambdach.

Zła nazwa

advisers.Single(t => t.AdviserId == adviserId);

Zła nazwa

advisers.Single(a => a.AdviserId == adviserId);

Dobra nazwa

advisers.Single(x => x.AdviserId == adviserId);

Bardzo zła nazwa (i na pewno trzeba z czymś innym kojarzyć)

interns.Single(i => i.Id == choosenInternId);

Gdy trzeba więcej ‘literek’ użyć

products.Where(x => x.Categories.Any(y => y.Id == categoryId));

Kopiowanie kodu i wędrujace złe nazwy

Ludzie kopiuja kod. Nie jest to złe samo w sobie. Np trzeba wykorzystać zgrabne kilkulinijkowe Linq w innym miejscu, a uwspólnianie kodu nie ma sensu. Przy takim kopiowaniu nie sprawdza się najczęściej czy nazwy lambd są sensowne i contractor nie stał się nagle literką h. Gdy trzymamy się konwencji x,y,z to nie zacznie nam się to rozjeżdżać.

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out /  Change )

Google photo

You are commenting using your Google account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )

Connecting to %s