@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 potrzebujeszPokaż całość

@FEAofTruss: Czytam sobie ten czalendż od jakiegoś czasu i mam wrażenie że takie lakoniczne zdania wyrwane z kontekstu są bardziej szkodliwe niż pożyteczne. Przydałoby się jakieś podbudowanie tego zdania komentarzem. W obecnej formie nie rozumiem o co chodzi i po co to komu. (╯︵╰,)

@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

@FEAofTruss: ale mógł byś odrazu napisać własnymi słowami dlaczego uważamy to za dobrą praktykę :) bo początkujący coder napewno będzie miał problemy z interpretacją powyższych metod :)

@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: Chyba pierwsza faktycznie jakąś madrzejsza i mniej oczywista rzecz. Zawsze programuj do abstrakcji. Jak pierwszy raz pracowałem w korpo byłem zaskoczony ile tam interfejsów ( ͡° ͜ʖ ͡°)

@dotnetboy
@Ostrzewtlumie
I potem powstaje milion interfejow z jedną implementacją NO ALE jakby kiedyś może chcieć zmienić implementację to się da( ͡º ͜ʖ͡º)

@FEAofTruss: jest tag, jest lista, przestań wołać do tego plusujacych

@FEAofTruss: Oni mają rację. Jeśli chcą być wołani zapisza się do listy. A jeśli dają plusy czy komentarze to tylko się udzielają.

ponad 70 razy zdefiniowane
var OK = 1;
var NO = 0;
bo przecież po co komu true/false

czyNieWybranoCalegoPrzedzialuDlaPsa xD

@FEAofTruss: a później idziesz do pracy, gdzie narzucone są narzędzia, które same formatują kod i dostajesz metody ułożone według modyfikatorów dostępu

@AnonimoweLwiatko: oddzielone publiczne od prywatnych jeszcze mogę jakoś zrozumieć. Ale alfabetycznie? Po co?

@FEAofTruss: Ciężko z tym dyskutować ( ͡º ͜ʖ͡º) Mogę ewentualnie dodać, że powinny też mieć jak najmniejszy zasięg czyli odpowiedni scope aby w momencie gdy nie były nam już potrzebne, miejsce w pamięci zostało automatycznie zwolnione. ({}).

większość zmiennych zadeklarowałem na początku w grupach ze względu na przeznaczenie.

@Mr_NiceGuy: Brzmi jak dużo zmiennych czyli dużo kodu, czyli kod robi dużo rzeczy. Obstawiam, że w ogóle nie powinieneś mieć takiej potrzeby, gdyby ten kawałek kodu nie robił zbyt dużo rzeczy.

@FEAofTruss: Komentowanie kodu doskonale sprawdza się jako narzędzie szybkiego debugowania w niektórych środowiskach. Czasem też chcesz wprowadzić tymczasowe zmiany lokalnie i zachować sobie stary kod, wtedy też szybki Ctrl+Shift+C są niezastąpione ( ͡º ͜ʖ͡º)

@FEAofTruss: żeby było śmieszniej to ludzie się denerwują ze się czepiam jak wytykam to w review (╥﹏╥)

@FEAofTruss: Znam firme, która zatrudnia setki osób w IT, a Git tam to głównie komentarze. Zakomentowac zmieniony kod, zostawić najbardziej aktualny, dodać komentarz kto kiedy co zmienił. ( ͡° ͜ʖ ͡°) #januszeprogramowania

@dotnetboy: aż mnie serce zabolało jak to przeczytałam

@FEAofTruss: no ale co to za odkrycie..To jedno z postaw programowania żeby nazwy metod czy pól mówiły co to oznacza... choć większość osób tak nie robi..I komentarzy też nie robi ;)

@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: Coraz bardziej mam wrażenie, że te rady odnoszą się do typowych projektów domowych, a nie takich z którymi masz styczność w korpo ( ͡° ͜ʖ ͡°)

@AnonimoweLwiatko zgadzam się, ja wyznaje zasadę, jak działa to nie ruszaj, jeśli nie musisz. Trzeba ograniczać ryzyko.

@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 jestPokaż całość

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

@zibizz1: gdyby tu tylko chodziło o to czy coś jest czytelne czy nie :) Ponadto CQRS, to może być ciekawa lektura.

@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: Dodałbym jeszcze, funkcje powinny być testowalne. Dobrze od razu pomyśleć jak będzie wyglądał unit test. Właśnie pisząc testy czesto też dzieli się funkcje na mniejsze.

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, pakietPokaż całość

@FEAofTruss: A teraz wyobraź sobie, że każdy zacznie zmieniać nazwy na jego zdaniem "bardziej pasujące". Kod będzie zmieniany w tą i w tamtą aż do posrania i wszystkie MD polecą na refaktoring :)

@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 chwilaPokaż całość

@FEAofTruss: ta ksiażka jest bardzo średnia, znudziły mnie te truizmy i przestałem czytać.

@nilphilus: każde normalne IDE jak najedziesz myszką na wywołanie metody - pokazuje jej sygnaturę, tam masz typy i nazwy argumentów.

Zaraz zaczniecie namawiać do pisania w notacji węgierskiej :)

@FEAofTruss: nie rozumiem, przecież użytkownik powinien wiedzieć że ma do czynienia z interfejsem a nie np klasą abstrakcyjną. Możesz mi to wyjaśnić?

@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.