add logging to exception handler and service

Author: ColtonRandall

src/main/java/com/taskmanagerapi/exception/GlobalExceptionHandler.java

@@ -1,5 +1,7 @@
 package com.taskmanagerapi.exception;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.MethodArgumentNotValidException;
@@ -14,8 +16,11 @@
 @RestControllerAdvice
 public class GlobalExceptionHandler {
 
+    private static final Logger log = LoggerFactory.getLogger(GlobalExceptionHandler.class);
+
     @ExceptionHandler(TaskNotFoundException.class)
     public ResponseEntity<Map<String, Object>> handleTaskNotFound(TaskNotFoundException ex) {
+        log.warn("Task not found: {}", ex.getMessage());
         return ResponseEntity.status(HttpStatus.NOT_FOUND).body(
                 Map.of(
                         "error", ex.getMessage(),
@@ -26,7 +31,8 @@ public ResponseEntity<Map<String, Object>> handleTaskNotFound(TaskNotFoundExcept
     }
 
     @ExceptionHandler(Exception.class)
-    public ResponseEntity<Map<String, Object>> handleGenericError() {
+    public ResponseEntity<Map<String, Object>> handleGenericError(Exception ex) {
+        log.error("Unexpected error: {}", ex.getMessage(), ex);
         return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body(
                 Map.of(
                         "error", "An unexpected error occurred",

src/main/java/com/taskmanagerapi/service/TaskService.java

@@ -5,6 +5,8 @@
 import com.taskmanagerapi.model.Task;
 import com.taskmanagerapi.model.TaskStatus;
 import com.taskmanagerapi.repository.TaskRepository;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -15,6 +17,8 @@
 @Service
 public class TaskService {
 
+    private static final Logger log = LoggerFactory.getLogger(TaskService.class);
+
     private final TaskRepository taskRepository;
 
     public TaskService(TaskRepository taskRepository) {
@@ -23,11 +27,14 @@ public TaskService(TaskRepository taskRepository) {
 
     public TaskResponse addTask(String name, String description) {
         Task task = new Task(name, description);
-        return TaskResponse.from(taskRepository.save(task));
+        TaskResponse response = TaskResponse.from(taskRepository.save(task));
+        log.info("Task created: id={}, name={}", response.getId(), response.getName());
+        return response;
     }
 
     public List<TaskResponse> getAllTasks() {
         List<Task> tasks = taskRepository.findAll();
+        log.info("Fetched {} tasks", tasks.size());
         return tasks.stream()
                 .map(TaskResponse::from)
                 .toList();
@@ -47,6 +54,7 @@ public TaskResponse searchTaskByName(String name) {
 
     public List<TaskResponse> searchTaskByStatus(TaskStatus status) {
         List<Task> tasks = taskRepository.findTasksByStatus(status);
+        log.info("Fetched {} tasks with status={}", tasks.size(), status);
         return tasks.stream()
                 .map(TaskResponse::from)
                 .toList();
@@ -57,6 +65,7 @@ public TaskResponse completeTask(String id) {
         Task task = taskRepository.findById(id)
                 .orElseThrow(() -> new TaskNotFoundException(id));
         task.setStatus(DONE);
+        log.info("Task completed: id={}", id);
         return TaskResponse.from(task);
     }
 
@@ -65,6 +74,7 @@ public TaskResponse deleteTask(String id) {
         Task task = taskRepository.findById(id)
                 .orElseThrow(() -> new TaskNotFoundException(id));
         task.setStatus(DELETED);
+        log.info("Task deleted: id={}", id);
         return TaskResponse.from(task);
     }
 
@@ -73,6 +83,7 @@ public TaskResponse undoDelete(String id) {
         Task task = taskRepository.findById(id)
                 .orElseThrow(() -> new TaskNotFoundException(id));
         task.setStatus(CREATED);
+        log.info("Task restored: id={}", id);
         return TaskResponse.from(task);
     }
 }