[Dominias]

#programowanie #java
Mirki mógłby ktoś zerknąć na mój kod i powiedzieć jak to wygląda pod względem jakości? https://github.com/Dezet-IT/TransitApp to moja pierwsza apka napisana w Springu (https://notehub.org/9pk10), chcę kilka takich prostych API zrobić żeby było co w CV pokazać

[jascen]

@Dominias skąd sie springa uczysz?

[Dominias]

@jascen: https://javastart.pl/kurs/spring

[Ostrzewtlumie]

(https://notehub.org/9pk10),

@Dominias: co powinno tu sie wyświetlić bo mam Not Found?
Ogólnie też zamierzam, coś w Springu napisać ale na razie JavieEE( ( ͡° ͜ʖ ͡°) ) robię prostą apkę CRUDową.

[Dominias]

@Ostrzewtlumie: https://notehub.org/9pk10

[jascen]

@Ostrzewtlumie: https://notehub.org/9pk10 poprawione

[l.....k]

@Dominias: nie przeglądałem całego, ale na pewno z takich baboli to:
1) dodaj .mvn do gitignore, żeby nie wrzucać jarek do repo
2) w klasie TransitController masz bardzo nieładnie zrobioną obsługę wyjątków: raz, że getMessage() tak naprawdę nic tam nie zrobi, bo zwróci Stringa którego ty zignorujesz, a dwa w konsekwencji jak trafisz nam na NPE to przejdzie bez jakiejkolwiek informacji, a to już jest bardzo zła praktyka.

[Ostrzewtlumie]

@Dominias: @jascen: a to jest to zadanie, co kiedyś jakiś ziomek tu wrzucił?

[Dominias]

@Ostrzewtlumie: tak on wrzucał kilka zadań ale nie pamiętam pod jakim tagiem

[l.....k]

@Dominias: A no i w aplikacjach enterprise już się raczej nie używa System.out.print tylko loggerów, więc ładniej będzie jak sobie takowego skonfigurujesz

[Ostrzewtlumie]

@Dominias: Pytanie czy wrzucanie nazwy użytkownika i hasła w pliku gdzie konfigurujesz połaczanie z bazą danych to dobra praktyka czy lepiej je ukryć np. przez szyfrowanie lub pobieranie z pliku, żeby nasztywno nie mieć tego w kodzie?

[Dominias]

@Ostrzewtlumie: jak lokalnie odpalasz aplikację to raczej nie ma to żadnego znaczenia

[Dominias]

@labadzak: dzięki już poprawiłęm

[dog_meat]

@Dominias: sekrety powinny być pobierane ze zmiennych środowiskowych

[l.....x]

@Dominias:

1) O dużo za dużo się dzieje w metodach kontrolera, najlepiej taką logikę robić w jakimś serwisie
2) DRY? Nie wydaje Ci się, że masz 2 kawałki prawie identycznego kodu?
3) Coś jest mocno nie tak z tymi nullami

[Dominias]

@ldefix kod podobny ale jeden endpoint do raportów w przedziale czasu, a drugi dla obecnego miesiąca od pierwszego do dnia który właśnie mamy

[Smevios]

@Dominias: ale 10 linijek takich samych, tylko źródło "transits" jest inne ;)

[Dominias]

@Smevios: czyli specjalnie dla tej części kodu muszę pisać osobną metodę typu getPriceAndDistance?

[Karol191]

@Dominias: Tak dokladnie, tzn. osobno getPrice, getDistance, metoda zazwyczaj ma robic jedna rzecz. Co do tego ssa to sa Optionale aby sprawdzac ladnie czy cos nie jest nullem. A zamiast petli for mozna to pewnie zrobic w streamie. Do tego parametry w konstruktorze /daily powinny byc zwiniete aby mozna bylo to przeczytac wygodnie a nie przewijac na prawo. Do tego mozesz sie zainteresowac biblioteka Lombok i adnotacja @Sneakythrows na przyszlosc.

[Karol191]

@Dominias: Dodatkowo zamiast laczenia stringow gdzie tak naprawde tworzy sie 5 stringow. Mozesz zastosowac String.format.