제 Spring 컨트롤러 클래스들이 지저분해서 고민이에요.

Question

제 컨트롤러 클래스 속 @GetMapping, @PostMapping 메서드들을 볼 때마다 지저분해서 고민입니다. 예를 들어, 페이지 정보를 받아서 게시글들을 보여주는 컨트롤러가 있다고 하면, @GetMapping("/posts") public String getPosts(@RequestParam("page") String requestPage, Model model) { // 엉터리 입력값(ex. "+$%", "abc" 등)을 검사하고, 제대로 된 페이지 생성 후 반환 Page page = Page.validateAndCreate(requestPage) // DB에서 Post들을 가져옴. List posts = PostService.getPostList(page); // Model에 Post들을 담아 View로 전달. model.addAttribute("posts", posts); // 전달할 View 경로 지정 return "/posts/list" }제 컨트롤러 구조가 대부분 위와 비슷한 구조인데,여기에 Page 뿐 아니라, 정렬조건 Sort, 꼭 들어가야할 키워드 조건 Keywords 등 이 추가가 되는 순간 지저분해집니다.@GetMapping("/posts") public String getPosts(@RequestParam("page") String requestPage, @RequestParam("sort") Sort requestSort, @RequestParam("keywords") Keywords requestKeywords ... 등등 Model model) { // 오만가지 매개변수 전처리 및 검증 로직 Page page = Page.validateAndCreate(requestPage); Sort sort = Sort.validateAndCreate(requestSort); .... 등등 // DB에서 Post들을 가져옴. List posts = PostService.getPostList(page, sort, keyword, ... 등등); // Model에 Post들을 담아 View로 전달. model.addAttribute("posts", posts); model.addAttribute("page", page); model.addAttribute("sort", sort); .... 등등 // 전달할 View 경로 지정 return "/posts/list" } 특히 지저분한 부분이 전처리 부분과 model에 데이터를 담는 부분들인데, 위 예시는 설명을 위한 예시라 깔끔하게 적어서 괜찮은데, 실제 제 토이프로젝트의 코드를 보면 너무 지저분해서 제 코드를 읽는 분이 화가 많이 날 것 같습니다... ㅋㅋㅋ 어떻게 해결하는 게 좋을까요?제 지금 생각으로는 저 전처리하는 부분들을 모두 ArgumentResolver에서 싹 처리해버릴까 생각 중인데 좋은 생각인지 궁금합니다.

전뚱이인데요 · Answer

컨트롤러에서 모든걸 처리하지 마세요컨트롤러는 단순히 클라이언트 측에서 데이터를 받고 검증 하고 값을 반환하는것으로 충분합니다.복잡한 로직은 서비스 레이어를 두시구요단순히 검증값이 복잡한거라면 Valid 어노테이션을 통해서 파라미터 값 어느정도 검증 가능하구요또한 파라미터로 값을 받을때 객체로 받아와서 해당 객체에서 검증 로직 작성해도 괜찮아요

뭉즈 · Answer

저같은 경우에는 Valid 어노테이션 사용하고 공통 검증 로직은 AOP로 뺐습니다.그리고 Response관련 DTO를 만들어서 반환하는 값들을 모두 받아서 한번에 페이지로 전송했습니다.

wharup · Answer

ComposeMethod 패턴을 이용해서 정리하는게 좋을 것 같습니다.긴 코드를 같은 레벨의 작은 메소드로 구성하는 방식입니다.간단하게 적용해보면 밑에 같은 느낌입니다.ComposeMethod 패턴을 이용해서 정리하는게 좋을 것 같습니다. 긴 코드를 같은 레벨의 작은 메소드로 구성하는 방식입니다. https://e-you.tistory.com/456 간단하게 적용해보면 밑에 같은 느낌입니다. public String getPosts(@RequestParam("page") String requestPage, @RequestParam("sort") Sort requestSort, @RequestParam("keywords") Keywords requestKeywords ... 등등 Model model) { validateRequest(requestPage, requestSort, requestKeywords, ...); List<Post> posts = PostService.getPostList(page, sort, keyword, ... 등등); setViewContent(posts, page, sort); return "/posts/list" }

제 Spring 컨트롤러 클래스들이 지저분해서 고민이에요.

3개의 답변

Q&A 목록