search

DDD-Community / undertheriver-sgsg-backend

✒ 사각사각 백엔드
9 stars 0 forks source link

feat: 메모 즐겨찾기 상태 변경 API #98

Closed plzprayme closed 3 years ago

plzprayme commented 3 years ago

84

변경 사항

기타

hongbin-dev commented 3 years ago

추가로 즐겨찾기를 put으로 한 이유가 있을까요? 즐겨찾기 등록 API 즐겨찾기 해제 API로도 나눌 수 있을 것 같은데요

plzprayme commented 3 years ago

즐겨찾기를 put으로 설계한 특별한 이유는 없습니다. 그냥 API를 하나로 사용하면 좋겠어서 그렇게 작업을 했습니다.

이번 작업 때 주로 리뷰해주신 "메세지를 보낸다" 는 관점에서 API를 분리하는게 더 자연스러울까요?

hongbin-dev commented 3 years ago

저도 구현방법은 선택이라고 생각하는데 어떤이유로 스위칭하는방식으로 선택하셨는지 이유가 궁금했습니다~

plzprayme commented 3 years ago

API 분리 했습니다.

분리의 장점을 생각해봤는데요

  1. URL로 의도를 더 명확하게 표현할 수 있다.
  2. request body가 필요 없다.
  3. 엔티티에 메세지를 보낼 수 있다.

정도가 좋은 것 같았습니다.

반면에 단점은

  1. 코드가 조금 반복된다

는 점이 있겠네요.

리뷰 부탁드립니다!!