코드 리뷰
공동 작업과 분업
- 과도한 분업은 좋지 않음
- 과도한 페어 프로그래밍은 효율적이지 않음
- 페어 프로그래밍인 경우 표시를 하는 것이 좋음
코딩 스타일
- 불필요한 개행 제거
- 개행이 필요한 경우 추가
- 네이밍 컨벤션
기록 남기기
- wiki에 기록을 잘 남기자
- 틈틈히 기록을 잘 하자
Git
PR과 Commit 크기
- Commit 메시지 작성 때 확인을 해야함
- 커밋 메시지와 실제 코드가 같은 내용인가
기본적인 명령어
- push / pull / commit / add
- reset /stash
- rebase
- cherry-pick
불필요한 파일 제거
- 팀원의 개발 환경은 다를 수 있음
- 따라서, 불필요한 파일은 제거하자
- .idea/.DS_Store 파일은 뭘까?
객체 역할 정의
- 비슷한 개념에 접근하는 클래스들이 존재한다면, 해당 객체에서 무슨 역할을 기대하는지 기준을 정할 필요가 있음
Mutable 변수
가시성 범위
- 외부에서 접근하는 것을 최대한 막는 것이 좋지 않을까