[krasnoludkolo]

Hej
Napisałem jakąś pierwszą wersję swojej aplikacji do liczenia czasu pracy. Założenie jest takie, żeby dodawać sobie czasy pracy (WorkActivity) i później na ich podstawie wysyłać sobie informacje na maila. Pliki kontrolerów i .html powstały trochę dodatkowo bo głównie chciałem się skupić na tym, aby warstwa aplikacji i wszystko co niżej działało i wyglądało dobrze (jak mi to wyszło to się okaże ;) )

https://github.com/krasnoludkolo/WorkCounter

Z rzeczy które wiem co trzeba zrobić:
-Ogarnąć kiedy używać obiektów entity (mniej), a kiedy biznesowych (wiecej)
-Oddzielić jakoś typowe CRUDowe działania od innych (chociaż to trochę już jest po przez osobne UCki, jeden do CRUDa, a reszta do innych rzeczy jak np. mail albo generowanie raportów)
-Klasa Result jest w sumie teraz po nic
-lepiej zorganizować pakiety ale nie wiem jak
-lepsze testy (!)

Drogi rozwoju
-dodać aby każdy użytkownik miał swoją listę
-wiecej opcji generowania raportów
-może jakis REST

Byłbym wdzięczny za każde sensowne uwagi ;)

Pozdrawiam ;)

#spring #springboot #java #codereview

[krasnoludkolo]

a, w pliku .properties brakuje danych które trzeba uzupełnić, żeby aplikacja wstała

[fegwegw]

@krasnoludkolo: Po co Ci te interfejsy serwisów, utilsów? :)

I nie używaj field-injection.

[krasnoludkolo]

@fegwegw: o field-injection już wiem ;)
Co do pierwszej części to w sumie nie wiem ale wynika to pewnie dlatego, że nie znam jeszcze springa ;)

[arkadiuszl4g]

@krasnoludkolo: Jak robisz domenę i separujesz model to powinieneś postarać się by owy nie ciekł do innych warstw aplikacji, walidacja modelu powinna odbywać się w nim samym i na miejscu pustych setterów/getterów powinny się pojawić metody ze znaczeniem biznesowym.
Może cqrs?

Ogólnie to nie pisze w javie tylko w innych językach skryptowych, ale projekt wygląda mi na strasznie zamieszany, tak jak by te warstwy były ze sobą wymieszane, każda z

[krasnoludkolo]

@arkadiuszl4g puste settery i gettery są wymagane przez framework a metody biznesowe są w obiektach biznesowych ;)

[arkadiuszl4g]

@krasnoludkolo: widocznie się pogubiłem

[fegwegw]

puste settery i gettery są wymagane przez framework

@krasnoludkolo: Przez który, i w którym momencie?

[krasnoludkolo]

@fegwegw thymeleaf z tego co widziałem

[fegwegw]

@krasnoludkolo: Ok, możesz mieć rację.

[krasnoludkolo]

@fegwegw bez setterow nie binduje poprawnie podczas wysyłania danych do aplikacji