코드리뷰의 목적 : 버그 예방

코드리뷰의 방법

• 모르는 코드를 작성자에게 물어보는 것이 아닌, 리뷰어가 직접 코드를 소리내어 읽어가면서 한줄 한줄 문맥을 파악
• 질답 형식으로 진행되면 리뷰가 아닌 단순한 설명 형식이 되므로 질답은 되도록 자제
• 큰 그림이나 객체들의 연관관계가 궁금할때는 핵심만 설명
  

코드리뷰 할 때 무엇을 보아야 하는가

• assert, trace, if 문등의 런타임 검사를 올바르게, 일관적으로 하고 있는가
• 문맥에 맞는 코드인가
• 객체나 함수의 하는 일이 이름에 맞는가
• 전체적으로 코드의 일관성이 지켜지고 있는가
• 네이밍은 적절한가
• 주석 사용은 적절한가
• 기타 버그를 유발 시킬수 있는 모든것

Cyclone팀에서는 위의 원칙에 맞추어 일주일에 한번씩 주간 코드리뷰를 진행하고 있습니다.

시행중인 Practice

• 테스트 주도 개발 : 테스트를 먼저 작성하고 작성된 테스트 케이스를 통과하기 위한 구현 코드를 나중에 작성합니다. 대부분의 코드를 커버하는 테스트 케이스가 작성되어 있습니다.
• 리팩토링 : 코드가 잘 작동하고 있음을 보증해 주는 테스트 케이스가 있기 때문에 리팩토링에도 망설임이 없습니다. 따로 날 잡아서 하는 대청소 같은 리팩토링이 아니라 작은 걸음으로 지속적으로 수행하여 코드의 품질을 개선합니다.
• 짝 프로그래밍 : 하루 30분의 짝 프로그래밍을 통해 신입 프로그래머는 TDD나 리팩토링의 노하우를 배우게 되고, 경력자는 상대 영역에 대한 이해와 작업 집중도를 높이게 됩니다.

     

• 코드리뷰 : 한주에 한번씩 2인 1조로 상대방의 코드를 리뷰 해줍니다. 코드 작성에서 자신만의 세계에 빠지기 쉬운 프로그래머에게 다른 사람의 피드백을 지속적으로 받음으로 인해 오픈 마인드와 코드 작성 능력 증진에 도움이 됩니다.
• 사용자 스토리 : 문서를 통해 기획내용를 파악하는 방식이 아니라, 사용자 스토리를 작성하면서 필연적으로 나누게 되는 많은 대화를 통해 기획의도를 파악하여 올바른 구현이 이루어지게 합니다.

      [싸이클론팀 입구에 위치한 사용자 스토리 보드 사진 - "가는 날이 장날이라고 사진찍으려고 하니까 카드가 별로 없네"]

• 일일회의 :  오전 10시20분에 모두 모여서 어제 한일, 오늘 할일, 작업 진행간에 발생한 문제에 대해 이야기 함으로써 문제점을 오픈시켜 빠르게 해결책을 찾습니다.

     
      [일일회의 중인 팀원들 - "근데 일어서서 하는거 아니던가?"]

• 교육 : Modern C++ Design, OOP 원칙, 수학과 같이 흥미로우면서도 개발에 반드시 필요한 주제로 팀장님이 한주에 한번씩 교육을 합니다.
  
• 지속적인 통합 : 조금씩 점진적으로 발전하는 프로젝트를 구성원 모두가 볼 수 있도록 2주에 한번씩 릴리즈 합니다.

필독서

   실용주의 프로그래머         Head First Design Pattern                      
                

익스트림 프로그래밍                       사용자 스토리                 테스트 주도 개발
         

개인 블로그 하나쯤은 다들 가지고 있는게 추세인지라 팀 사람들도 개인 블로그 하나씩 다 가지고 있어서 소개할까 합니다.

원화작가
http://blog.naver.com/goeloe
http://blog.naver.com/hiyase
http://blog.naver.com/kakorachi

프로그래머
http://blog.naver.com/goli81
http://javawork.egloos.com

레벨 디자이너
http://blog.naver.com/gunil2
http://blog.naver.com/d7844

시나리오
http://stoneblue.egloos.com
http://brightnoa.tistory.com

3D 아티스트
http://blog.daum.net/mark2
http://phochu.tistory.com
http://blog.naver.com/ckdansduv

게임 디자인
http://blog.naver.com/jaehongjin
http://blog.naver.com/sammuel310
http://www.princeod.net

안녕하세요. 싸이클론팀 최피디입니다.
저희 팀블로그에 오신 모든 분들께 감사하다는 말씀 올리며 팀블로그에 대해서 짧게 몇 가지만 말씀드리고 싶습니다.

우선 저희 팀블로그는 비공식 블로그라는 점 확실히 해 두고 싶습니다. 특히 여기 올라오는 글은 순수하게 개발자 입장에서 글을 올리는 것이기 때문에 회사의 공식 입장이 아닙니다.

그래서 저희 회사 공식 입장과 위배되는 것도 있을 것이고 오탈자도 많을 수 있습니다.

그냥 게임하나 만든 팀에서 다른 개발자분들과 호흡을 같이 하면서 정보도 주고 받고 근황도 전하는 순수한 블로그로 봐 주셨으면 합니다.

다시 한번 강조합니다만 이 블로그는 비공식 블로그입니다.

감사합니다.