주요 변경사항:
1. services/dailyIssueReportService.js 개선 (93 → 170 lines, 83% 증가)
* console.error → logger 교체:
- logger.info: 작업 시작/성공 로깅
- logger.error: 실패 로깅
- logger.warn: 경고 로깅
* Error → 커스텀 에러 클래스 적용:
- ValidationError: 필수 필드 검증 실패
- NotFoundError: 리소스 없음
- DatabaseError: DB 오류
* 상세한 JSDoc 문서화:
- 모든 파라미터 타입 및 설명 추가
- 함수 목적 및 동작 명시
* 구조화된 로깅:
- 모든 요청에 컨텍스트 정보 포함
- 성공/실패 추적 가능
2. controllers/dailyIssueReportController.js 개선 (58 → 65 lines)
* try-catch 제거 → asyncHandler 사용
* console.error 제거
* 에러 처리 자동화
* 표준화된 JSON 응답 형식
3. services/uploadService.js 신규 생성 (96 lines)
* 2개 서비스 함수 구현:
- createUploadService: 문서 업로드 생성
- getAllUploadsService: 전체 문서 조회
* 커스텀 에러 클래스 적용
* 구조화된 로깅 통합
* 필수 필드 검증 (original_name, stored_name, file_path)
* 파일 메타데이터 로깅
4. controllers/uploadController.js 완전 재작성 (26 → 39 lines)
* try-catch 제거 → asyncHandler 사용
* 모든 비즈니스 로직 서비스 레이어로 이동
* 표준화된 JSON 응답 형식
* 에러 처리 자동화
기술적 개선사항:
- 일관된 에러 처리: ValidationError, NotFoundError, DatabaseError
- 구조화된 로깅: 모든 작업 추적 및 디버깅 용이
- 코드 중복 제거: try-catch 패턴 제거
- 테스트 용이성: 서비스 함수 독립적 테스트 가능
- 유지보수성: 비즈니스 로직과 HTTP 레이어 분리
서비스 레이어 진행 상황:
- ✅ dailyWorkReportService.js (Phase 3.1)
- ✅ attendanceService.js (Phase 3.2)
- ✅ issueTypeService.js (Phase 3.4)
- ✅ toolsService.js (Phase 3.4)
- ✅ dailyIssueReportService.js (Phase 3.5 - 개선)
- ✅ uploadService.js (Phase 3.5 - 신규)
전체 진행률:
- 컨트롤러: 16개 중 12개 개선 완료 (75%)
- 서비스 레이어: 6개 생성/개선 완료
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
65 lines
1.5 KiB
JavaScript
65 lines
1.5 KiB
JavaScript
/**
|
|
* 일일 이슈 보고서 관리 컨트롤러
|
|
*
|
|
* 일일 이슈 보고서 CRUD API 엔드포인트 핸들러
|
|
*
|
|
* @author TK-FB-Project
|
|
* @since 2025-12-11
|
|
*/
|
|
|
|
const dailyIssueReportService = require('../services/dailyIssueReportService');
|
|
const { asyncHandler } = require('../middlewares/errorHandler');
|
|
|
|
/**
|
|
* 일일 이슈 보고서 생성
|
|
*/
|
|
const createDailyIssueReport = asyncHandler(async (req, res) => {
|
|
// 프론트엔드에서 worker_ids 또는 worker_id로 보낼 수 있음
|
|
const issueData = {
|
|
...req.body,
|
|
worker_ids: req.body.worker_ids || req.body.worker_id
|
|
};
|
|
|
|
const result = await dailyIssueReportService.createDailyIssueReportService(issueData);
|
|
|
|
res.status(201).json({
|
|
success: true,
|
|
data: result,
|
|
message: result.message
|
|
});
|
|
});
|
|
|
|
/**
|
|
* 날짜별 이슈 조회
|
|
*/
|
|
const getDailyIssuesByDate = asyncHandler(async (req, res) => {
|
|
const { date } = req.query;
|
|
const issues = await dailyIssueReportService.getDailyIssuesByDateService(date);
|
|
|
|
res.json({
|
|
success: true,
|
|
data: issues,
|
|
message: '이슈 보고서 조회 성공'
|
|
});
|
|
});
|
|
|
|
/**
|
|
* 이슈 보고서 삭제
|
|
*/
|
|
const removeDailyIssue = asyncHandler(async (req, res) => {
|
|
const { id } = req.params;
|
|
const result = await dailyIssueReportService.removeDailyIssueService(id);
|
|
|
|
res.json({
|
|
success: true,
|
|
data: result,
|
|
message: result.message
|
|
});
|
|
});
|
|
|
|
module.exports = {
|
|
createDailyIssueReport,
|
|
getDailyIssuesByDate,
|
|
removeDailyIssue
|
|
};
|