프로그래머스 Birdview 과제 테스트 피드백

#1

• 좋은 부분
  rest framework를 사용해서 코드를 깔끔하게 작성한 점이 좋습니다.
  모델을 잘 활용했습니다.
• 아쉬운 부분
  Product 모델의 imageId나 monthlySales처럼 파이썬 스타일 가이드인 PEP8을 지키지 못한 부분이 있습니다.
  모델 관계가 없는 것이 아쉽습니다.
  rest framework의 APIView대신 ListAPIView와 RetrieveAPIView를 쓰는게 더 좋았을 것 같습니다.
• 총평
  전체적으로 코드는 깔끔하지만 모델링이 아쉽습니다. 확장성이나 정규화를 위해 Product와 Ingredient를 다대다 모델로 설정하고 카테고리를 테이블로 구성했으면 더 좋았을 것 같습니다.

#2

• 좋은 부분
  테스트 코드를 작성하였습니다.
  바퀴를 재발명하지 않고, API 작성을 위해 외부 라이브러리인 DRF를 적절히 사용한 것이 좋았습니다.
• 아쉬운 부분
  시리얼라이저의 상속이 굳이 필요한 부분인지 의문이 듭니다. 더 나은 방법을 고민해 볼 필요가 있습니다.
  필드명이 파이썬 PEP-8을 지키지 않고 있습니다.
  모델 프로퍼티 사용을 적절하게 활용했으면 굳이 불필요하게 복잡한 방식으로 처리를 할 필요가 없었을텐데 하는 아쉬움이 있습니다. 프로퍼티를 사용해서 가독성과 유지보수성, 재사용성을 높일 수 있지 않았을까 싶네요.
  빈 목록일 때 응답 상태 값이 204인게 부적절해보입니다. 상태값에 대한 학습이 더 필요해보입니다.
• 총평
  역할에 맞게 소스를 잘 분리해서 작성하였습니다.
  Ingredient 모델과 Product 모델의 다대다 관계 설정을 해서 정규화를 해서 모델링을 했으면 더 나았을거 같습니다. DB에 대한 학습을 권유드립니다.

후기

• 생각했던 것보다 더 상세한 피드백이 돌아와서 많이 놀랐다. 코드의 중복을 무작정 피하는 것이 그렇게 의미는 없는 것 같다.
• 파이썬 스타일 가이드 PEP8을 따라서 좀 더 체계적으로 개발을 하는 습관을 가져야 할 것 같다.