컨트롤러 서비스 관심사 분리

Question

컨트롤러// 제품 검색(관리자)@GetMapping("/manager/product/search/session/check")public ProductManageListSearchResultResponseDto getProductManageListSearchResultResponseDto(@RequestParam("productName") String productName) { List productManageListItemDtoList = productService.getProductManageListSearchResult(productName); boolean searchRrsultEmpty =productManageListItemDtoList.isEmpty(); return ProductManageListSearchResultResponseDto.builder() .serachResultEmpty(searchRrsultEmpty) .productManageListItemDtoList(productManageListItemDtoList) .build(); }서비스// 관리자 페이지 제품 검색 리스트 가져오기(관리자)public List getProductManageListSearchResult(String productName) { List productManageListItemDtoLsit =productRepositoryByJpa.findProductManageListItemDtoByProductName(productName);return productManageListItemDtoLsit;}제품리스트에서 검색 결과를 가져오는 코드인데요 지금 현재 코드는 컨트롤러에서 응답DTO를 최종적으로 조립하게 되는 상황입니다(비어있는지 확인하는건 ui화면 때문에 편의상 넣는 값이기 때문에 컨트롤러에서 확인하고 값을 넣어줬습니다)아예 서비스에서 ProductManageListSearchResultResponseDto(최종응답DTO) 를 조립해서 반환해도 괜찮을까요?책임 분리를 어디까지 해야하는지 기준이 잘 잡히지 않아서 질문 드립니다...아예 응답 dto를 서비스에서 만들어서 컨트롤러는 반환만 하는 형식으로 가는게 맞는걸까요?만약에 그게 맞다면 @GetMapping("/manager/product/delete/session/check")public Map deleteProduct(@RequestParam("productId") long productId){boolean success = productService.deleteProduct(productId);Map resultMap = new HashMap<>();resultMap.put("success", success);return resultMap;}이렇게 맵을 사용하는 경우에는 맵도 서비스에서만들어서 컨트롤러에서 반환만 하는게 맞는건가요?

하루히즘 · Answer

컨트롤러는 그냥 리턴만 해도 되고 객체 만들어서 자기가 직접 반환해도 되고 둘 다 맞는 방법인거같아요근데 저는 보통 컨트롤러에서 한번 더 조립하려고 하면 귀찮아서 서비스에서 모델 만들어서 던져주는 식으로 하긴 합니다엑셀파일 다운로드같은 건 컨트롤러에 일이 좀 가긴 하지만여

장독깨기 · Answer

프로그래밍이란게 정해진 규칙에 정답이 있고 그걸 맞춰서 코딩하고 이런게 아닙니다.이 생각 자체를 버리세요. 맞다 안맞다 이런거 없습니다.어떤식으로 하든 그냥 일관성만 좀 지키시구요, 진도 나가세요. : )

컨트롤러 서비스 관심사 분리

2개의 답변

#service 목록