Junior Java Developer - pierwsza praca (recenzja kodu)

Witam wszystkich. Zakładam drugi wątek odnośnie recenzji kodu. Pierwszy o mnie jest tutaj - https://4programmers.net/Forum/Kariera/307061-junior_java_developer_first_job?p=1466834#id1466834

W skrócie - chciałbym za 2.5 miesiąca znaleźć (najlepiej) pracę (ewentualnie staż) w Warszawie jako Junior Java Developer i chciałbym prosić forumowiczów oraz bardziej doświadczonych od siebie o zerknięcie do projektu, który robię od niedawna jako dowód jakichś umiejętności.

Mój projekt znajduje się tutaj -> https://github.com/oputyk/Librick i jest zrobiony w Springu (Boot, Security, JPA (Hibernate), MVC, REST) oraz Angularze (Typescript).

Dotychczas dostałem info (w poprzednim wątku), że zamiast java.util.Date powinienem używać java.time oraz DTO powinno być niemutowalne.

Ma ktoś jakieś rady odnośnie użytych narzędzi bądź stylu czy też błędów, które popełniam w projekcie?
W sumie też jak by ktoś zobaczył coś, co robię dobrze, a nie jest takie oczywiste, to też chętnie przyjmę taką informację do siebie i spróbuję iść w tą stronę.

Z góry dziękuję za każdą radę.

• https://github.com/oputyk/Librick/blob/master/spring/src/main/java/com/oputyk/librick/Application.java - Formatowanie kodu
• https://github.com/oputyk/Librick/blob/master/spring/src/main/java/com/oputyk/librick/author/converter/AuthorConverterImpl.java#L34 - converter chyba nie powinien zapisywać niczego w bazie? imo robi za dużo
• https://github.com/oputyk/Librick/blob/master/spring/src/main/java/com/oputyk/librick/book/converter/BookConverterImpl.java#L25 - nie rób @Autowired na polu bo potem to jest ciężej testować. Dodaj go nad konstruktorem i przyjmij w nim serwisy, których chcesz używać. Wtedy w testach możesz sobie łatwiej wpiąć implementację i nie potrzebujesz uruchamiać całego contextu springa żeby Ci je tam wstrzyknął
• https://github.com/oputyk/Librick/blob/master/spring/src/main/java/com/oputyk/librick/book/domain/BookEntity.java#L25 - nie wiem czy to standard ale w projektach często widziałem, że encja w takim przypadku po prostu nazywałaby się "Book"
• https://github.com/oputyk/Librick/blob/master/spring/src/main/java/com/oputyk/librick/book/BookSecureController.java#L13 - GET/POST/DELETE? określ
• Używaj finali w parametrach metod żeby sobie czasem czegoś nie nadpisać. Jak chcesz to finale dla lokalnych zmiennych też by były ok (takie dodatkowe zabezpieczenie się ale rzadko to gdzieś widywałem :) )
• https://github.com/oputyk/Librick/blob/master/spring/src/main/java/com/oputyk/librick/user/dto/ChangePasswordUserDto.java#L8 - czy builder na pewno jest tu potrzebny? Jeśli oba pola są wymagane to lepiej żebyś ich po prostu wymagał konstruktorem niż pozwalał stworzyć jedno z pól puste z użyciem buildera

Co do konwerterów to możesz sobie rzucić okiem na mapstructa, ograniczy trochę pracy + działa ze springiem