도메인 네이밍 어떻게 하십니까?

Question

DB에서 select해온 List가 있습니다. 이 데이터는 1분에 한번씩 전력센서에서 수집한 데이터 입니다.1시간 데이터 수집을 하면 60개의 데이터가 수집 됩니다. 이 데이터를 15분마다 평균 또는 Sum을 해서 를 4개로 요약하는 기능입니다. 질문은 이 기능이 Service에 들어갈만한 기능인지 따로 도메인을 만들어 주는 것이 좋을지 입니다.지금 PanelBoardChartService는 Repository와 디펜던시가 없어서 이 로직을 PanelBoardChartService 라는 Service에 굳이 넣지 않아도 좋을 것 같다는게 제 생각입니다. calculate()와 reduce()라는 책임을 특정 클래스로 할당 해야 할지 그리고 이름은 어떻게 지으시는지 궁금합니다.테스트class PanelBoardChartServiceTest { @Test void panelBoardChartService_calculate() { LocalDateTime startDatetime = LocalDateTime.of(2024, 1, 1, 0, 0, 0); LocalDateTime endDatetime = startDatetime.plusHours(1); PanelBoardChartService panelBoardChartService = new PanelBoardChartService(); List monitorings = PanelBoardMonitoringDtoFixture.panelBoardMonitoringDtos(startDatetime); System.out.println(monitorings.size()); // 16개 List calculated = panelBoardChartService.calculate(monitorings, startDatetime, endDatetime); assertEquals(2, calculated.size()); // 15개마다 1개로 요약되므로 16개이므로 총 2개 } }비즈니스 로직@Service @RequiredArgsConstructor public class PanelBoardChartService { public List calculate(List records, LocalDateTime startDatetime, LocalDateTime endDatetime) { // to 1일치/15개 from 1일치 List rows = new ArrayList<>(); // timestamp마다로 변경 long[] timestamps = TimeSlotCalculator.makeTimestamps(startDatetime, endDatetime, 15); List[][] powerDataSum = new List[24][4]; // 24시간 1시간에 4개 for (PanelBoardMonitoringDto record : records) { // 이 부분이 가장 중요한 부분으로 index를 구해서 배열의 특정 위치로 분류 합니다. TimeIndices indices = TimeIndicesCalculator.calculateTimeIndices(record.getTimestamp(), startDatetime); if(powerDataSum[indices.getHourIndex()][indices.getQuarterIndex()] == null){ powerDataSum[indices.getHourIndex()][indices.getQuarterIndex()] = new ArrayList<>(); } powerDataSum[indices.getHourIndex()][indices.getQuarterIndex()].add(record); } for (int i = 0; i < powerDataSum.length; i++) { for (int j = 0; j < powerDataSum[i].length; j++) { List l1 = powerDataSum[i][j]; if(l1 != null){ rows.add(reduce(l1)); } } } return rows; } private PanelBoardChartRow reduce(List panelBoardMonitoringDtos) { int size = panelBoardMonitoringDtos.size(); float current = 0.0f; float voltageLine = 0.0f; float powerFactor = 0.0f; float totalActivePower = 0.0f; float totalActiveEnergy = 0.0f; for (PanelBoardMonitoringDto panelBoardMonitoringDto : panelBoardMonitoringDtos) { current += panelBoardMonitoringDto.getTotalLineCurrent(); voltageLine += panelBoardMonitoringDto.getLtlVoltageL1l2(); powerFactor += panelBoardMonitoringDto.getTotalPowerFactor(); totalActivePower += panelBoardMonitoringDto.getTotalActivePower(); } // totalActiveEnergy는 latest - oldest를 하면 되는데 1개 있을때는 구할 수 없으므로 totalActivePower 평균으로 처리 합니다. return new PanelBoardChartRow(current / size, voltageLine / size, powerFactor / size, totalActivePower / size, totalActivePower / size); } }

Iswn · Answer

둘중에 고른다면 calculate가 맞지 않을까요? 단순히 줄이는 것 보다 복잡한 계산이 수반된다는 의미를 포함해야 할것같습니다.만일 업무적 용어가 별도로 있다면 그걸따라야 할것같네요. calculate나 reduce는 업무적 처리를 표현하기보단처리 그 자체에 목적을 둔 단어로 보이거든요

삽살개발 · Answer

죄송합니다 도파미네이션으로 보고 들어왔어요이름은 그냥 읽기쉽고 타이핑 하기 쉬운게 1순위 나머진 주석과 문서가...도 쉽지 않긴 하죠에이아이 추천도 좋을거 같네요

도메인 네이밍 어떻게 하십니까?

2개의 답변

Q&A 목록