java에서 if문 분기를 더 좋은 방향으로 설계 할 수 있을까요??

Question

현재 저희 조회 관련 API 일부는 및 Controller은 다음과 같습니다[GET] api/study-group?sortby=latest [GET] api/study-group?sortby=star [GET] api/study-group?sortby=level@RestController @RequiredArgsConstructor @RequestMapping("/api/study-group") public class StudyGroupController { private final StudyGroupService studyGroupService; @GetMapping public ResponseEntity> findStudyGroup( @RequestParam(name = "sortby", defaultValue = "createdat") String sortBy) { studyGroupService.findAll(sortBy); ...... } }해당 Controller에 대한 로직을 처리하는데, 정렬 방식에 대한 분기 처리를 하는데 어떤 방식이 좋을 지 의문입니다.첫 번째 방식은 if문을 통한 분기처리입니다@Service @RequiredArgsConstructor public class StudyGroupService { public List findAll(String sortBy) { if(sortBy.equals("star") { searchStudyOrderByStar()l }else if(sortBy.equals("...")) { .... }else { ... } } private List sort(Function function) { return studyGroupRepository .findAll() .stream() .sorted(Comparator.comparing(function, Comparator.reverseOrder()) .thenComparing(StudyGroup::getSeason, Comparator.reverseOrder())) .map(studyGroupMapper::toResponseDTO) .collect(Collectors.toList()); } private List searchStudyOrderByCreatedAt() { return sort(studyJournalService::searchLatestJournalCreatedAt); } private List searchStudyOrderByStar() return sort(studyGroup -> studyGroup.getJournals().size()); } private List searchStudyOrderByLike() { return sort(StudyGroup::getLike); } }물론, String 대신 Enum을 쓰는 것도 생각하고 있습니다. 다만, 변경한다고 하더라도, if문으로 분기를 설정하는 것은 변함이 없을 것 같습니다.다르게 생각한 방식으로는 Map을 사용하는 방식입니다. @Service @RequiredArgsConstructor public class StudyGroupService { private final Map sortMap; public List findAll(String sortBy) { sortMap.get(sortBy); .... } }여러분의 생각에는 어떤 방식이 제일 나아보일까요?아니면 위에 명시한 것 보다 더 좋은 방법이 있을까요??실무에서는 위와 같은 케이스에서 어떻게 처리를 하시는 지 궁금합니다!좋은 의견 있으시면 피드백 주시면 감사하겠습니다!

루핑투핑 · Accepted Answer

정답을 찾은 것 같습니다... @GetMapping(params = "sortby=latest") public ResponseEntity> findStudyGroupSortByLatest() { //logic for sort by latest } @GetMapping(params = "sortby=star") public ResponseEntity> findStudyGroupSortByStar() { //logic for sort by star }Controller 쪽에서 해결하면 되는 문제였네요...

yjj8353 · Answer

switch 쓰는게 낫지 않을까요?

그럴수도있지 · Answer

Enum 타입으로 정의를 해두고, 서비스 및 repository layer 전부 타입을 파라미터로 사용하면 되지 않을까요?조회할 때도 해당 타입을 파라미터로 받아서 조건절로 넣어주고요.

java에서 if문 분기를 더 좋은 방향으로 설계 할 수 있을까요??

3개의 답변

#refactoring 목록