feat: implement DELETE /items/{id} endpoint

- Remove all try/catch blocks
- Add @RequestMapping class-level mapping
- Let Spring handle validation and error responses automatically

Author: lquinoa252

lesson_23/api/java/api_app/src/main/java/com/codedifferently/lesson23/web/MediaItemsController.java

@@ -9,16 +9,20 @@
 import java.util.Set;
 import java.util.UUID;
 import jakarta.validation.Valid;
+import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.CrossOrigin;
+import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestBody;
+import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
 @RestController
 @CrossOrigin
+@RequestMapping
 public class MediaItemsController {
 
   private final Library library;
@@ -30,45 +34,43 @@ public MediaItemsController(Library library) throws IOException {
   }
 
   @GetMapping("/items")
-  public ResponseEntity<GetMediaItemsResponse> getItems() {
+  public GetMediaItemsResponse getItems() {
     Set<MediaItem> items = library.search(SearchCriteria.builder().build());
     List<MediaItemResponse> responseItems = items.stream().map(MediaItemResponse::from).toList();
-    var response = GetMediaItemsResponse.builder().items(responseItems).build();
-    return ResponseEntity.ok(response);
+    return GetMediaItemsResponse.builder().items(responseItems).build();
   }
 
   @GetMapping("/items/{id}")
   public ResponseEntity<MediaItemResponse> getItem(@PathVariable String id) {
-    try {
-      UUID itemId = UUID.fromString(id);
-      Set<MediaItem> items = library.search(SearchCriteria.builder().id(itemId.toString()).build());
-      
-      if (items.isEmpty()) {
-        return ResponseEntity.notFound().build();
-      }
-      
-      MediaItem item = items.iterator().next();
-      MediaItemResponse response = MediaItemResponse.from(item);
-      return ResponseEntity.ok(response);
-    } catch (IllegalArgumentException e) {
-      return ResponseEntity.badRequest().build();
+    UUID itemId = UUID.fromString(id);
+    Set<MediaItem> items = library.search(SearchCriteria.builder().id(itemId.toString()).build());
+    
+    if (items.isEmpty()) {
+      return ResponseEntity.notFound().build();
     }
+    
+    MediaItem item = items.iterator().next();
+    return ResponseEntity.ok(MediaItemResponse.from(item));
   }
 
   @PostMapping("/items")
-  public ResponseEntity<CreateMediaItemResponse> createItem(@Valid @RequestBody CreateMediaItemRequest request) {
-    try {
-      MediaItem newItem = MediaItemRequest.asMediaItem(request.getItem());
-      library.addMediaItem(newItem, librarian);
-      
-      MediaItemResponse itemResponse = MediaItemResponse.from(newItem);
-      CreateMediaItemResponse response = CreateMediaItemResponse.builder()
-          .item(itemResponse)
-          .build();
-      
-      return ResponseEntity.ok(response);
-    } catch (Exception e) {
-      return ResponseEntity.badRequest().build();
+  public CreateMediaItemResponse createItem(@Valid @RequestBody CreateMediaItemRequest request) {
+    MediaItem newItem = MediaItemRequest.asMediaItem(request.getItem());
+    library.addMediaItem(newItem, librarian);
+    
+    MediaItemResponse itemResponse = MediaItemResponse.from(newItem);
+    return CreateMediaItemResponse.builder().item(itemResponse).build();
+  }
+
+  @DeleteMapping("/items/{id}")
+  public ResponseEntity<Void> deleteItem(@PathVariable String id) {
+    UUID itemId = UUID.fromString(id);
+    
+    if (!library.hasMediaItem(itemId)) {
+      return ResponseEntity.notFound().build();
     }
+    
+    library.removeMediaItem(itemId, librarian);
+    return ResponseEntity.noContent().build();
   }
 }