9월 11일 백엔드 컨벤션 맞추기 회의 결과

[tjdtls690]

1. 각 Controller와 Service들을 Query(R), Command(CUD) 특성을 가진 클래스로 분리한다.

   • 일관성과 응집도를 위해 코드의 양과 상관없이 무조건 분리하는 것으로 컨벤션 결정

2. S3에 이미지 업로드하는 것은 백엔드를 거쳐서 전송하자.

   • 다른 조로부터 얻은 정보
     • 현재 Client에서 바로 S3에 접근하면 CORS 에러(Origin error)가 뜬다.
     • 그래서 Client에 대한 Origin을 S3에서 풀어달라고 코치에 요청
     • 토미 왈 : 이러면 보안상에 문제가 있을 수 있다. 백엔드를 거쳐서 이미지 파일에 대한 검증을 하게끔 구현하는 것이 좋지 않겠나?

3. 메서드 순서

   • Controller
     • private 메서드 분리가 많이 일어나지 않을 것으로 예상한다.
     • 또한 어떤 API 요청들을 받는지가 더 주요한 부분이기에 API 메서드와 private 메서드들을 완벽히 분리한다.
     • 결정된 컨벤션
       • public(API) 메서드들을 전부 위에 두고, private 메서드들은 전부 밑에 둔다.
   • Service
     • Service 로직은 비즈니스 로직으로써, 코드의 흐름 파악이 더 중요하다고 판단한다.
     • 그래서 private 메서드들이 자신을 호출한 public 메서드 바로 밑에 둔다.
     • 결정된 컨벤션
       • private 메서드들이 자신을 호출한 public 메서드 바로 밑에 둔다.
       • 단, 자신을 호출하는 public 메서드가 2개 이상일 시, 공용 메서드라는 의미로 순서 상관없이 가장 아래에 둔다.