[#31] Feat : 태스크 상태 수정 API 구현

/task/status 경로에 patch 요청 시 해당 idx를 가진 Task의 status를 변경하도록 구현.
Service 단에서 매개변수의 범위가 올바르지 않을 때는 BAD REQUEST를 리턴하도록 하는 검증 로직(isChangeInfoInvalid) 추가.

todo-list-server/src/main/java/kr/codesquad/todo/controller/TaskController.java

@@ -41,4 +41,9 @@ public ResponseEntity<Task> add(@RequestBody Task task) {
     public ResponseEntity<Map<Integer, List<Task>>> load() {
         return taskService.getAllTasks();
     }
+
+    @PatchMapping("/task/status")
+    public ResponseEntity<Task> move(@RequestParam long idx, int status) {
+        return taskService.changeTaskStatus(idx, status);
+    }
 }

todo-list-server/src/main/java/kr/codesquad/todo/repository/JdbcTaskRepository.java

@@ -1,14 +1,12 @@
 package kr.codesquad.todo.repository;
 
 import kr.codesquad.todo.domain.Task;
-import org.springframework.http.ResponseEntity;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.jdbc.core.namedparam.MapSqlParameterSource;
 import org.springframework.jdbc.core.simple.SimpleJdbcInsert;
 import org.springframework.stereotype.Repository;
 
 import javax.sql.DataSource;
-import java.sql.SQLException;
 import java.time.LocalDateTime;
 import java.time.ZoneId;
 import java.util.HashMap;
@@ -19,7 +17,7 @@
 public class JdbcTaskRepository implements TaskRepository {
     private final JdbcTemplate jdbcTemplate;
 
-    public JdbcTaskRepository(DataSource dataSource) throws SQLException {
+    public JdbcTaskRepository(DataSource dataSource) {
         this.jdbcTemplate = new JdbcTemplate(dataSource);
     }
 
@@ -47,13 +45,26 @@ public Task add(Task task) {
     @Override
     public List<Task> getAll() {
         String sql = "select * from task order by created_at desc";
-        List<Task> tasks = jdbcTemplate.query(sql, (rs, rowCount) -> {
+        return jdbcTemplate.query(sql, (rs, rowCount) -> {
             Task task = new Task(rs.getString("title"), rs.getString("content"), rs.getString("author"), rs.getInt("status"));
 
             task.setCreateAt(rs.getTimestamp("created_at").toLocalDateTime());
             task.setIdx(rs.getInt("idx"));
             return task;
         });
-        return tasks;
+    }
+
+    @Override
+    public Task changeStatus(long idx, int status) {
+        String updateSql = "update task set status = ? where idx = ?";
+        jdbcTemplate.update(updateSql, status, idx);
+
+        String selectSql = "select * from task where idx = ?";
+        return jdbcTemplate.queryForObject(selectSql, (rs, rowNum) -> {
+            Task newTask = new Task(rs.getString("title"), rs.getString("content"), rs.getString("author"), rs.getInt("status"));
+            newTask.setIdx(rs.getLong("idx"));
+            newTask.setCreateAt(rs.getTimestamp("created_at").toLocalDateTime());
+            return newTask;
+        }, idx);
     }
 }

todo-list-server/src/main/java/kr/codesquad/todo/repository/TaskRepository.java

@@ -1,12 +1,13 @@
 package kr.codesquad.todo.repository;
 
 import kr.codesquad.todo.domain.Task;
-import org.springframework.http.ResponseEntity;
 
 import java.util.List;
 
 public interface TaskRepository {
-    public Task add(Task task);
+    Task add(Task task);
 
     List<Task> getAll();
+
+    Task changeStatus(long idx, int status);
 }

todo-list-server/src/main/java/kr/codesquad/todo/service/TaskService.java

@@ -67,4 +67,22 @@ public ResponseEntity<Map<Integer, List<Task>>> getAllTasks() {
 
         return ResponseEntity.status(HttpStatus.OK).body(taskMap);
     }
+
+    public ResponseEntity<Task> changeTaskStatus(long idx, int status) {
+        if (isChangeInfoInvalid(idx, status)) {
+            return ResponseEntity.status(HttpStatus.BAD_REQUEST).build();
+        }
+        Task task;
+        try {
+            task = taskRepository.changeStatus(idx, status);
+        } catch (Exception e) {
+            e.printStackTrace();
+            return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).build();
+        }
+        return ResponseEntity.status(HttpStatus.OK).body(task);
+    }
+
+    public boolean isChangeInfoInvalid(long idx, int status) {
+        return !(status == 1 || status == 2 || status == 3) || idx < 1;
+    }
 }