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?
_512_513_resized.png)
3 komentarze:
Pawel, mozesz przytoczyc przypadek, gdzie obiekt "opinion" bedzie nullem? Wg. mnie w tym kontekcie, ktory podales czescia newralgiczna jest:"opinion.OpininsList", gdzie to moze byc faktycznie nullem...
pozdro!
No właśnie nigdy nie będzie nullem. I rzeczywiście kluczowe jest sprawdzenie listy, ale nie dla autora.
Paweł
Uwaga, dowcip z brodą.
- Czemu napisałeś drugi raz tego samego if-a?
- Na wypadek, gdyby pierwszy nie zadziałał.
Prześlij komentarz