Code-review spring web app

Question

Witam, jest to mój pierwszy większy projekt, byłbym wdzięczny jeśli ktoś mógłby zerknąć i dać jakieś wskazówki

Jeśli ktoś odpali projekt u siebie to w db jest 2 użytkowników, jeden zwykły 

email: user1@email.com 

hasło: pass1

i drugi to pracownik
email: employee1@email.com

hasło: pass1

Użyte technologie

Spring Boot, Spring Validation, Spring Security, Hibernate, AngularJs, Bootstrap

https://github.com/Kamil159/bookstore

Możliwa niekonsekwencja w nazwach w kilku miejscach

Answer 1

Tak po przejrzeniu na szybko

1. Polecam lepiej walidować dane i np. dawać górny limit długości nazw.

2. Możesz sprawdzić czym jest Lombok, dzięki niemu będziesz mógł ograniczyć boilerplate.

3. Sprawdź czym jest DTO, przydaje się w przekazywaniu obiektów między warstwami.

4. Popraw formatowanie kodu, bo czasami jest tu ono średnie

5. Jeśli nie zwracasz czegoś dodatkowego to lepiej odpuść ResponseEntity, lepiej zwrócić normalne obiekty, dodatkowo jeśli chcesz ich używać to raczej nie powinny być one budowane w serwisach bo to trochę mieszanie warstw.

6. Co do security to możesz użyć też adnotacji @PreAuthorize i @PostAuthorize

7. Czasami w ścieżkach masz coś takiego jak 

 @GetMapping(value = "/mergeCarts")

z zasady w rest używa się rzeczowników i pomyślałbym nad zmianą nazwy.

8. Role użytkownika dał bym osobiście jako ENUM zamiast Stringa

9. Przydało by się dodać jakąś obsługę błędów