Code review projektu SearchEarthquakes w Java 8

0

Cześć,
Prosiłbym o ocenę aplikacji do znajdowania najbliższych trzęsień ziemi w okolicy dla podanego punktu.
Aplikacja napisana w Javie 8.
Zależy mi szczególnie na tym czy kod jest czytelny oraz co poprawić aby był bardziej czytelny.
https://github.com/BoalaKear/ProjectEarthquakes

Pozdrawiam Tomadeo

PS. Najbardziej cenię uwagi zgodne z tematem "Jak krytykować z korzyścią dla wszystkich".

2

Na szybko to tak:

  1. Tworzysz pusty Set i potem dodajesz do niego w streamie.
  2. Niby używasz lomboka, ale tak jakby go nie było. Używasz @UtilityClass a potem i tak piszesz public static final. Tam gdzie możesz go użyć to go nie używasz, np mógłbyś dać @AllArgsConsyructor lub val i var.
  3. Nazwy zmiennych/parametrów z numerami.

Co do lomboka to jeśli go używasz w projekcie to korzystaj z niego, a nie tylko importuj 🙂.

1

W sumie mnie się generalnie podoba :) Małe uwagi:

  1. Brak podziału na moduły. Masz np. gdzieśtam wydzielone interfejsy, ale zaraz obok masz ich (jedyną) implementacje. Jaki był sens ich wydzielania w takim razie? Gdybyś miał osobny moduł na DTO+Interfejsy to miałoby to jakiś sens, bo można teraz wciagnąć taki moduł i zrobić jakąś własną implementacje.
  2. https://github.com/BoalaKear/ProjectEarthquakes/blob/master/src/main/java/application/service/EarthquakesService.java#L56 takie wielopoziomowe lambdy są mało czytelne, lepiej zrobić tam extract method
  3. Można by do tego zrobić jakiś bardziej przystępny interfejs -> chociażby RESTowy backend a nie konsole.
0

Dziękuję serdecznie za uwagi.

1 użytkowników online, w tym zalogowanych: 0, gości: 1