We.TIL 26 : 코드 줄이기

서론

프로젝트 기간 동안 멘토분들에게 PR(Pull Request)에서 자주 받은 피드백 중 하나는 CleanCode(깔끔한 코드)로 바꾸라는 요청이였다.

깔끔한 코드, 가독성 높은 코드란 어떤 것이라는 내 질문에 멘토님들은 이렇게 명료하게 답해주셨다.

"짧으면서 알아보기 쉬운 코드"

다음 코드는 미숙하지만 나름대로 프로젝트 관련된 상품페이지 엔드포인트의 코드를 나름대로 줄여본 것이다.

원본

기능구현에만 촛점을 맞추고 코드를 짜다보니 (쓸데없는)변수도 과다하게 많고, 쓸데없는for loof와 정참조를 써서 코드 길이가 꽤나 길어졌다. 이 코드를 다음과 같이 줄였다.

수정본

다음의 사항을 적용해 코드를 간결하게 바꿔봤다.

1. 정참조를 역참조로 변경
2. 쓸데없는 변수 제거
3. 쓸데없는 for loof를 지우고 인덱스값을 넣어서 리스트를 바로 생성

수정본2

1. 쿼리스트링을 사용해 두 개의 body를 하나로 만듬
   (이전에는 if문을 이용해 구분)
2. 카테고리명도 쿼리스트링을 활용해 가변적으로 입력에 따라 바뀌게 만듬
3. RestfulAPI 원칙에 따라 View이름 수정