12/20 퍼실리테이터 면담, 12/23 자바 코드 리뷰

12/20 퍼실리테이터 님들과 팀 프로젝트에 대하여 면담을 가졌다.

1. 우리팀은 일어나지 않은 일들에 대해 너무 많은 고민을 한다.
   이런 깊은 고민들은 그 일이 일어난 후에 고쳐도 늦지 않다.
   (ex. 과도한 트래픽, 비용 문제, 유별난 사용자 등등)
   이러한 고민들을 무시하고 개발을 먼저 진행한 후,
   실제 발생한 일에 대해 서비스 개선 작업을 하자.

 

2. 우리팀 프로젝트가 상대적으로 어려운 기술이 없다.

    새로운 시도하는 부분이 필요할수도 있겠다.

    단, 주어진 시간내에 완성하는 역량이 중요하다.

 

3. 다른 사람이 한 일도 잘 이해하고 있어야한다.

    면접장에서 질문받았을때 이러한 부분들을 서포트했다고 말할 수 있어야한다.

    서로의 태스크를 잘 이해하고

    트러블 슈팅을 꼼꼼히 적어놓을 것.

 

4. 기능이 구현 되던 안되던 최대한 빨리 apk가 나오는 것이 좋다.

    사용자 테스트를 빨리 받으면 받을수록 완성도 높은 작품이 나온다.

---

12/23 자바 멘토님께 코드 스타일, 코드 리뷰를 받았다.

스타일 관련

1. 외래키를 사용하지 않음 -> 모두 id를 가지고 호출하는 형태 ?

2. service를 두가지로 나누어서 관리하는데

    조회만 하는 것은 queries,

    그 외 RUD는 service

    원래는 service에서 모두 관리하는데 너무 길어지면 분리한다고 하셨다.

3. 요즘은 DDD 개발과 MicroArchitecture가 떠오르는 추세 ?

4. 테스트 코드 귀찮아서 잘 안짠다 -> 팀에따라 다르다. (열심히 하는 팀에가면 열심히 함)

5. Repository는 꼭 구현체와 interface를 분리할 것

 

여기서 부터는 코드 리뷰

1. entity에는 @Builder 쓰지말고 생성자로만 생성

 

2. image 테이블이 따로 분리되어 있었는데

    entity 안에 넣을 것.

    enum도 크기가 작다면 entity 안에 넣을 것.

 

3. baseEntity(createdAy, modifiedAt) 사용안하면 삭제

 

4. querydsl 굳이 사용하지 않음

    Spring Data JPA의 기본 기능만으로도 대부분 사용할 수 있음 충분히

 

5. null 체크는 entity에서 하는게 좋다.

 

6. responseEntity로 반환하지말고

    response object to json 검색 또는

    뷰 객체 자체로 반환하기

 

7.  yml로 value 값들 따로 관리하기

 

8. 예외 처리, 에러 처리 매우 중요함

 

9. 조인쓰면 조회가 느릴수밖에 없다.

    조인을 아예 안쓰는 것을 추천 (외래키를 사용하지않아도 됨)

    id로 그냥 가져오면 된다.

 

10. 코드 최대한 일급 컬렉션 등 고급 기술로 간결화 할 것

 

11. 리스트는 isEmpty()가 아니라 CollectionUtils.isNullOrEmpty 등을 써야한다.

 

12. initCount를 썼는데 resetCount로 바꿔라