Poniżej fragment kodu znaleziony w jednym z projektów. Anty-przykład. tak nie należy pisać:
Opinion opinion = new Opinion();
opinion.GetOpinion(Info.pCode);
if (opinion != null && opinion.OpininsList.Count > 0)
{
return true;
}
Sprawdzanie czy obiekt nie jest równy null – już po wykonaniu na nim operacji. Przydatne! Pisownia oryginalna, ale trochę pozmieniałem nazwy obiektów, aby za dużo nie wyjawiać :).
A Wy, jakie złe kawałki kodu widzieliście ostatnio?
Founder of Octal Solutions a .NET software house.
Passionate dev, blogger, occasionally speaker, one of the leaders of Wroc.NET user group. Microsoft MVP. Podcaster – Ostrapila.pl