[FEAofTruss]

21/100 dni z książką

“Nie zwracamy null.
Nie zliczę widzianych przeze mnie aplikacji, w których niemal każdy wiersz kodu zawierał test wartości null. [...] Gdy zwracamy wartość null, w rzeczywistości tworzymy sobie dodatkową pracę i powodujemy problemy w funkcjach wywołujących. W takich przypadkach brak jednego testu wartości null powoduje, że aplikacja wymyka się spod kontroli.”

[Więcej infomacji]


#feaoftruss #czystykod #programowanie #programista15k #webdev #gamedev

Podobało się? To zaplusuj i zapisz się do wołania (link w stopce)

************

[Chcesz być wołany?]

[nilphilus]

@bruner88: aua... Szkoda tylko ze apka nie działa. Global handler trzeba dać, ale niedoprowadzac do tego ze bedzie lapalo NullException

[krasnoludkolo]

@Flypho owijasz co się da w optionala i liczysz, że jakoś to będzie :p

[krasnoludkolo]

@Vetinari powtórzę się z wczoraj
"swoją drogą należy według mnie rozróżniać wyjątki od błędów Wyjątek- sytuację wyjątkowe jak brak internetu, brak połączenia z bazą itp i je obsługiwać normalnie Exceptionami Błędy - normalne sytuacje takie brak użytkownika, niedozwolona akcja itp i je już powinno według mnie się obsługiwać normalnie z poziomu kodu a nie jakieś magiczne goto w postaci wyjątku. Pomocne tu są chociażby Either z vavra"

[krasnoludkolo]

Ale jeśli ktoś pisze w nowym kodzie return null świadomie to znaczy że prawdopodobnie jest coś nie tak

[Vetinari]

@krasnoludkolo No tak, ale chodziło mi o to, że zastępowanie nulla na siłę nie ma sensu.

[krasnoludkolo]

@Vetinari jeśli coś działa to pewnie firma nie będzie chciała tracić czasu na zmianę tego, ale jeśli idzie o nowy kod imo nie powinno przechodzić zwracanie nulli

[Vetinari]

@krasnoludkolo Okej, ale nadal nie powiedziałeś jak to zastąpić.

[krasnoludkolo]

@Vetinari która sytuację?

[krasnoludkolo]

@Vetinari pobranie czegoś czego może nie być -> Optional
Działanie które może dać błąd albo wynik ->Either

[Vetinari]

@krasnoludkolo: I dzięki temu nie robimy testu wartości? O.o

[krasnoludkolo]

Jeszcze jedna ważna rzecz pomiędzy sprawdzeniem nulla a Optionalem. Przy Optionalu już po sygnaturze wiesz że może nie być wyniki. Bez tego zgadujesz albo tracisz czas na czytanie kodu czy tu przypadkiem będzie null czy nigdy

[Vetinari]

@krasnoludkolo: Ale musisz za każdym razem dodatkowo pobierać ze środka wartość, więc finalnie piszesz więcej.

[krasnoludkolo]

@Vetinari optionali aż do wypchnięcia danych na zewnątrz nie powinno się wypakowywac chyba że wiesz co chcesz w zamian to masz .orElse

[krasnoludkolo]

@Vetinari tak w temacie jeszcze
https://koziolekweb.pl/2017/08/07/kiedy-wypakowac-optional/

[panTralalinski]

@FEAofTruss: https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/nullable-types/

Polecam

[kawal_leszcza]

@Vetinari: Mam nadzieję, że to nie Ty przeprowadzasz code review :)

[ZasilaczKomputerowy]

@FEAofTruss: Potrafię sobie wyobrazić metodę, która coś pobiera z sesji, lecz nie wiadomo czy wcześniej ktoś coś tam wkładał. I jeżeli tego obiektu tam nie ma to co ma zwrócić? "null" jest taką samą wartością jak każda inna (w sensie, ani gorszą ani lepszą) i w zależności od kontekstu jej zwracanie jest dobre, a czasem nie powinno mieć miejsca.

[Vetinari]

@panTralalinski Faktycznie kod się zrobił od tego prostszy :-D

[krasnoludkolo]

@ZasilaczKomputerowy nie bo sygnatura metody nie sugeruję czy może zwrócić null czy nie, a pokazuje jaki obiekt. Dlatego null nie jest taki jak każdy obiekt tylko gorszy

[ZasilaczKomputerowy]

@krasnoludkolo: Jest dokładnie taki sam. W przypadku który podałem null sygnalizuje, że obiekt nie był ustawiony. To jest dla nas cenna informacja. Idąc twoim tokiem null jest lepszy niż niepełny obiekt z lazy polami, bo nulla wyłapiesz odrazu, a pola lazy ci #!$%@?ą aplikację.