CNAI 소식
도전과 끊임없는 연구를 하는 CNAI의 소식을 전합니다.
보도자료와 새로운 소식을 만나보세요.
씨앤에이아이 개발문화 - 코드리뷰 중요성과 장점,예외처리 검토
[2022-05-20]#2 - DX플랫폼 개발 2팀 개발 이야기
씨앤에이아이 쿠캣 코드리뷰 중요성 및 장점
코드리뷰 중요성 및 장점
개발2팀 코드리뷰 문화는 팀간의 협업과 대화를 원활하게 만들어 주는 매우 중요한 절차이다.
'코드리뷰를 하는 이유'와 중요성 및 장점을 아래와 같이 공유한다.
1
가독성 및 유지보수의 편의성
2
개발표준(Convention) 준수여부 확인
3
중복 코드 방지 및 모듈의 재사용성 증대
4
다른사람의 코드를 보며 학습의 기회
5
개개인의 코딩 실력 격차 해소의 기회
6
본인이 알아차리지 못한 버그/오류의 조기발견
7
조직의 역량증대
8
개발 결과물에 대한 책임이 코드를 만든사람에 게 있지 않고 우리모두에게 있다는 문화 정착 -> 조직내 책임감 증대
씨앤에이아이 개발2팀 코드리뷰
리뷰어 칭찬하기
리뷰어의 소스코드 리뷰 중 실수에만 초점을 맞추지 않고, 좋은 점을 발견하거나, 훌륭하게 처리했을 때 모범 사례에 대한 격려와 감사도 제공해야 한다.
때로는 멘토링의 관점에서 개발자에게 무엇을 잘못했는지 알려주는 것보다 개발자에게 무엇을 잘했는지 알려주는 것이 훨씬 더 가치가 있다.
  • 코드량이 적당해서 읽기 편하네요.
  • 많은 고민이 코드에서 엿보이네요.
  • 성능에 아주 유리한 코드라고 생각되네요.
  • 전에 코드보다 훨씬 좋아진 거 같네요.
  • 예외 처리가 꽤 꼼꼼해서 좋네요.
  • 함수, 변수명이 직관적이어서 좋네요.
예외처리 체크리스트 검토
1
에러코드와 예외의 호출 함수는 분명하게 반환되고 있는가?
2
호출 함수가 어려워 예외처리 코드를 가지고 있는가?
3
Null 포인트 음수가 처리되는 구조인가?
4
에러모드에 대해서 명쾌하게 선언하고 처리하고 있는가?
5
switch문에 default가 존재하고, 예외처리를 하고 있는가?
6
배열 사용시에 index 범위를 체크하는가?
7
포인트 사용시에 유요한 범위를 체크하는가?
8
Garbage collection 을 제대로 하고 있는가?
9
에러조건이 체크되고 에러 밸생 시 로깅 정보를 남기는가?
10
에러 메세지와 에러코드가 에러 의미를 잘 전달 하는가?
11
Try/ catch에러 핸들링 사용방법은 적절하게 구현되었는가?
이전글이전글 | 씨앤에이아이 콜센터구축 - ASP고객센터 구축, 관리
다음글다음글 | 씨앤에이아이 개발문화 - 코드리뷰란? 코드리뷰 방법과 필수 내용