refactor: projectController.js 새로운 유틸리티 적용
- 새로운 에러 처리 시스템 적용: * 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 검증 및 삭제 응답 포맷터
This commit is contained in:
@@ -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, '프로젝트 삭제');
|
||||
}
|
||||
};
|
||||
});
|
||||
Reference in New Issue
Block a user