From 775cd12a251fdfe30ed39ee7b715c1c1ec4d4a6e Mon Sep 17 00:00:00 2001
From: Hyungi Ahn <hyungiahn@Hyungiui-MacBookPro.local>
Date: Mon, 3 Nov 2025 10:55:44 +0900
Subject: [PATCH] =?UTF-8?q?refactor:=20taskController.js=20=EC=83=88?=
 =?UTF-8?q?=EB=A1=9C=EC=9A=B4=20=EC=9C=A0=ED=8B=B8=EB=A6=AC=ED=8B=B0=20?=
 =?UTF-8?q?=EC=A0=81=EC=9A=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- 새로운 에러 처리 시스템 적용:
  * asyncHandler로 비동기 함수 래핑
  * ApiError 클래스로 유효성 검사 에러 처리
  * handleDatabaseError로 DB 에러 표준화
  * handleNotFoundError로 404 에러 표준화

- 새로운 응답 포맷터 적용:
  * res.created() - 작업 생성 성공 응답
  * res.list() - 작업 목록 조회 응답
  * res.success() - 단일 작업 조회 응답
  * res.updated() - 작업 수정 성공 응답
  * res.deleted() - 작업 삭제 성공 응답

- 유효성 검사 개선:
  * ID 파라미터 유효성 검사 추가
  * NaN 체크로 잘못된 ID 요청 차단

- 함수별 개선사항:
  * createTask: 생성 응답 포맷터 적용
  * getAllTasks: 목록 응답 포맷터 적용
  * getTaskById: ID 검증 및 404 처리 개선
  * updateTask: ID 검증 및 수정 응답 포맷터
  * removeTask: ID 검증 및 삭제 응답 포맷터
---
 api.hyungi.net/controllers/taskController.js | 89 ++++++++++++++------
 1 file changed, 61 insertions(+), 28 deletions(-)

diff --git a/api.hyungi.net/controllers/taskController.js b/api.hyungi.net/controllers/taskController.js
index 9ac6142..c0d6f79 100644
--- a/api.hyungi.net/controllers/taskController.js
+++ b/api.hyungi.net/controllers/taskController.js
@@ -1,69 +1,102 @@
 const taskModel = require('../models/taskModel');
+const { ApiError, asyncHandler, handleDatabaseError, handleNotFoundError } = require('../utils/errorHandler');
+const { validateSchema, schemas } = require('../utils/validator');
 
 // 1. 생성
-exports.createTask = async (req, res) => {
+exports.createTask = asyncHandler(async (req, res) => {
+  const taskData = req.body;
+  
   try {
-    const taskData = req.body;
     const lastID = await new Promise((resolve, reject) => {
       taskModel.create(taskData, (err, id) => (err ? reject(err) : resolve(id)));
     });
-    res.json({ success: true, task_id: lastID });
+    
+    res.created({ task_id: lastID }, '작업이 성공적으로 생성되었습니다.');
   } catch (err) {
-    res.status(500).json({ error: err.message || String(err) });
+    handleDatabaseError(err, '작업 생성');
   }
-};
+});
 
 // 2. 전체 조회
-exports.getAllTasks = async (req, res) => {
+exports.getAllTasks = asyncHandler(async (req, res) => {
   try {
     const rows = await new Promise((resolve, reject) => {
       taskModel.getAll((err, data) => (err ? reject(err) : resolve(data)));
     });
-    res.json(rows);
+    
+    res.list(rows, '작업 목록 조회 성공');
   } catch (err) {
-    res.status(500).json({ error: err.message || String(err) });
+    handleDatabaseError(err, '작업 목록 조회');
   }
-};
+});
 
 // 3. 단일 조회
-exports.getTaskById = async (req, res) => {
+exports.getTaskById = asyncHandler(async (req, res) => {
+  const id = parseInt(req.params.task_id, 10);
+  
+  if (isNaN(id)) {
+    throw new ApiError('유효하지 않은 작업 ID입니다.', 400);
+  }
+  
   try {
-    const id = parseInt(req.params.task_id, 10);
     const row = await new Promise((resolve, reject) => {
       taskModel.getById(id, (err, data) => (err ? reject(err) : resolve(data)));
     });
-    if (!row) return res.status(404).json({ error: 'Task not found' });
-    res.json(row);
+    
+    if (!row) {
+      handleNotFoundError('작업', id);
+    }
+    
+    res.success(row, '작업 조회 성공');
   } catch (err) {
-    res.status(500).json({ error: err.message || String(err) });
+    handleDatabaseError(err, '작업 조회');
   }
-};
+});
 
 // 4. 수정
-exports.updateTask = async (req, res) => {
+exports.updateTask = asyncHandler(async (req, res) => {
+  const id = parseInt(req.params.task_id, 10);
+  
+  if (isNaN(id)) {
+    throw new ApiError('유효하지 않은 작업 ID입니다.', 400);
+  }
+  
+  const taskData = { ...req.body, task_id: id };
+  
   try {
-    const id = parseInt(req.params.task_id, 10);
-    const taskData = { ...req.body, task_id: id };
     const changes = await new Promise((resolve, reject) => {
       taskModel.update(taskData, (err, ch) => (err ? reject(err) : resolve(ch)));
     });
-    if (changes === 0) return res.status(404).json({ error: 'Task not found or no change' });
-    res.json({ success: true, changes });
+    
+    if (changes === 0) {
+      handleNotFoundError('작업', id);
+    }
+    
+    res.updated({ changes }, '작업 정보가 성공적으로 수정되었습니다.');
   } catch (err) {
-    res.status(500).json({ error: err.message || String(err) });
+    handleDatabaseError(err, '작업 수정');
   }
-};
+});
 
 // 5. 삭제
-exports.removeTask = async (req, res) => {
+exports.removeTask = asyncHandler(async (req, res) => {
+  const id = parseInt(req.params.task_id, 10);
+  
+  if (isNaN(id)) {
+    throw new ApiError('유효하지 않은 작업 ID입니다.', 400);
+  }
+  
   try {
-    const id = parseInt(req.params.task_id, 10);
     const changes = await new Promise((resolve, reject) => {
       taskModel.remove(id, (err, ch) => (err ? reject(err) : resolve(ch)));
     });
-    if (changes === 0) return res.status(404).json({ error: 'Task not found' });
-    res.json({ success: true, changes });
+    
+    if (changes === 0) {
+      handleNotFoundError('작업', id);
+    }
+    
+    res.deleted('작업이 성공적으로 삭제되었습니다.');
   } catch (err) {
-    res.status(500).json({ error: err.message || String(err) });
+    handleDatabaseError(err, '작업 삭제');
   }
-};
\ No newline at end of file
+});
\ No newline at end of file