[p.....k]

Koliedzy, pół dnia siedzę i refactoryzuję gównokody. Doszedłem do czegoś takiego.
Co można więcej zrobić by zamieniło się to w czystokod nowoczesny?

Nie chodzi o to byście rozkminiali, tylko byście spojrzeli, bo może coś się Wam rzuca w oczy.

#programowanie #czystykod #czystokodnowoczesny

[e.....e]

@programista4k: Dodałem sobie do ulubionych. Nie usuwaj, bo chcę poczytać co progamiści15k na to powiedzą.

[p.....k]

@emcedwierence: a Ty co powiesz?

[Campell]

@programista4k: nie jestem może wielce zaawansowany, ale w pokazanym fragmencie przede wszystkim zmienna c ma nazwę beznadziejną, coś lepszego by się przydało, albo os razu wrzucić to w optionala

[p.....k]

@Campell: no tylko, ze wtedy mam tak: Optional(..................many long commends............................................)

[e.....e]

@programista4k: Ja nie jestem programista, dopiero coś dłubać zaczynam. :D

[Saly]

@programista4k:
- używania mutowalnych zmiennych do zwracania wartości (przykładowo count). Zamiast takiego zwracaj wartość w miejscu
- w getResponses pętlę for można zamienić na map()
- nie wiem do końca, czy to aby na pewno nie Scala, bo dawno w tym nie pisałem, ale chyba powinieneś używać kolekcji z Scali a nie z Javy?

Inne rzeczy ciężko wyłapać, bo za mało kodu

[p.....k]

@Saly: dzięki

ad 1
no wlasnie kiedys w książce "clean code" przeczytałem by robić tak:

function foo() {
result = 0 // na poczatku to co bedziemy zwracac
result = tryToGetResult()
if (...) result = ...
return result
}

to jest złe?
Ponadto czytałem w poradniku scali, by nie używać nigdy return.

[Saly]

@programista4k: to podejście może mieć sens, jak klepiesz imperatywne, choć ja nie jestem fanem nawet w tym paradygmacie. W językach funkcyjnych tak się nie robi, bo łamiesz fundament, którym jest niemutowalność. Jak masz wszystko immutable to dużo łatwiej rozumować o kodzie: zamiast rozkminiać, w którym ifie przypisujesz wartość result to wystarczy popatrzeć co od czego zależy. W twoim przykładzie metoda extract mogłaby zwracać jako wynik wyrażenie if. Oczywiście to co piszę

[tronez]

@programista4k: nie ma tam czasem jakiegos uproszczenia typu? Czy czegos podobnego do tego na dole?
return Optional
.doSomething
.orElse(Optional.empty)

[CuckCuckKlan]

@programista4k: O gurwa, jak tak się pisze teraz w Javie to mam #!$%@? jak w ruskim czołgu ( ಠ_ಠ).

[p.....k]

@CuckCuckKlan: czemu?

[CuckCuckKlan]

@programista4k: Streamy, optionale, variable - tego nie ma w Javi 6 i to jest kwestia przywyknięcia do używania tego, a z tym ciężko jest (╯︵╰,).

[p.....k]

@CuckCuckKlan: przecież to łatwe, kwestia oswojenia i praktyki

[CuckCuckKlan]

@programista4k: Streeamów nie trawię...niby się ich uczyłem, ale są dla mnie kontr-intuicyjne.

[p.....k]

@CuckCuckKlan: to musisz zacząć trawić, nie ciałbym pracować z kimś kto unika streamów i #!$%@? kodem z 20 wieku

[CuckCuckKlan]

@programista4k: Pewno bym w nich pisał, ale ze 80 razy wolniej, bo z dokumentacją (╯︵╰,)

[p.....k]

@CuckCuckKlan: no i dobrze, a po 2 tyg. byś śmigał. Zazwyczaj to się używa tylko map, filter i collect.