feat: Phase 3.8 - 복잡한 분석 컨트롤러 개선
두 개의 복잡한 분석 컨트롤러를 현대적인 패턴으로 전면 개선: ## workReportAnalysisController.js (381 → 430 lines) - 7개 SQL 쿼리 기반 복합 분석 엔드포인트 개선 - console.error → logger.info/error/warn 전환 - try-catch → asyncHandler 미들웨어 적용 - Error → ValidationError, DatabaseError 전환 - JSDoc 문서화 및 구조화된 로깅 추가 - 4개 함수: getAnalysisFilters, getAnalyticsByPeriod, getProjectAnalysis, getWorkerAnalysis ## workAnalysisController.js (523 → 622 lines) - 클래스 기반 → 함수 기반 컨트롤러 전환 - console.error → logger.info/error/debug 전환 - try-catch → asyncHandler 미들웨어 적용 - Error → ValidationError, DatabaseError 전환 - validateDateRange 헬퍼 함수 개선 (상세한 에러 컨텍스트) - JSDoc 문서화 및 구조화된 로깅 추가 - 12개 함수: getStats, getDailyTrend, getWorkerStats, getProjectStats, getWorkTypeStats, getRecentWork, getWeekdayPattern, getErrorAnalysis, getMonthlyComparison, getWorkerSpecialization, getDashboardData, getProjectWorkTypeAnalysis ## 기술적 개선사항 - 통합 에러 처리: 커스텀 에러 클래스로 일관된 에러 핸들링 - 구조화된 로깅: 모든 API 호출에 컨텍스트 정보 포함 - 자동 에러 전파: asyncHandler로 보일러플레이트 코드 제거 - 향상된 유효성 검사: 상세한 에러 메시지와 컨텍스트 - 프로덕션 준비: 표준화된 응답 형식 및 에러 처리 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
File diff suppressed because it is too large
Load Diff
@@ -1,14 +1,26 @@
|
||||
// controllers/workReportAnalysisController.js - 데일리 워크 레포트 분석 전용 컨트롤러
|
||||
const dailyWorkReportModel = require('../models/dailyWorkReportModel');
|
||||
/**
|
||||
* 데일리 워크 레포트 분석 컨트롤러
|
||||
*
|
||||
* 작업 보고서 종합 분석 API 엔드포인트 핸들러
|
||||
*
|
||||
* @author TK-FB-Project
|
||||
* @since 2025-12-11
|
||||
*/
|
||||
|
||||
const { getDb } = require('../dbPool');
|
||||
const { ValidationError, DatabaseError } = require('../utils/errors');
|
||||
const { asyncHandler } = require('../middlewares/errorHandler');
|
||||
const logger = require('../utils/logger');
|
||||
|
||||
/**
|
||||
* 📋 분석용 필터 데이터 조회 (프로젝트, 작업자, 작업유형 목록)
|
||||
* 분석용 필터 데이터 조회 (프로젝트, 작업자, 작업유형 목록)
|
||||
*/
|
||||
const getAnalysisFilters = async (req, res) => {
|
||||
try {
|
||||
const db = await getDb();
|
||||
const getAnalysisFilters = asyncHandler(async (req, res) => {
|
||||
logger.info('분석 필터 데이터 조회 요청');
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
try {
|
||||
// 프로젝트 목록
|
||||
const [projects] = await db.query(`
|
||||
SELECT DISTINCT p.project_id, p.project_name
|
||||
@@ -33,7 +45,7 @@ const getAnalysisFilters = async (req, res) => {
|
||||
ORDER BY wt.name
|
||||
`);
|
||||
|
||||
// 날짜 범위 (최초/최신 데이터)
|
||||
// 날짜 범위
|
||||
const [dateRange] = await db.query(`
|
||||
SELECT
|
||||
MIN(report_date) as min_date,
|
||||
@@ -41,6 +53,12 @@ const getAnalysisFilters = async (req, res) => {
|
||||
FROM daily_work_reports
|
||||
`);
|
||||
|
||||
logger.info('분석 필터 데이터 조회 성공', {
|
||||
projects: projects.length,
|
||||
workers: workers.length,
|
||||
workTypes: workTypes.length
|
||||
});
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: {
|
||||
@@ -48,47 +66,48 @@ const getAnalysisFilters = async (req, res) => {
|
||||
workers,
|
||||
workTypes,
|
||||
dateRange: dateRange[0]
|
||||
}
|
||||
},
|
||||
message: '분석 필터 데이터 조회 성공'
|
||||
});
|
||||
|
||||
} catch (error) {
|
||||
console.error('필터 데이터 조회 오류:', error);
|
||||
res.status(500).json({
|
||||
success: false,
|
||||
error: '필터 데이터 조회 중 오류가 발생했습니다.',
|
||||
detail: error.message
|
||||
});
|
||||
logger.error('분석 필터 데이터 조회 실패', { error: error.message });
|
||||
throw new DatabaseError('필터 데이터 조회 중 오류가 발생했습니다');
|
||||
}
|
||||
};
|
||||
});
|
||||
|
||||
/**
|
||||
* 📊 기간별 작업 분석 데이터 조회
|
||||
* 기간별 작업 분석 데이터 조회
|
||||
*/
|
||||
const getAnalyticsByPeriod = async (req, res) => {
|
||||
const getAnalyticsByPeriod = asyncHandler(async (req, res) => {
|
||||
const { start_date, end_date, project_id, worker_id } = req.query;
|
||||
|
||||
if (!start_date || !end_date) {
|
||||
throw new ValidationError('start_date와 end_date가 필요합니다', {
|
||||
required: ['start_date', 'end_date'],
|
||||
received: { start_date, end_date },
|
||||
example: 'start_date=2025-08-01&end_date=2025-08-31'
|
||||
});
|
||||
}
|
||||
|
||||
logger.info('기간별 분석 데이터 조회 요청', {
|
||||
start_date,
|
||||
end_date,
|
||||
project_id,
|
||||
worker_id
|
||||
});
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
try {
|
||||
const { start_date, end_date, project_id, worker_id } = req.query;
|
||||
|
||||
if (!start_date || !end_date) {
|
||||
return res.status(400).json({
|
||||
success: false,
|
||||
error: 'start_date와 end_date가 필요합니다.',
|
||||
example: 'start_date=2025-08-01&end_date=2025-08-31'
|
||||
});
|
||||
}
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
// 기본 조건
|
||||
let whereConditions = ['dwr.report_date BETWEEN ? AND ?'];
|
||||
let queryParams = [start_date, end_date];
|
||||
|
||||
// 프로젝트 필터
|
||||
if (project_id) {
|
||||
whereConditions.push('dwr.project_id = ?');
|
||||
queryParams.push(project_id);
|
||||
}
|
||||
|
||||
// 작업자 필터
|
||||
if (worker_id) {
|
||||
whereConditions.push('dwr.worker_id = ?');
|
||||
queryParams.push(worker_id);
|
||||
@@ -96,7 +115,7 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const whereClause = whereConditions.join(' AND ');
|
||||
|
||||
// 1. 전체 요약 통계 (에러 분석 포함)
|
||||
// 1. 전체 요약 통계
|
||||
const overallSql = `
|
||||
SELECT
|
||||
COUNT(*) as total_entries,
|
||||
@@ -129,7 +148,7 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const [dailyStats] = await db.query(dailyStatsSql, queryParams);
|
||||
|
||||
// 2.5. 일별 에러 발생 통계
|
||||
// 3. 일별 에러 통계
|
||||
const dailyErrorStatsSql = `
|
||||
SELECT
|
||||
dwr.report_date,
|
||||
@@ -144,7 +163,7 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const [dailyErrorStats] = await db.query(dailyErrorStatsSql, queryParams);
|
||||
|
||||
// 3. 에러 유형별 분석 (간단한 방식으로 수정)
|
||||
// 4. 에러 유형별 분석
|
||||
const errorAnalysisSql = `
|
||||
SELECT
|
||||
et.id as error_type_id,
|
||||
@@ -161,7 +180,7 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const [errorAnalysis] = await db.query(errorAnalysisSql, queryParams);
|
||||
|
||||
// 4. 작업 유형별 분석
|
||||
// 5. 작업 유형별 분석
|
||||
const workTypeAnalysisSql = `
|
||||
SELECT
|
||||
wt.id as work_type_id,
|
||||
@@ -180,7 +199,7 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const [workTypeAnalysis] = await db.query(workTypeAnalysisSql, queryParams);
|
||||
|
||||
// 5. 작업자별 성과 분석
|
||||
// 6. 작업자별 성과 분석
|
||||
const workerAnalysisSql = `
|
||||
SELECT
|
||||
w.worker_id,
|
||||
@@ -201,7 +220,7 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const [workerAnalysis] = await db.query(workerAnalysisSql, queryParams);
|
||||
|
||||
// 6. 프로젝트별 분석
|
||||
// 7. 프로젝트별 분석
|
||||
const projectAnalysisSql = `
|
||||
SELECT
|
||||
p.project_id,
|
||||
@@ -222,6 +241,13 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
|
||||
const [projectAnalysis] = await db.query(projectAnalysisSql, queryParams);
|
||||
|
||||
logger.info('기간별 분석 데이터 조회 성공', {
|
||||
start_date,
|
||||
end_date,
|
||||
total_entries: overallStats[0].total_entries,
|
||||
total_hours: overallStats[0].total_hours
|
||||
});
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: {
|
||||
@@ -234,35 +260,41 @@ const getAnalyticsByPeriod = async (req, res) => {
|
||||
projectAnalysis,
|
||||
period: { start_date, end_date },
|
||||
filters: { project_id, worker_id }
|
||||
}
|
||||
},
|
||||
message: '기간별 분석 데이터 조회 성공'
|
||||
});
|
||||
|
||||
} catch (error) {
|
||||
console.error('기간별 분석 데이터 조회 오류:', error);
|
||||
res.status(500).json({
|
||||
success: false,
|
||||
error: '기간별 분석 데이터 조회 중 오류가 발생했습니다.',
|
||||
detail: error.message
|
||||
logger.error('기간별 분석 데이터 조회 실패', {
|
||||
start_date,
|
||||
end_date,
|
||||
error: error.message
|
||||
});
|
||||
throw new DatabaseError('기간별 분석 데이터 조회 중 오류가 발생했습니다');
|
||||
}
|
||||
};
|
||||
});
|
||||
|
||||
/**
|
||||
* 📈 프로젝트별 상세 분석
|
||||
* 프로젝트별 상세 분석
|
||||
*/
|
||||
const getProjectAnalysis = async (req, res) => {
|
||||
const getProjectAnalysis = asyncHandler(async (req, res) => {
|
||||
const { start_date, end_date, project_id } = req.query;
|
||||
|
||||
if (!start_date || !end_date) {
|
||||
throw new ValidationError('start_date와 end_date가 필요합니다', {
|
||||
required: ['start_date', 'end_date'],
|
||||
received: { start_date, end_date }
|
||||
});
|
||||
}
|
||||
|
||||
logger.info('프로젝트별 분석 조회 요청', {
|
||||
start_date,
|
||||
end_date,
|
||||
project_id
|
||||
});
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
try {
|
||||
const { start_date, end_date, project_id } = req.query;
|
||||
|
||||
if (!start_date || !end_date) {
|
||||
return res.status(400).json({
|
||||
success: false,
|
||||
error: 'start_date와 end_date가 필요합니다.'
|
||||
});
|
||||
}
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
let whereConditions = ['dwr.report_date BETWEEN ? AND ?'];
|
||||
let queryParams = [start_date, end_date];
|
||||
|
||||
@@ -273,7 +305,6 @@ const getProjectAnalysis = async (req, res) => {
|
||||
|
||||
const whereClause = whereConditions.join(' AND ');
|
||||
|
||||
// 프로젝트별 통계
|
||||
const projectStatsSql = `
|
||||
SELECT
|
||||
dwr.project_id,
|
||||
@@ -292,40 +323,52 @@ const getProjectAnalysis = async (req, res) => {
|
||||
|
||||
const [projectStats] = await db.query(projectStatsSql, queryParams);
|
||||
|
||||
logger.info('프로젝트별 분석 조회 성공', {
|
||||
start_date,
|
||||
end_date,
|
||||
projectCount: projectStats.length
|
||||
});
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: {
|
||||
projectStats,
|
||||
period: { start_date, end_date }
|
||||
}
|
||||
},
|
||||
message: '프로젝트별 분석 조회 성공'
|
||||
});
|
||||
|
||||
} catch (error) {
|
||||
console.error('프로젝트별 분석 데이터 조회 오류:', error);
|
||||
res.status(500).json({
|
||||
success: false,
|
||||
error: '프로젝트별 분석 데이터 조회 중 오류가 발생했습니다.',
|
||||
detail: error.message
|
||||
logger.error('프로젝트별 분석 조회 실패', {
|
||||
start_date,
|
||||
end_date,
|
||||
error: error.message
|
||||
});
|
||||
throw new DatabaseError('프로젝트별 분석 데이터 조회 중 오류가 발생했습니다');
|
||||
}
|
||||
};
|
||||
});
|
||||
|
||||
/**
|
||||
* 👤 작업자별 상세 분석
|
||||
* 작업자별 상세 분석
|
||||
*/
|
||||
const getWorkerAnalysis = async (req, res) => {
|
||||
const getWorkerAnalysis = asyncHandler(async (req, res) => {
|
||||
const { start_date, end_date, worker_id } = req.query;
|
||||
|
||||
if (!start_date || !end_date) {
|
||||
throw new ValidationError('start_date와 end_date가 필요합니다', {
|
||||
required: ['start_date', 'end_date'],
|
||||
received: { start_date, end_date }
|
||||
});
|
||||
}
|
||||
|
||||
logger.info('작업자별 분석 조회 요청', {
|
||||
start_date,
|
||||
end_date,
|
||||
worker_id
|
||||
});
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
try {
|
||||
const { start_date, end_date, worker_id } = req.query;
|
||||
|
||||
if (!start_date || !end_date) {
|
||||
return res.status(400).json({
|
||||
success: false,
|
||||
error: 'start_date와 end_date가 필요합니다.'
|
||||
});
|
||||
}
|
||||
|
||||
const db = await getDb();
|
||||
|
||||
let whereConditions = ['dwr.report_date BETWEEN ? AND ?'];
|
||||
let queryParams = [start_date, end_date];
|
||||
|
||||
@@ -336,7 +379,6 @@ const getWorkerAnalysis = async (req, res) => {
|
||||
|
||||
const whereClause = whereConditions.join(' AND ');
|
||||
|
||||
// 작업자별 통계
|
||||
const workerStatsSql = `
|
||||
SELECT
|
||||
dwr.worker_id,
|
||||
@@ -355,23 +397,29 @@ const getWorkerAnalysis = async (req, res) => {
|
||||
|
||||
const [workerStats] = await db.query(workerStatsSql, queryParams);
|
||||
|
||||
logger.info('작업자별 분석 조회 성공', {
|
||||
start_date,
|
||||
end_date,
|
||||
workerCount: workerStats.length
|
||||
});
|
||||
|
||||
res.json({
|
||||
success: true,
|
||||
data: {
|
||||
workerStats,
|
||||
period: { start_date, end_date }
|
||||
}
|
||||
},
|
||||
message: '작업자별 분석 조회 성공'
|
||||
});
|
||||
|
||||
} catch (error) {
|
||||
console.error('작업자별 분석 데이터 조회 오류:', error);
|
||||
res.status(500).json({
|
||||
success: false,
|
||||
error: '작업자별 분석 데이터 조회 중 오류가 발생했습니다.',
|
||||
detail: error.message
|
||||
logger.error('작업자별 분석 조회 실패', {
|
||||
start_date,
|
||||
end_date,
|
||||
error: error.message
|
||||
});
|
||||
throw new DatabaseError('작업자별 분석 데이터 조회 중 오류가 발생했습니다');
|
||||
}
|
||||
};
|
||||
});
|
||||
|
||||
module.exports = {
|
||||
getAnalysisFilters,
|
||||
|
||||
Reference in New Issue
Block a user