게시판 리스트를 보여주는 매서드가 지저분해 보이는데, 어떻게 리팩토링하면 좋을까요?

Question

질문글을 올리는 게시판을 만들고 있고, Thymeleaf를 사용해서 View로 model을 넘겨주며 화면을 만드는 방식으로 프로젝트를 하고 있습니다. 저는 Controller - Facade - Service - Repository 구조를 사용하고 있고, Controller에서는 오직 model을 적절한 View로 보내주는 라우팅의 역할만,Facade에서는 Service가 반환하는 데이터를 조합해서 model을 조립하는 역할을 수행하도록 했습니다. 아래는 QuestionFacade 속 게시글 리스트를 조회하는 함수인데 제가 문득 보기에 조금 지저분해보이고,코드를 처음 보는 다른 사람들이 보기에 읽기 어렵지 않을까, 너무 주저리주저리 흐름대로 작성한 것은 아닐까 판단이 잘 안섭니다.// 질문글 리스트 조회 public QuestionListViewModel getQuestions( Long requestedPageNumber, // 사용자가 요청한 페이지 번호 (ex. 1page 요청) SortType sortType, // 사용자가 원하는 정렬타입 (ex. 날짜순, 좋아요 순, 조회순) FilterRequest filter) // 필터링 정보 (ex. java에 대한 글만, spring에 대한 글만) { // DB에서 필터를 적용한 질문글 총 개수를 가져옴. // 페이지 정보를 생성하는 데 사용. Long filteredQuestionCount = questionService.getQuestionsCount(filter); // 사용자가 요청한 페이지 번호, 질문글 개수를 사용하여 페이지 정보를 생성. PageDetailDTO pageInfo = PageDetailDTO.create( requestedPageNumber, filteredQuestionCount, DEFAULT_QUESTION_COUNT_ON_SINGLE_PAGE); // 페이지 정보를 사용해서 한 페이지에 해당하는 게시글만 DB에서 조회. List pagedQuestions = questionService.getQuestions(pageInfo, sortType, filter); // 페이징된 질문글과 관련된 태그를 쿼리 한번으로 DB에서 모두 가져옴. // {질문글 ID1 : [ 태그1, 태그2, 태그3], // 질문글 ID2 : [ 태그3, 태그4] } // 의 형식으로 데이터를 보여줌. Map> questionTagsMap = tagService.getTags(QuestionDetailDTO.getIdList(pagedQuestions)); // 질문글 정보, 태그 정보를 모두 가져와서 조립. List questionSummaries = QuestionSummaryDTO.from(pagedQuestions, questionTagsMap); // 질문글 리스트의 사이드 바에서 보여줄, 카테고리 정보를 모두 가져옴. List sidebarCategories = categoryService.getAllCategories(); // 모든 데이터를 사용하여 viewModel 생성 후 반환 return QuestionListViewModel.from(pageInfo, sidebarCategories, questionSummaries); }개선할 부분이 있을 것 같은데, 전혀 모르겠습니다. 혹시 개선할 부분이 있다면 편하게 말씀해주시면 감사하겠습니다.

yeori · Accepted Answer

로직이 복잡하지는 않은데 전체적으로 변수명이나 클래스 이름이 긴 듯합니다.아래와 같이 변수명을 간결하게 지으면 보기 편할거 같고..public QuestionVM getQuestions(Long pageNum, SortType sort, FilterRequest filter) { // 필터를 적용한 질문 개수 조회 Long total = questionService.countQuestions(filter); // 페이지 정보 생성 Pagination pgn= Pagination.create(pageNum, total);DEFAULT_QUESTION_COUNT_ON_SINGLE_PAGE 은 생김새가 페이지당 게시물 갯수가 아닌가 싶은데, 이런 상수는 클래스 Pagination 안으로 넣고 페이지 갯수를 따로 지정하는 경우 Paginaton.create(pagenum, total, itemCnt)와 같이 별도의 메소드를 준비하는게 좋아보임.게시글 가져온 후에 태그를 다시 조회하는데.... List postings = questionService.getQuestions(pgn, sort, filter); // 질문 ID별 태그 맵 조회 Map> tagMap = tagService.getTags(postings);가급적이면 게시물을 가져올 때 태그도 함께 가져오는게 좋습니다. join으로...이게 안된다면 디비 설계를 살펴보는게 좋고, mybatis든 hibernate든 이거는 다 됩니다.게시물 따로, 태그 따로 가져오니 이걸 조립하려고 summary 무시기가 다시 필요해집니다.사이드바에 보여질 카테고리를 sidebarCategories라고 이름을 지었는데, 카테고리 종류가 여러개가 아니라면 그냥 categories, cates 와 같이 간단하게 이름 짓는게 좋지 않을까 싶습니다.(개인 취향)왜냐하면 카테고리를 위에 붙이든 옆에 붙이든, 이거는 view 의 관점이고 데이터 자체는 그냥 카테고리니까요.마찬가지로 코드 곳곳에서 question이라는 표현을 쓰시는데, 질문은 글의 속성입니다. 그니까, 이 글이 질문인지 공지인지, 답글인지 게시물의 속성이라는 얘기... 뭐 이것도 개인취향인데 저같으면 Posting으로 통일할겁니다. Posting.type = 'question' 으로...Controller와 Service 사이에 Facade 계층을 끼워넣는다면 Service들이 dto가 아닌 entity를 반환하게 하는 것을 고려해볼 필요가 있습니다. 모든 Controller가 반드시 Facade를 통해서 요청 응답을 처리한다면 Service의 반환 타입을 일관성 있게 통일시키는게 좋습니다.

동도인 · Answer

일단 주석 만 봐서는 전부 기존 데이터를 활용해서 새로운 데이터를 다시 가져오는데 이경우 차라리 JOIN 이나 프로시져 등을 사용해서 코드를 극단적으로 단축 시킬수 있을거 같습니다.

게시판 리스트를 보여주는 매서드가 지저분해 보이는데, 어떻게 리팩토링하면 좋을까요?

2개의 답변

Q&A 목록