Skip to content

Latest commit

 

History

History
40 lines (29 loc) · 1.87 KB

reviewee-guide.md

File metadata and controls

40 lines (29 loc) · 1.87 KB

image

리뷰이 미션 진행 가이드

시간확보

  • 친구들과의 약속 보름 뒤로 미루기 (오랜만에 봐야 더 정겹고 할 말이 많다.)
  • 게임하지 않기 (오랜만에 해야 더 재밌다.)
  • 야근하면서 미션 구현하기 (퇴근하고 집가면 하기 싫다.)

매일 1~2시간씩 미션 진행

  • 일주일에 최소 3회 이상 코드 리뷰 요청을 보내 코드 리뷰 받기
  • 한 번에 모두 구현하기보다 매일 일정한 시간 투자하는 것이 정말 중요함
    • 밀리면 밀릴수록 시작하기 부담스러워진다.

리뷰어의 의견을 적극 수용하기

  • 요구 사항 규모보다 극단적인 리팩터링을 요구할 수 있다.
  • 거부감이 들 수 있지만 일단 적용해 보고, 적용하기 전과 후의 코드를 분석해 본다.
  • 자신이 가진 것을 비울 때 가장 많은 것을 배울 수 있다.

그렇지만 리뷰어의 의견에 적극 반대도 해보기

  • 리뷰어는 선생님이 아니다. 같은 업계에 종사하는 동료 개발자.
  • 프로그래밍 설계와 구현에 정답은 없다.
  • 서로 정답을 찾기보다, 요구 사항에 적합한 최선의 설계와 구현 코드를 찾기 위해 노력하고 토론하자.

미션 내에서 무조건 지켜야할 객체 지향 생활 체조 원칙

  • 한 메서드에 오직 한 단계의 들여쓰기(indent)만 한다.
    • for 문 내부에서 if 문이 포함되는 순간 2 indent 가 된다.
  • else 예약어를 쓰지 않는다.
  • 모든 원시 값과 문자열을 포장(VO)한다.
  • 한 줄에 점을 하나만 찍는다.
  • 줄여 쓰지 않는다(축약 금지).
  • 모든 엔티티를 작게 유지한다.
  • 3개 이상의 인스턴스 변수를 가진 클래스를 쓰지 않는다.
  • 일급 컬렉션을 쓴다.
  • 무의미한 getter/setter/프로퍼티를 쓰지 않는다.