fabricerenecorail / mdd050

eval-mdd-050

Eval MDD 050

Télécharger l'archive https://github.com/pjvilloud/ipi-mdd-050-eval-web/releases/download/V1.2/ipi-mdd-050-eval-web-v1.2.zip et la décompressert quelque part.

Le fichier index.html contient toutes les informations pour effectuer le TP.

mdd050's Issues

Correction

En cours d'acquisition. Du travail fourni mais de trop nombreuses erreurs et une gestion d'erreur quasi inexistante. A approfondir !

Ton package controller n'est pas au bon endroit puisque tu l'as mis à la racine au lieu de le mettre dans com.ipiecoles.java.eval.mdd050. Tes contrôleurs ne sont donc pas scannés par Spring, ce qui les empêche de marcher (comment as-tu pu tester l'application ?).

id ne peut pas être null sinon on ne rentre pas dans ce mapping. Si jamais il pouvait l'être ce serait une 400 qui devrait être renvoyée. La 404 doit être renvoyée si l'id est renseigné mais qu'il n'existe pas en base (ce que tu ne testes pas)

mdd050/src/main/java/controller/artistController.java

Lines 31 to 32 in 637f6c6

	if(id==null) {
	ResponseEntity.notFound().build();

Idem pour ça

mdd050/src/main/java/controller/artistController.java

Lines 40 to 41 in 637f6c6

	if(name==null) {
	ResponseEntity.notFound().build();

Idem ici (+non gestion de la 404)

mdd050/src/main/java/controller/artistController.java

Lines 65 to 66 in 637f6c6

	if (id == null) {
	return ResponseEntity.badRequest().body("Cannot remove character with null ID");

Les conventions REST ne sont pas respectées => /artists?name=

mdd050/src/main/java/controller/artistController.java

Line 38 in 637f6c6

@GetMapping("/name")

L'erreur 409 (conflit) n'est pas gérée

mdd050/src/main/java/controller/artistController.java

Line 50 in 637f6c6

artist = artisteService.save(artist);

La 404 n'est pas gérée

mdd050/src/main/java/controller/artistController.java

Line 57 in 637f6c6

return artisteService.save(id);

Tu castes AlbumService en AlbumService ?? 409 non gérée

mdd050/src/main/java/controller/albumController.java

Line 26 in 637f6c6

album = ((AlbumService) albumService).save(album);

Le save de l'album appelé dans le service appelle cette méthode qui ne fait rien

mdd050/src/main/java/com/ipiecoles/java/eval/mdd050/service/AlbumService.java

Lines 30 to 33 in 637f6c6

	public Album save(Album album) {
	// TODO Auto-generated method stub
	return null;
	}

404 non gérée dans la suppression d'un album

mdd050/src/main/java/com/ipiecoles/java/eval/mdd050/service/AlbumService.java

Line 26 in 637f6c6

albumRepository.deleteById(id);

400 non gérée lorsque les paramètres d'entrée sont incorrects

mdd050/src/main/java/com/ipiecoles/java/eval/mdd050/service/ArtistService.java

Line 36 in 637f6c6

Sort sort = Sort.by(Sort.Direction.fromString(sortDirection),sortProperty);

Ca c'est bien, le seul problème, c'est que tu n'as pas de GlobalExceptionHandler pour gérer l'EntityNotFoundException. Le code HTTP généré sera donc 500 et non 404.

mdd050/src/main/java/com/ipiecoles/java/eval/mdd050/service/ArtistService.java

Lines 43 to 49 in 637f6c6

	public Artist findById(Long id) {
	Optional<Artist> artist = this.artistRepository.findById(id);
	if(!artist.isPresent()){
	throw new EntityNotFoundException("Impossible de trouver l'artiste d'identifiant " + id);
	}
	return artist.get();
	}

Ta suppression d'artiste ne supprime pas les albums liés, elle ne fonctionnera que si l'artiste n'a pas d'album sinon elle plantera en erreur 500

mdd050/src/main/java/com/ipiecoles/java/eval/mdd050/service/ArtistService.java

Line 60 in 637f6c6

artistRepository.deleteById(id);

??

mdd050/src/main/java/com/ipiecoles/java/eval/mdd050/service/ArtistService.java

Lines 76 to 80 in 637f6c6

	public Artist save(Artist artist) {
	// TODO Auto-generated method stub
	return null;
	}
	}