zeliard
commented
9 years ago - 마일스톤 배포 및 업데이트 노트 OK.
- 코드 전반적인 피드백
- 일관성이 없는 코드가 여기저기 보임: 어떤 멤버 함수는 소문자로 시작하고 어떤 놈은 대문자로 시작하고... 코딩 컨벤셩을 통일할 것.
- PacketType.h에 공유 타입 다 때려박아놓은 느낌인데.. 이거 분리할것.. 저기에는 순수하게 패킷 종류만 들어갈 것. 그리고, 패킷 안에 다른 종류의 struct를 막 박아 넣는 것은 상당히 주의해야 함. (메모리 alignment 문제 발생 가능성) 특히, POD 타입이 아닌 struct가 들어가면 큰일 날 수 있다.
- 배열 크기에 그냥 숫자 상수 100.. 이런식으로 막 써놓은거 있는데..이거다 기술 부채(debt)다..
- 헤더 파일에는 웬만하면 구현 부분을 넣지 말 것.. 2줄 이상 되면 cpp에 넣을 것
- 상속받은 클래스는 가상 소멸자 사용 잊지 말것.
- 위험한 코드들 피드백 785a7fd8ed733b285e2683aa88094459f8f30bc5