Wszystko

Wszystkie

Archiwum

FEAofTruss
FEAofTruss
  • 9
22/100 dni z książką

Wczoraj rozgorzała gorąca dyskusja - dzisiaj temat nawiązujący.

“Nie przekazujemy null.
Zwracanie wartości null z metod jest niedobrą praktyką, ale przekazywanie wartości null do metod jest jeszcze gorsze. O ile nie korzystamy z API, które oczekuje wartości null, i o ile mamy taką możliwość, powinniśmy unikać przekazywania null we własnym kodzie. [...] W większości języków programowania nie istnieje dobra metoda obsługi wartości null przypadkowo przekazywanych przez wywołującą
FEAofTruss
FEAofTruss
  • 19
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
Vetinari
Vetinari
  • 7
@FEAofTruss: Ta książka trochę przesadza w drugą stronę. W jaki sposób zwrócisz błąd wykonania (alternatywa dla nulla)?

Wyjątek? Jeszcze więcej kodu do złapania i zwijanie stosu.

Optional? Pole w obiekcie? Nadal potrzebujesz ifa.

Zwracanie pary obiektów? Nadal potrzebujesz ifa i na dodatek paskudnie wygląda.

Bardzo często jeżeli coś zwraca null to znaczy, że coś się nie udało. Jeżeli coś się nie udało to zwykle nie olewa się takiej sytuacji tylko potrzebujemy
krasnoludkolo
krasnoludkolo
  • 4
@FEAofTruss 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
krasnoludkolo
  • 1
@nilphilus nie chodzi mi o sytuację jak mówisz że jeden moduł pobiera sobie z drugiego coś (tak jak mowisz) tylko o to gdzie z obiektu dobierasz do obiektu gdzie dobierasz się do obiektu na którym potem coś robisz, jeśli te gety nie są jakimiś operacjami biznesowymi, a tylko getterami na pola
FEAofTruss
FEAofTruss
  • 61
17/100 dni z książką

“A zatem powinniśmy tworzyć krótkie wiersze. Stare ograniczenie Holleritha do 80 znaków jest restrykcyjne [...] Ja staram się nie przekraczać długości 120 znaków.”


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

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

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

[Chcesz być wołany?]
FEAofTruss
FEAofTruss
  • 97
16/100 dni z książką

“Jeżeli jedna funkcja wywołuje inną, powinny być one położone blisko siebie, a funkcja wywołująca powinna być umieszczona powyżej wywoływanej, o ile jest to możliwe.”


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

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

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

[Chcesz być wołany?]
FEAofTruss
FEAofTruss
  • 57
14/100 dni z książką

“Niewiele jest praktyk tak nieprofesjonalnych, jak zakomentowanie kodu: Nie rób tego!

InputStreamResponse = response = new InputStreamResponse();
response.setBody(formatter.getResultStream(), formatter.getByteCount());
//InputStream resultStream = formatter.getResultStream();
//StreamReader reader = new StreamReader(resultStream);
”


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

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

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

[Chcesz być wołany?]
FEAofTruss
FEAofTruss
  • 67
13/100 dni z książką

“/*Dodane przez Ricka*/
Systemy kontroli wersji świetnie nadają się do zapamiętywania, kto (i kiedy) dodał określony fragment. Nie ma potrzeby zaśmiecania kodu tymi małymi dopiskami. Można uważać, że tego typu komentarze będę przydatne do sprawdzenia, z kim można porozmawiać na temat danego fragmentu kodu. Rzeczywistość jest inna - zwykle zostają tam przez lata, tracąc na dokładności i użyteczności.”


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

Podobało się? To
FEAofTruss
FEAofTruss
  • 116
12/100 dni z książką

“W wielu przypadkach kod mógłby zupełnie obejść się bez komentarzy [...] Co wolelibyśmy zobaczyć? To:

if((employee.flags & HOURLY_FLAG) && (employee.age > 65))
czy to:

if(employee.isEligibleForFullBenefits())”


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

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

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

[Chcesz być wołany?]
AnonimoweLwiatko
AnonimoweLwiatko
  • 28
