From 99210fec1b114f21cd485c7c2a39904535d76dfe Mon Sep 17 00:00:00 2001
From: Hyungi Ahn <hyungiahn@Hyungiui-MacBookPro.local>
Date: Mon, 3 Nov 2025 10:52:39 +0900
Subject: [PATCH] =?UTF-8?q?refactor:=20projectController.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() - 프로젝트 삭제 성공 응답

- 유효성 검사 시스템 적용:
  * validateSchema로 스키마 기반 검증
  * schemas.createProject 스키마 사용
  * ID 파라미터 유효성 검사 추가

- 함수별 개선사항:
  * createProject: 스키마 검증 및 생성 응답 포맷터
  * getAllProjects: 목록 응답 포맷터 적용
  * getProjectById: ID 검증 및 404 처리 개선
  * updateProject: ID 검증 및 수정 응답 포맷터
  * removeProject: ID 검증 및 삭제 응답 포맷터
---
 .../controllers/projectController.js          | 92 +++++++++++++------
 1 file changed, 64 insertions(+), 28 deletions(-)

diff --git a/api.hyungi.net/controllers/projectController.js b/api.hyungi.net/controllers/projectController.js
index cc96ebd..dd3158d 100644
--- a/api.hyungi.net/controllers/projectController.js
+++ b/api.hyungi.net/controllers/projectController.js
@@ -1,69 +1,105 @@
 const projectModel = require('../models/projectModel');
+const { ApiError, asyncHandler, handleDatabaseError, handleNotFoundError } = require('../utils/errorHandler');
+const { validateSchema, schemas } = require('../utils/validator');
 
 // 1. 프로젝트 생성
-exports.createProject = async (req, res) => {
+exports.createProject = asyncHandler(async (req, res) => {
+  const projectData = req.body;
+  
+  // 스키마 기반 유효성 검사
+  validateSchema(projectData, schemas.createProject);
+  
   try {
-    const projectData = req.body;
     const id = await new Promise((resolve, reject) => {
       projectModel.create(projectData, (err, lastID) => (err ? reject(err) : resolve(lastID)));
     });
-    res.json({ success: true, project_id: id });
+    
+    res.created({ project_id: id }, '프로젝트가 성공적으로 생성되었습니다.');
   } catch (err) {
-    res.status(500).json({ error: err.message || String(err) });
+    handleDatabaseError(err, '프로젝트 생성');
   }
-};
+});
 
 // 2. 전체 조회
-exports.getAllProjects = async (req, res) => {
+exports.getAllProjects = asyncHandler(async (req, res) => {
   try {
     const rows = await new Promise((resolve, reject) => {
       projectModel.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.getProjectById = async (req, res) => {
+exports.getProjectById = asyncHandler(async (req, res) => {
+  const id = parseInt(req.params.project_id, 10);
+  
+  if (isNaN(id)) {
+    throw new ApiError('유효하지 않은 프로젝트 ID입니다.', 400);
+  }
+  
   try {
-    const id = parseInt(req.params.project_id, 10);
     const row = await new Promise((resolve, reject) => {
       projectModel.getById(id, (err, data) => (err ? reject(err) : resolve(data)));
     });
-    if (!row) return res.status(404).json({ error: 'Project 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.updateProject = async (req, res) => {
+exports.updateProject = asyncHandler(async (req, res) => {
+  const id = parseInt(req.params.project_id, 10);
+  
+  if (isNaN(id)) {
+    throw new ApiError('유효하지 않은 프로젝트 ID입니다.', 400);
+  }
+  
+  const data = { ...req.body, project_id: id };
+  
   try {
-    const id = parseInt(req.params.project_id, 10);
-    const data = { ...req.body, project_id: id };
     const changes = await new Promise((resolve, reject) => {
       projectModel.update(data, (err, ch) => (err ? reject(err) : resolve(ch)));
     });
-    if (changes === 0) return res.status(404).json({ error: 'Project not found or no changes' });
-    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.removeProject = async (req, res) => {
+exports.removeProject = asyncHandler(async (req, res) => {
+  const id = parseInt(req.params.project_id, 10);
+  
+  if (isNaN(id)) {
+    throw new ApiError('유효하지 않은 프로젝트 ID입니다.', 400);
+  }
+  
   try {
-    const id = parseInt(req.params.project_id, 10);
     const changes = await new Promise((resolve, reject) => {
       projectModel.remove(id, (err, ch) => (err ? reject(err) : resolve(ch)));
     });
-    if (changes === 0) return res.status(404).json({ error: 'Project 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