review: customer 일부 및 restaurant 리뷰

[wooyounggggg]

No description provided.

[wooyounggggg]

2차 코드리뷰입니다!

customer와 restaurant에 대해 리뷰했고, 다른 곳도 비슷해서 코드 리뷰는 2차로 마무리하면 될 것 같네요~

제가 리뷰하면서 느낀 점들을 아래에 정리해보았습니다.

표준화 된 코드 작성 및 코드 품질 관리

성능이나 인프라 및 시스템 아키텍처도 중요하지만, 서비스 코드 품질에도 심혈을 기울여야 하고, 가장 기본적인 부분이 되어야한다고 생각합니다. (f-lab 프로젝트 특성상 코드 외의 요소들에 좀 더 많은 시간을 들이셨으리라 생각됩니다)

exception을 던지는 것, custom exception을 정의하는 것, restful api 설계, 사용하지 않는 코드 제거, 실수 여지가 있는 코드 방어, 의미있는 메서드 및 변수 네이밍 등 다양한 방법을 찾아보시고, 이들을 서비스 코드 작성의 표준으로 만드신다면 전반적인 코드 퀄리티를 향상할 수 있을 것 같습니다.

객체 지향 설계

객체들의 책임 관계가 모호한 경우가 많았습니다.

이와 관련해서는 아래의 객체지향 서적들을 추천드립니다.

• 개발자가 반드시 정복해야 할 객체 지향과 디자인 패턴, 최범균
• 객체 지향의 사실과 오해, 조영호
• 오브젝트, 조영호

이후 더 시간이 지나서는 DDD도 읽어보시면 좋을 것 같습니다~

그리고 현재는 controller, service, repository 정도로 크게 레이어가 나누어져 있는데 infra layer를 하나 더 추가해서 controller / service / infrastructure / repository로 구성해보시는 것도 좋을 것 같습니다.