@FEAofTruss: A mi się wydaje, że to:

if((employee.flags & HOURLY_FLAG) && (employee.age > 65))

może być ciałem metody employee.isEligibleForFullBenefits()

Można też nie bawić się w takie rzeczy, że klasa employee która przechowuje zapewne informacje o pracownikach jest odpowiedzialna też za sprawdzanie ich, a napisać zgrabny interfejs który będzie się zajmował takimi rzeczami ( ͡º ͜ʖ͡º)
FEAofTruss
FEAofTruss
  • 47
10/100 dni z książką

“Funkcje powinny coś wykonywać lub odpowiadać na jakieś pytanie, ale nie powinny robić tych dwóch operacji jednocześnie. [...] Jako przykład weźmy następującą funkcję:

Public boolean set(String attribute, String value);
Lepszym rozwiązaniem jest oddzielenie polecenia od zapytania, dzięki czemu niejasność nie występuje:

if(attributeExists(“username”) {
setAttribute(“username”, “unclebob”);
}”


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

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

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

[Chcesz
zibizz1
zibizz1
  • 7
@nilphilus: Jesli nawet nie do bazy to i tak przykład jest słaby
Coś jak:
if(zmienna == true ) zmienna = false;

jest masa funkcji które coś robią i coś zwracają i nie uważam że to nie jest czytelne.

BTW
Jesli już jesteśmy w temacie na temat tego co powinna robic funkcja to osoby początkujące czesto mają problem z takim czymś jak

list.Clear(); //modyfikuje[czyści] liste
name.Replace("old","new"); //nie modyfikuje obiektu, tylko zwraca zmodyfikowany
FEAofTruss
FEAofTruss
  • 34
9/100 dni z książką

“Idealną liczbą argumentów dla funkcji jest zero (funkcja bezargumentowa). [...] Należy unikać konstruowania funkcji o trzech argumentach. [...] Więcej niż trzy argumenty wymagają specjalnego uzasadnienia - a nawet wtedy takie funkcje nie powinny być stosowane”


#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
nilphilus
  • 5
@AnonimoweLwiatko: bo im więcej argumentów tym więcej zależności. Chociaż ja bym nie powiedział że 0 to idealna liczba, bo skads przecież dane trzeba mieć. Oczywiście pomaga DI i korzystanie z tego co wstrzyknieto ale jednak. No ale już co do 3 to się zgadzam, lepiej unikać. No i unikac bool w argumentach bo to prawie na 100% oznacza że w środku jest if który zmusza funkcje do dzialania zupełnie inaczej
FEAofTruss
FEAofTruss
  • 47
8/100 dni z książką

“Warto zapamiętać zasadę Warda: Wiemy, że pracujemy na czystym kodzie, jeżeli każda procedura okazuje się taką, jakiej się spodziewaliśmy. Połową sukcesu w osiągnięciu tego stanu jest wybór dobrych nazw dla małych funkcji wykonujących jedną operację. Im mniejsze i lepiej ukierunkowane są funkcje, tym łatwiej wybrać dla nich opisową nazwę.”


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

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

************
FEAofTruss
FEAofTruss
  • 1
Wołam przez MirkoListy plusujących i komentujących ten wpis (55)

Sponsor: Grupa Facebookowa z promocjami z chińskich sklepów

Nie chcesz być wołany/a jako plusujący/a? Włącz blokadę na https://mirkolisty.pvu.pl/call lub odezwij się do @IrvinTalvanen

Uważasz, że wołający nadużywa MirkoList? Daj znać @IrvinTalvanen

! @czolgistka93 @lesny_druh @piaskovitz @laki1 @Pan_Jerzy @Kierman @Draakul @killermannnnn @cotoo @grabtom_ @fantastyczny_pan_lis @Wade94 @nabroleon @kosa1410 @janusz-lece @cccn @budyn @zakopiak @alphabeth @piotrtrojan @mavfly @arti040 @ThePaueu @SciulanyLuftem @sesh_ @TheAmazingRak @Bladyy @Patronka_seriali @
FEAofTruss
FEAofTruss
  • 59
7/100 dni z książką

“Funkcje nie powinny mieć 100 wierszy długości. Funkcje powinny mieć właśnie nie więcej niż 20 wierszy.
[...]
Poziom wcięć w funkcji nie powinien przekraczać dwóch.
[...]
FUNKCJE POWINNY WYKONYWAĆ JEDNĄ OPERACJĘ. POWINNY ROBIĆ TO DOBRZE. POWINNY ROBIĆ TYLKO TO.”


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

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

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

[Chcesz być wołany?]
zakopiak
zakopiak
  • 14
FUNKCJE POWINNY WYKONYWAĆ JEDNĄ OPERACJĘ. POWINNY ROBIĆ TO DOBRZE. POWINNY ROBIĆ TYLKO TO.”


@FEAofTruss: W skrócie SRP.

Jednak jest jedno "ale". W uproszczeniu powiem, że skoro funkcja ma 20 linii, to znaczy, że wykonuje 20 rzeczy (trzeba odliczyć nawiasy itp, ale upraszczam). To powoduje często błędne zrozumienie SRP i przesadzone rozdrobnienie kodu.
Więc jak ma robić jedną? Otóż brakuje tutaj pojęcia poziomu abstrakcji. Funkcja (lub ogólnie również klasa, metoda, pakiet itd)
FEAofTruss
FEAofTruss
  • 22
6/100 dni z książką

“Ludzie często nie zmieniają nazw elementów z obawy, że inni programiści będą mieli zastrzeżenia. [...] Prawdopodobnie zaskoczymy kogoś, gdy zmienimy nazwę, podobnie jak w przypadku innych usprawnień kodu, ale nie powstrzymuje to nas przed dokonywaniem zmian.”


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

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

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

[Chcesz być wołany?]
asciiterror
asciiterror
  • 3
@FEAofTruss: @chopak: zmienianie nazw modułów czy jakichś podstawowych klas oczywiście stworzy bałagan i tego lepiej nie robić.
Ale poprawianie np. zmiennych które są widoczne w jednym miejscu jak najbardziej jest ok. W większych projektach wiele klas ruszanych jest nawet raz na kilka miesięcy, więc po zmianie nazwy zmiennej nikt nie będzie tym później zaskoczony bo i tak nie będzie pamiętał co tam było, ani nie będzie co chwila zmian w
FEAofTruss
FEAofTruss
  • 23
5/100 dni z książką


“W hipotetycznej aplikacji o nazwie Luksusowa stacja benzynowa nie należy prefiksować każdej klasy skrótem LSB. Po prostu będziemy mieli przeciwko sobie używane narzędzia. Wpisujemy L, naciskamy klawisz dokańczania i otrzymujemy długą na kilometr listę wszystkich klas w systemie.”


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

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

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

[Chcesz być wołany?]
FEAofTruss
FEAofTruss
  • 11
3/100 dni z książką

“Osobiście pozostawiam nazwy interfejsów bez dekoracji. Początkowe I, tak częste w istniejącej bazie kodu, jest w najlepszym przypadku zakłóceniem, a w najgorszym nośnikiem zbyt dużej ilości informacji. Nie chcę, aby moi użytkownicy wiedzieli, że przekazuję im interfejs.”


#feaoftruss #czystykod #programowanie #programista15k #webdev #gamedev
SwordPL
SwordPL
  • 1
@kovalski: Otóż nie.

Dla Ciebie jako użytkownika interfejsu jest bardzo wszystko jedno, co dostaniesz pod spodem. To, co Ciebie interesuje to fakt, że rzecz, którą dostajesz wyraża pewne zachowanie.

Na przykład:
Jeżeli masz interfejs Printable z metodą print() to Tobie obojętne jakiego typu obiekty do Ciebie przychodzą. Mogą przyjść PDFy, obrazki, pasta o serwerowni. Tak długo jak każdy z tych obiektów obsługuje Printable to będzie traktowany kompletnie jednakowo.

Powiązane tagi