search

klardeau / myaudiolibrary

look master branch !
0 stars 0 forks source link

Correction #1

Open pjvilloud opened 3 years ago

pjvilloud commented 3 years ago

Java POO : Acquis. Bon travail ! Pense à à redéfinir les méthodes d'Object Java Web Services : Maîtrisé. Très bien ! Quelques petites erreurs Ajoute equals, toString et hashCode dans tes entités (surtout si tu utilises des HashSetd'entités comme pour albumsdans Artist

Pas la peine de déclarer ces méthodes, elles sont déjà fournies par le CrudRepository ! https://github.com/klardeau/myaudiolibrary/blob/d7303ba4f4c932f1cfa4883f67e111085905da70/src/main/java/com/myaudiolibary/web/repository/AlbumRepository.java#L10-L11

Attention, la EntityNotFoundException est mappée en erreur 404, pas 409 ! https://github.com/klardeau/myaudiolibrary/blob/d7303ba4f4c932f1cfa4883f67e111085905da70/src/main/java/com/myaudiolibary/web/service/AlbumService.java#L25

Indentation à soigner https://github.com/klardeau/myaudiolibrary/blob/d7303ba4f4c932f1cfa4883f67e111085905da70/src/main/java/com/myaudiolibary/web/service/ArtistService.java#L39-L44

Je pense pas que ça marche l'update car tu vas te trouver toi même ! Il faudrait qu'un artiste avec le même nom mais un id différent existe... https://github.com/klardeau/myaudiolibrary/blob/d7303ba4f4c932f1cfa4883f67e111085905da70/src/main/java/com/myaudiolibary/web/service/ArtistService.java#L60-L62

Tu n'as pas mappé les IllegalArgumentException en erreur 400 dans le GlobalExceptionHandler ! Cela doit donf renvoyer des erreurs 500