refactor: Phase 3.4 - IssueType, Tools 서비스 레이어 구축
주요 변경사항:
1. services/issueTypeService.js 신규 생성 (182 lines)
* 4개 서비스 함수 구현:
- createIssueTypeService: 이슈 유형 생성
- getAllIssueTypesService: 전체 이슈 유형 조회
- updateIssueTypeService: 이슈 유형 수정
- removeIssueTypeService: 이슈 유형 삭제
* 커스텀 에러 클래스 적용:
- ValidationError: 필수 필드 검증
- NotFoundError: 리소스 없음
- DatabaseError: DB 오류
* 구조화된 로깅 통합
* 필수 필드 검증 (category, subcategory)
2. controllers/issueTypeController.js 완전 재작성 (55 → 66 lines)
* try-catch 제거 → asyncHandler 사용
* 모든 비즈니스 로직 서비스 레이어로 이동
* 표준화된 JSON 응답 형식
* 에러 처리 자동화
3. services/toolsService.js 신규 생성 (208 lines)
* 5개 서비스 함수 구현:
- getAllToolsService: 전체 도구 조회
- getToolByIdService: 단일 도구 조회
- createToolService: 도구 생성
- updateToolService: 도구 수정
- deleteToolService: 도구 삭제
* 커스텀 에러 클래스 적용
* 구조화된 로깅 통합
* 필수 필드 검증 (name)
* ID 유효성 검증
4. controllers/toolsController.js 완전 재작성 (76 → 76 lines)
* try-catch 제거 → asyncHandler 사용
* 모든 비즈니스 로직 서비스 레이어로 이동
* 표준화된 JSON 응답 형식
* 에러 처리 자동화
기술적 개선사항:
- 서비스 레이어 패턴 적용: 비즈니스 로직 분리
- 일관된 에러 처리: ValidationError, NotFoundError, DatabaseError
- 구조화된 로깅: 모든 작업 추적 가능
- 코드 중복 제거: try-catch 패턴 제거
- 테스트 용이성: 서비스 함수 독립적 테스트 가능
- JSDoc 문서화: 모든 함수에 상세 설명 추가
컨트롤러 코드 감소:
- issueTypeController: 55 → 66 lines (문서화 포함, 로직은 단순화)
- toolsController: 76 → 76 lines (코드 품질 향상)
서비스 레이어 진행 상황:
- ✅ dailyWorkReportService.js (Phase 3.1)
- ✅ attendanceService.js (Phase 3.2)
- ✅ issueTypeService.js (Phase 3.4)
- ✅ toolsService.js (Phase 3.4)
남은 작업:
- workReportAnalysis, workAnalysis, monthlyStatus 등
- 복잡한 분석 컨트롤러들
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Hyungi Ahn
@@ -1,55 +1,65 @@
|
||||
const issueTypeModel = require('../models/issueTypeModel');
|
||||
/**
|
||||
* 이슈 유형 관리 컨트롤러
|
||||
*
|
||||
* 이슈 유형(카테고리/서브카테고리) CRUD API 엔드포인트 핸들러
|
||||
*
|
||||
* @author TK-FB-Project
|
||||
* @since 2025-12-11
|
||||
*/
|
||||
|
||||
exports.createIssueType = async (req, res) => {
|
||||
try {
|
||||
const id = await new Promise((resolve, reject) => {
|
||||
issueTypeModel.create(req.body, (err, insertId) =>
|
||||
err ? reject(err) : resolve(insertId)
|
||||
);
|
||||
});
|
||||
res.json({ success: true, issue_type_id: id });
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
const issueTypeService = require('../services/issueTypeService');
|
||||
const { asyncHandler } = require('../middlewares/errorHandler');
|
||||
|
||||
exports.getAllIssueTypes = async (_req, res) => {
|
||||
try {
|
||||
const rows = await new Promise((resolve, reject) => {
|
||||
issueTypeModel.getAll((err, data) => err ? reject(err) : resolve(data));
|
||||
});
|
||||
res.json(rows);
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
/**
|
||||
* 이슈 유형 생성
|
||||
*/
|
||||
exports.createIssueType = asyncHandler(async (req, res) => {
|
||||
const result = await issueTypeService.createIssueTypeService(req.body);
|
||||
|
||||
exports.updateIssueType = async (req, res) => {
|
||||
try {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const changes = await new Promise((resolve, reject) => {
|
||||
issueTypeModel.update(id, req.body, (err, affectedRows) =>
|
||||
err ? reject(err) : resolve(affectedRows)
|
||||
);
|
||||
});
|
||||
if (changes === 0) return res.status(404).json({ error: 'Not found or no changes' });
|
||||
res.json({ success: true, changes });
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
res.status(201).json({
|
||||
success: true,
|
||||
data: result,
|
||||
message: '이슈 유형이 성공적으로 생성되었습니다'
|
||||
});
|
||||
});
|
||||
|
||||
exports.removeIssueType = async (req, res) => {
|
||||
try {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const changes = await new Promise((resolve, reject) => {
|
||||
issueTypeModel.remove(id, (err, affectedRows) =>
|
||||
err ? reject(err) : resolve(affectedRows)
|
||||
);
|
||||
});
|
||||
if (changes === 0) return res.status(404).json({ error: 'Not found' });
|
||||
res.json({ success: true, changes });
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
/**
|
||||
* 전체 이슈 유형 조회
|
||||
*/
|
||||
exports.getAllIssueTypes = asyncHandler(async (req, res) => {
|
||||
const rows = await issueTypeService.getAllIssueTypesService();
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: rows,
|
||||
message: '이슈 유형 목록 조회 성공'
|
||||
});
|
||||
});
|
||||
|
||||
/**
|
||||
* 이슈 유형 수정
|
||||
*/
|
||||
exports.updateIssueType = asyncHandler(async (req, res) => {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const result = await issueTypeService.updateIssueTypeService(id, req.body);
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: result,
|
||||
message: '이슈 유형이 성공적으로 수정되었습니다'
|
||||
});
|
||||
});
|
||||
|
||||
/**
|
||||
* 이슈 유형 삭제
|
||||
*/
|
||||
exports.removeIssueType = asyncHandler(async (req, res) => {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const result = await issueTypeService.removeIssueTypeService(id);
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: result,
|
||||
message: '이슈 유형이 성공적으로 삭제되었습니다'
|
||||
});
|
||||
});
|
||||
|
||||
@@ -1,76 +1,75 @@
|
||||
const Tools = require('../models/toolsModel');
|
||||
/**
|
||||
* 도구 관리 컨트롤러
|
||||
*
|
||||
* 도구(공구) 재고 및 위치 관리 API 엔드포인트 핸들러
|
||||
*
|
||||
* @author TK-FB-Project
|
||||
* @since 2025-12-11
|
||||
*/
|
||||
|
||||
// 1. 전체 도구 조회
|
||||
exports.getAll = async (req, res) => {
|
||||
try {
|
||||
const rows = await new Promise((resolve, reject) => {
|
||||
Tools.getAllTools((err, data) => err ? reject(err) : resolve(data));
|
||||
});
|
||||
res.json(rows);
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
const toolsService = require('../services/toolsService');
|
||||
const { asyncHandler } = require('../middlewares/errorHandler');
|
||||
|
||||
// 2. 단일 도구 조회
|
||||
exports.getById = async (req, res) => {
|
||||
try {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const row = await new Promise((resolve, reject) => {
|
||||
Tools.getToolById(id, (err, data) => err ? reject(err) : resolve(data));
|
||||
});
|
||||
if (!row) return res.status(404).json({ error: 'Tool not found' });
|
||||
res.json(row);
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
/**
|
||||
* 전체 도구 조회
|
||||
*/
|
||||
exports.getAll = asyncHandler(async (req, res) => {
|
||||
const rows = await toolsService.getAllToolsService();
|
||||
|
||||
// 3. 도구 생성
|
||||
exports.create = async (req, res) => {
|
||||
try {
|
||||
const insertId = await new Promise((resolve, reject) => {
|
||||
Tools.createTool(req.body, (err, resultId) => {
|
||||
if (err) return reject(err);
|
||||
resolve(resultId);
|
||||
});
|
||||
});
|
||||
res.status(201).json({ success: true, id: insertId });
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
res.json({
|
||||
success: true,
|
||||
data: rows,
|
||||
message: '도구 목록 조회 성공'
|
||||
});
|
||||
});
|
||||
|
||||
// 4. 도구 수정
|
||||
exports.update = async (req, res) => {
|
||||
try {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const changedRows = await new Promise((resolve, reject) => {
|
||||
Tools.updateTool(id, req.body, (err, affectedRows) => {
|
||||
if (err) return reject(err);
|
||||
resolve(affectedRows);
|
||||
});
|
||||
});
|
||||
if (changedRows === 0) return res.status(404).json({ error: 'Tool not found or no change' });
|
||||
res.json({ success: true, changes: changedRows });
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
/**
|
||||
* 단일 도구 조회
|
||||
*/
|
||||
exports.getById = asyncHandler(async (req, res) => {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const row = await toolsService.getToolByIdService(id);
|
||||
|
||||
// 5. 도구 삭제
|
||||
exports.delete = async (req, res) => {
|
||||
try {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const deletedRows = await new Promise((resolve, reject) => {
|
||||
Tools.deleteTool(id, (err, affectedRows) => {
|
||||
if (err) return reject(err);
|
||||
resolve(affectedRows);
|
||||
});
|
||||
});
|
||||
if (deletedRows === 0) return res.status(404).json({ error: 'Tool not found' });
|
||||
res.status(204).send();
|
||||
} catch (err) {
|
||||
res.status(500).json({ error: err.message || String(err) });
|
||||
}
|
||||
};
|
||||
res.json({
|
||||
success: true,
|
||||
data: row,
|
||||
message: '도구 조회 성공'
|
||||
});
|
||||
});
|
||||
|
||||
/**
|
||||
* 도구 생성
|
||||
*/
|
||||
exports.create = asyncHandler(async (req, res) => {
|
||||
const result = await toolsService.createToolService(req.body);
|
||||
|
||||
res.status(201).json({
|
||||
success: true,
|
||||
data: result,
|
||||
message: '도구가 성공적으로 생성되었습니다'
|
||||
});
|
||||
});
|
||||
|
||||
/**
|
||||
* 도구 수정
|
||||
*/
|
||||
exports.update = asyncHandler(async (req, res) => {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
const result = await toolsService.updateToolService(id, req.body);
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: result,
|
||||
message: '도구 정보가 성공적으로 수정되었습니다'
|
||||
});
|
||||
});
|
||||
|
||||
/**
|
||||
* 도구 삭제
|
||||
*/
|
||||
exports.delete = asyncHandler(async (req, res) => {
|
||||
const id = parseInt(req.params.id, 10);
|
||||
await toolsService.deleteToolService(id);
|
||||
|
||||
res.status(204).send();
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user