From 680dc5f6a8ee92d0bc72aa1fbee3c4acc297bf5f Mon Sep 17 00:00:00 2001 From: Bae Hyeonseo Date: Sat, 18 Nov 2023 17:18:22 +0900 Subject: [PATCH 1/3] =?UTF-8?q?[refactor]:Main=20#117=20roadmapManageServi?= =?UTF-8?q?ce=20=EB=A9=94=EC=86=8C=EB=93=9C=20=EB=B6=84=EB=A6=AC?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../article/controller/ArticleController.java | 2 +- .../roadmap/service/RoadmapManageService.java | 26 ++++++++++--------- 2 files changed, 15 insertions(+), 13 deletions(-) diff --git a/MainService/src/main/java/com/kusitms/mainservice/domain/article/controller/ArticleController.java b/MainService/src/main/java/com/kusitms/mainservice/domain/article/controller/ArticleController.java index 5eae8207..b3f33390 100644 --- a/MainService/src/main/java/com/kusitms/mainservice/domain/article/controller/ArticleController.java +++ b/MainService/src/main/java/com/kusitms/mainservice/domain/article/controller/ArticleController.java @@ -18,7 +18,7 @@ public class ArticleController { private final ArticleService articleService; @GetMapping - public ResponseEntity> getAritcleList(){ + public ResponseEntity> getArticleList(){ final List articleResponseDtoList = articleService.getArticleList(); return SuccessResponse.ok(articleResponseDtoList); } diff --git a/MainService/src/main/java/com/kusitms/mainservice/domain/roadmap/service/RoadmapManageService.java b/MainService/src/main/java/com/kusitms/mainservice/domain/roadmap/service/RoadmapManageService.java index 0bfa498a..d8da6779 100644 --- a/MainService/src/main/java/com/kusitms/mainservice/domain/roadmap/service/RoadmapManageService.java +++ b/MainService/src/main/java/com/kusitms/mainservice/domain/roadmap/service/RoadmapManageService.java @@ -37,22 +37,25 @@ public void createSharingRoadmap(Long userId, RoadmapSharingRequestDto roadmapSh RoadmapType roadmapType = getEnumRoadmapTypeFromStringRoadmapType(roadmapSharingRequestDto.getRoadmapType()); Roadmap createdRoadmap = Roadmap.createRoadmap(roadmapSharingRequestDto, roadmapType, user); saveRoadmap(createdRoadmap); + saveRoadmapSpace(roadmapSharingRequestDto,createdRoadmap); + } + private void saveRoadmapSpace(RoadmapSharingRequestDto roadmapSharingRequestDto,Roadmap createdRoadmap){ for (int i = 0; i < roadmapSharingRequestDto.getSteps().size(); i++) { StepDto stepDto = roadmapSharingRequestDto.getSteps().get(i); RoadmapSpace createdRoadmapSpace = RoadmapSpace.createRoadmapSpace(stepDto, createdRoadmap, i+1); saveRoadmapSpace(createdRoadmapSpace); - for(Long templateId : stepDto.getTemplateIdList()){ - Template template = getTemplateById(templateId); - RoadmapTemplate createRoadmapTemplate = RoadmapTemplate.createRoadmapTemplate(createdRoadmapSpace,template); - roadmapTemplateRepository.save(createRoadmapTemplate); - } - // 반복문 => step에 해당하는 template List 수 만큼 - // template 객체 생성 - // template 저장 - // roadmap_template 관계 만들기 => roadmap, template - // roadmap_template 저장 + saveRoadmapTemplate(stepDto, createdRoadmapSpace); } - + } + private void saveRoadmapTemplate(StepDto stepDto, RoadmapSpace createdRoadmapSpace){ + for(Long templateId : stepDto.getTemplateIdList()){ + Template template = getTemplateById(templateId); + RoadmapTemplate createRoadmapTemplate = RoadmapTemplate.createRoadmapTemplate(createdRoadmapSpace,template); + saveRoadmapTemplate(createRoadmapTemplate); + } + } + private void saveRoadmapTemplate(RoadmapTemplate createRoadmapTemplate){ + roadmapTemplateRepository.save(createRoadmapTemplate); } private Template getTemplateById(Long templateId){ Optional