[pulla]

#programowanie #java

http://pastebin.com/2sN0ahfC
To jest moja metoda z serwisu, która pokazuję listę od najbardziej do najmniej skorelowanych spółek dla wybranego tickera, okresu i metody korelacji. Teraz się zastanawiam:
a) czy zmienne z linii 12-15 wydzielić i inicjalizować w osobnej metodzie
b) linie 20-25 czy wydzielić to do osobnej metody? Lub napisać wewnętrzną klasę, która implementuje interfejs runnable i tworzyć w każdej iteracji nowy obiekt? Coś takiego: http://pastebin.com/iiSurGWc i wtedy mam executor.submit(new CorrelationUtil itd)
Jak uważacie?

[Pipcieo]

@pulla: ta metoda ma realnie mniej niż 30 linii, więc odpowiedź nie jest prosta

a) tylko jeżeli masz pomysł co jeszcze mogłoby korzystać z tej inicjalizacji
b) nie, przecież w praktyce to jest 1 linia kodu: przerost formy na treścią

[pulla]

@Pipcieo:
a) no właśnie nic z tego nie korzysta, przynajmniej na ten moment
Ok czyli jak rozumiem metoda jest czytelna i raczej chciałem przekombinować? Dzięki za odpowiedź

[Pipcieo]

@pulla: tak, jak się pojawią nowe przypadki użycia to pomyślisz o refaktoryzacji (jakże prostej i szybkiej w tym przypadku).

[pulla]

@Pipcieo: Super dzięki :) Pozdrawiam