Code Review Spring Rest- amator

Witam,

Niedawno zacząłem się bawić w Springu i zrobiłem aplikacje Rest. Nie chciałem też się porywać nie wiadomo na co na początku.
Zanim zacznę jeszcze sprawdzać całą walidację i testy, to prosiłbym o Code Review / ocenę samego projektu

https://github.com/rmaduzia/SpringBootRestApplication-MatchingUserProfile

Za wszelkie merytoryczne uwagi z góry dzięki.

PS. jakby co to właśnie w tym momencie przypomniałem sobie, żeby zautomatyzować Mappera
Nie jestem jeszcze przekonany jak powinien np wyglądać Update User - czy wymagane hasło czy bez ?

Ale co tu oceniać? Przecież tam nic nie ma. To jest Generic CRUD sklonowany z jakiegoś tutoriala starszego od ciebie.

• jest bezsensowne JPA (no dobra, to CRUD więc niech ci już będzie z tym JPA...)
• jest mapowanie 1:1 @Entity na DTO
• są potężne testy https://github.com/rmaduzia/SpringBootRestApplication-MatchingUserProfile/blob/master/src/test/java/pl/matchingprofile/DemoApplicationTests.java :D
• jest świetny error handling https://github.com/rmaduzia/SpringBootRestApplication-MatchingUserProfile/blob/master/src/main/java/pl/matchingprofile/components/user/StringToEnumConverter.java#L13
• jest cała logika aplikacji w kontrolerze https://github.com/rmaduzia/SpringBootRestApplication-MatchingUserProfile/blob/master/src/main/java/pl/matchingprofile/components/user/UserResource.java#L24

Przyznam że nie wiem czy to repo to jest trolling/żart czy nie...

Pan u góry chyba dawno nie był początkującym ;)
Pamiętam jak ja zaczynałem - konfiguracja samego springa (jeszcze tylko XML był dostępny) żeby w ogóle pozbyć się wyjątków to były +/- 3-4 tygodnie. Nawet nie wiedziałem czym rózni się POST/GET.

Fakt, kod jest słaby, nie ma co oceniać ALE sam plus że COŚ już jest - dodawaj logike, refaktoruj, pisz proste testy i za 3 miesiące będziesz się śmiał, że takie proste rzeczy sprawiały Ci problemy.

Jan Ko napisał(a):

Fakt, kod jest słaby, nie ma co oceniać ALE sam plus że COŚ już jest - dodawaj logike, refaktoruj, pisz proste testy i za 3 miesiące będziesz się śmiał, że takie proste rzeczy sprawiały Ci problemy.

Sam post nie miał na celu pokazać, jak zajebisty jest to projekt, czy coś. Chciałem tylko poznać wstępnie opinie. Stwierdziłem, że lepiej będzie napisać prosty projekt niż od razu zabrać się za coś skomplikowanego(Rozumiem uczenie się na błędach etc, ale czemu tracisz czas na różne obejścia, zmieniać praktycznie pół projektu, bo nagle okazuje się inne rozwiązanie jest lepsze(a powinno być to oczywiste), temu wstępnie ten projekt służy.

Dopiszę testy, walidację, wprowadzę dodatkowe funkcje matchowania, ale nie wiążę z tym jakiejś przyszłości, to miał być wstęp na zapoznanie się z Springiem.

Po prostu chciałem się dowiedzieć, ile tak naprawdę błędów popełniłem, zanim przystąpię do czegoś większego.

@hugarili: Ugruntuj Javę, naucz się jej porządnie, bez kwiatków jak tutaj (porównania i sprawdzanie stringów). Naucz się myśleć obiektowo, a nie proceduralnie.
Uczenie się na tym etapie Springa zrobi ci krzywdę.