[bartoneczek]

#programowanie

Może jestem jakiś zboczony, ale mam bzika na punkcie czystego i ładnego kodu. Często napotykam na sytuację w której jakaś metoda musiałaby być bardzo długa. Dlatego co oczywiste, wydzielam metody. Wtedy moja jedna duża metoda wygląda tak:

int wyliczPensje() {
 wyliczZus();
 wyliczUbezpieczenie();
 wyliczPremie();
 wyliczCosTam();
 odejmijCosTam();
 return pensja;
}

Strasznie nie podobają mi się metody w których wywoływane są inne metody, jedna po drugiej. :< Jednocześnie nie widzę tutaj błędu w sztuce bo wszystko jest zgodne z zasadą jednej odpowiedzialności. A Wy co o tym sądzicie?

[sokytsinolop]

@bartoneczek: sądzę, że nie programuje się po polsku :D

[s.....a]

@bartoneczek: mi się nie podobają metody void

[w.....j]

@bartoneczek: https://www.youtube.com/watch?v=0ZNIQOO2sfA

[bartoneczek]

@sokytsinolop: to tylko przykład..

[m.....t]

@bartoneczek: To oznaka, że twoja klasa robi za dużo.

[M4ks]

@bartoneczek: masz bzika i piszesz po polsku xD return pensja no wcale nie razi w oczy

[bartoneczek]

@M4ks: ja #!$%@?ę, nie piszę kodu po polsku, wrzuciłem to w ramach przykładu, skupcie się #!$%@? na przedstawionym problemie a nie #!$%@? się do szczegółów.

[k.....e]

@bartoneczek: no czego jesteś taki #!$%@? agresywany już bardziej się skupić nie mogę


Jest ok tylko ze to wtedy powinna być może fasada? Albo jakiś inny wzorzec, zależy od reszty aplikacji w sumie.

[Yahoo_]

@bartoneczek: ale w zasadzie to o co Ci chodzi? masz klasę, masz wywołania kolejnych rzeczy, które mają się wydarzyć. Jak inaczej chciałbyś to zrobić? Oczywiście wszystko zależy od konkretnego przypadku, ale jeśli te metody są ściśle biznesowe (tzn. nie chcemy, żeby to było mocno konfigurowalne, bo dodanie kolejnego to ważna decyzja biznesowa i świadomie nie chcemy tego zostawiać konfiguracji) to nie bardzo wiem jakbyś chciał to zmienić. Oczywiście zakładając podejście OO.

[b0lec]

@bartoneczek: jest ok ale najlepiej jak metody coś zwracają- wtedy masz czyściej i możesz wywalić z klasy pola które służą tylko do współdzielonej logiki a nie będą nigdy zwrócone na zewnątrz. I widzisz w metodzie cały flow aplikacji.

Edit
To oczywiście dotyczy sytuacji w której kolejne metody bazują na wynikach poprzednich

[bartoneczek]

@b0lec: API jest tutaj metodą wyliczPensję dla pracownika. I tego oczekuję od klasy. Wyliczyć pensję dla pracownika. Reszta metod to metody prywatne i nie chcę ich udostępniać zewnętrznie właśnie dlatego że chcę żeby pracowały na polach współdzielonych dla klasy. Gdyby nie współdzielone pola, jaki byłby sens klas? Równie dobrze można by było wtedy używać metod statycznych. ;)

Ja wiem, ja rozumiem o co Ci chodzi. Można zrobić te metody prywatne, zwracające

[biwalencik]

@bartoneczek
Zależy od języka. W C++ sam dbasz o zbieranie śmieci po kodzie i zwalnianie zasobów. Ale taki python robi to automatycznie.
Wywoływanie metody w metodzie jest uważane za dobrą praktykę, bo wtedy zmienne z lokalnego zakresu funkcji są wywołane w niebyt, są zwalniane przez automat w trakcie pracy programu.

[d.....m]

@bartoneczek: mój kolega powiedział ongiś - taki kod czyta się jak dobrą książkę ze spisem treści. Jeśli wywołań takich "zbiorczych" obliczeń jest dużo to jedyny minus to narzut na wywoływanie tych funkcji.