기업소개 소식 레터・영상

코드리뷰 중요성과 장점, 예외처리 검토

[2022-05-20]#02 - 씨앤에이아이 레터 / 개발2팀 개발문화
씨앤에이아이 회의실, 코드리뷰 중
코드리뷰 중요성 및 장점

개발2팀 코드리뷰 문화는 팀간의 협업과 대화를 원활하게 만들어 주는 매우 중요한 절차입니다.
'코드리뷰를 하는 이유'와 중요성 및 장점을 아래와 같이 공유합니다.

  • 1
    가독성 및 유지보수의 편의성
  • 2
    개발표준(Convention) 준수여부 확인
  • 3
    중복 코드 방지 및 모듈의 재사용성 증대
  • 4
    다른사람의 코드를 보며 학습의 기회
  • 5
    개개인의 코딩 실력 격차 해소의 기회
  • 6
    본인이 알아차리지 못한 버그/오류의 조기발견
  • 1
    조직의 역량증대
  • 1
    개발 결과물에 대한 책임이 코드를 만든사람에 게 있지 않고 우리모두에게 있다는 문화 정착 -> 조직내 책임감 증대
개발자 코드리뷰 그림
리뷰어 칭찬하기

리뷰어의 소스코드 리뷰 중 실수에만 초점을 맞추지 않고, 좋은 점을 발견하거나, 훌륭하게 처리했을 때 모범 사례에 대한 격려와 감사도 제공해야합니다. 때로는 멘토링의 관점에서 개발자에게 무엇을 잘못했는지 알려주는 것보다 개발자에게 무엇을 잘했는지 알려주는 것이 훨씬 더 가치가 있습니다.

  • 코드량이 적당해서 읽기 편하네요.
  • 많은 고민이 코드에서 엿보이네요.
  • 성능에 아주 유리한 코드라고 생각되네요.
  • 전에 코드보다 훨씬 좋아진 거 같네요.
  • 예외 처리가 꽤 꼼꼼해서 좋네요.
  • 함수, 변수명이 직관적이어서 좋네요.
예외처리 체크리스트 검토
  • 1
    에러코드와 예외의 호출 함수는 분명하게 반환되고 있는가?
  • 2
    호출 함수가 어려워 예외처리 코드를 가지고 있는가?
  • 3
    Null 포인트 음수가 처리되는 구조인가?
  • 4
    에러모드에 대해서 명쾌하게 선언하고 처리하고 있는가?
  • 5
    switch문에 default가 존재하고, 예외처리를 하고 있는가?
  • 6
    배열 사용시에 index 범위를 체크하는가?
  • 7
    포인트 사용시에 유요한 범위를 체크하는가?
  • 8
    Garbage collection 을 제대로 하고 있는가?
  • 9
    에러조건이 체크되고 에러 밸생 시 로깅 정보를 남기는가?
  • 10
    에러 메세지와 에러코드가 에러 의미를 잘 전달 하는가?
  • 11
    Try/ catch에러 핸들링 사용방법은 적절하게 구현되었는가?