직고래

기술 블로그

직고래

기술 블로그

Navigate back to the homepage

리팩토링 일기 - REST API

Gyeongjun Kim
September 1st, 2020 · 1 min read

안녕하세요. 셀러리 컴퍼니에서 직고래를 개발하고 있는 코즈입니다.

Why REST?

REST하지 못한 API를 마주할때가 있습니다.

“URI에 행동을 담으면 안돼!” “왜?” “어.. 그건 REST하지 않으니까…”

API를 만들면서, 처음으로 마주하게 되는 고민은 어떤 HTTP method를 사용할지, 그리고 어떤 URI에 Mapping을 할지 입니다. 그리고, 그런 고민들을 해결해주는 것은 ‘RESTful하려면,,’ 입니다.

물론 원래의 REST는, Roy Fielding이 말하는 REST는 그런 뜻은 아니었지만요.

그런 REST API를 사용하고 있는 저에게 REST는, 팀의 혼란을 줄여주는 방법 정도로 다가옵니다. 다른 팀원이 개발한 API를 사용하는 상황에서 URI와 method가 제 각각이면 혼란스러우니까요. 정답이 없는 상황에서 ‘REST스럽게’ 를 정답으로 정한다면, 팀원간의 불필요한 논쟁도 줄어들 것이고, 생산적인 개발을 할 수 있지 않을까요?

“이런 API인데 요청을 어떻게 날려야하지?” “조회니까 GET으로 /articles에 카테고리 param으로 날리삼 그게 제일 RESTful” “ㅇㅇ”

물론 login, logout 같은건 팀원들과의 상의하에 REST 하지 않게 구성하는게 더 나을수도 있다고 생각합니다.


이전 코드

1@RequestMapping(ARTICLE_URI)
2public class ArticleController {
3 public static final String ARTICLE_URI = "/articles";
4
5 ...
6
7 @GetMapping("/trade-state")
8 public ResponseEntity<List<ArticleCardResponse>> showByTradeState(
9 @RequestParam String tradeState, @LoginMember Member loginMember) {
10 List<ArticleCardResponse> responses = articleViewService.showByTradeState(loginMember, tradeState);
11 return ResponseEntity.ok(responses);
12 }

요구 사항은 request로 주어진 trade-state에 맞는 article의 전체 조회입니다. trade-state는 article에 담겨있는 정보입니다. 현재 요청은 header에 로그인을 위한 token과 함께 /articles/trade-state?tradeState={tradeState} 로 날라가고 있습니다.

만약, GET /articles/1/trade-state 이라면, id가 1인 article의 trade-state 조회 정도로 해석할 수 있겠습니다.

GET /articles/trade-state?tradeState={tradeState} 를 해석해보면, article의 모든 trade-state중 tradeState가 ~ 인 trade-state 조회 요청 정도로 해석이 되네요.


바뀐 코드

1@RequestMapping(ARTICLE_URI)
2public class ArticleController {
3 public static final String ARTICLE_URI = "/articles";
4
5 ...
6
7 @GetMapping(params = "tradeState")
8 public ResponseEntity<List<ArticleCardResponse>> showByTradeState(
9 @RequestParam String tradeState, @LoginMember Member loginMember) {
10 List<ArticleCardResponse> responses = articleViewService.showByTradeState(loginMember, tradeState);
11 return ResponseEntity.ok(responses);
12 }

생각해보면 이 코드가 나온 배경은 이미 /articles에 GET 메소드가 매핑 되어있기 때문이었을지도 모르겠습니다. param이 달라도 URI와 method가 동일한 mapping이 2개 이상 존재할땐 Ambiguous mapping으로 bean 생성에 실패하게 됩니다. 하나의 요청을 받을땐 명시하지 않아도 괜찮지만, 그렇지 않은 상황에선 @GetMapping 어노테이션에 params를 사용하여, URI와 method는 동일하고 params만 다른 요청을 처리할 수 있습니다.

More articles from Jikgorae

프로젝트 기술 스택 적용기

들어가며 안녕하세요. 셀러리 컴퍼니의 팀장 터틀입니다. 7월…

July 9th, 2020 · 3 min read

코드 분석 도구 적용기 - 3편, SonarQube 적용하기

안녕하세요. 우아한테크코스 2기, 셀러리 컴퍼니에서 직고래 를 개발하고 있는 스티치입니다. 코드 분석 도구 적용기 - 2편, JaCoCo 적용하기 에 이어서, 이번에는 프로젝트에 정적 코드 분석 도구인 SonarQube…

November 9th, 2020 · 6 min read
© 2020 Jikgorae
Link to $https://github.com/seller-lee