2014 10 21 회의록

#코드 리뷰

virtual 대신에 ABSTARACT OVERRIDE 매크로 사용할 것!
전방선언 웬만하면 써라! get set 헤더에서 inline 하지말자.
소멸자는 무조건 virtual로!
헤더에 메소드 -> 멤버변수 순으로 작성
헤더파일 맨위에 작성자 내용 설명등 필히 작성
이름만으로 알기 힘든 타입을 typedef로 변경해서 쓰자.
(ex typedef std::map<int, std::vector> ModuleDataContainer;)
인자만 다르고 같은 일을 하는 함수는 그냥 오버로딩을 하자.
공통된 부분이 반복되어서는 안된다. 공통되는 부분을 함수화해라.
함수 하나가 300줄짜리; 6중 Loop; 함수를 의미 단위로 짤라서 써라.
의미불명의 코드. 주석 달거나 제대로된 이름이 있는 함수로 따로 빼라.(함수가 권장됨)
하나의 함수, 하나의 변수는 하나의 역할만 할 수 있도록. 의미가 명확해진다.
한줄이 너무 길지 않도록 적당히 짤라줘라.
전체 구조가 지나치게 복잡. 순차적으로 파악할 수 있도록, 흐름을 볼 수 있도록
내용을 안봐도 알 수 있는 get / set 함수는 제일 밑에다 배치해라.
_ASSERT로 제대로된 input / output값을 받는지 수시로 확인하자. 어디서 잘못되었는지 알기 쉽게!
retain을 쓸때 따로 표시할 것. 적어도 주석.

꾸준한 리뷰