🔥 인사이트 한줄 소개!

서치하다가 우연히 본 글인데 LG유플러스 아이들나라 팀이 효과적인 코드 리뷰를 위해 우테코에서 아이디어를 얻어 “미션 스터디”를 도입한 내용이에요 👀 우테코의 프리코스와 거의 유사하지만, 필력도 엄청 좋고 내용도 흥미로워서 가져와 봤습니다!! 특히 리뷰이/리뷰어를 위한 규칙 부분은 제가 평소에 지키지 못하고 있는 부분이 많은 것 같아서 반성하게 되었네여,,,

복붙이 안 되어서 아래는 제가 따로 정리한 내용이에요…!!

🔎 본문 내용

• 리뷰를 해 주는 쪽은 확증 편향으로, 리뷰를 받는 쪽은 동기 추론 편향으로 효과적인 코드 리뷰가 이루어지지 않을 수 있다!

→ 코드 리뷰의 필요성을 깨닫게 하는 게 중요할 것 같아, 우테코 등을 참고해 사내에서 미션 스터디 진행

• 쉬운 난이도의 미션을 진행하면서 기존에 자신이 생각했던 좋은 코드와 좋은 테스트를 적극적으로 작성하고, 이를 리뷰하면서 더 나은 코드로 다듬는 것과 동시에 새로운 지식을 공유하는 효과도 기대할 수 있다

• 단계

  1. 리뷰이는 단계별 요구사항을 충족시킨 후, 리뷰어에게 Pull Request를 남김
  2. 리뷰어는 코드 품질을 높이는 방법과 리뷰이에게 도움이 될 만한 정보를 전달하기 위한 적극적인 코멘트를 작성하여 Code Review 진행
  3. 리뷰이는 리뷰어가 만족할 만큼의 코드를 작성할 때까지 리뷰 내용을 반영하거나 토의 진행
  4. 리뷰어가 만족할 만큼의 코드가 완성되었다면 Approve & merge를 진행하고, 그렇지 않다면 Code Review 반복

• 필자는 미션 난이도가 쉬운 편이기 때문에, 평소 고민하던 객체지향 설계 및 TDD를 적극 시도해 볼 수 있었고, 각자 알고 있었던 프로그래밍 언어 관련 소소한 팁도 공유할 수 있었다!

• 미션 스터디를 알차게 진행하기 위해 규칙 및 가이드 설정

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

→ 9가지 규칙을 지키면서 의식적으로 가독성이 높은 코드를 작성할 수 있고, 리뷰 코멘트의 근거가 되어줄 수도 있음

✔️ 리뷰이를 위한 가이드 → 개인적으로 이 부분이 좋았다 👀