C++技术复查表

原文地址 - http://blog.youkuaiyun.com/gltop/article/details/7040124

版本 - 0.8

最近更新 - 2011-12-27

• 是否使用了一致的代码风格？
• 那些non-const成员函数是否应该是const的？是否有不必要const的返回值或参数用了const了？

• 有没有哪些函数不应该是virtual的？

• 需要被访问的数据成员是否私有化并用get/set函数包装了？

• 避免写硬代码(数值)。

• 那些公有的函数是应该定义为公有的(作为开放的接口)还是保护的(只是用于继承)？有哪些成员函数只要定义在本地的匿名名字空间就可以吗？

• 如果创建一个有可能被继承的类，那么它的析构函数是否被定义为virtual的了。

• 新的类/函数/变量是否遵从名字约定了？他们的拼写是否正确？

• 是否消除了所有的编译警告？

• 注意strcpy, memcpy, printf和其他的内存操作函数，是否有可能内存越界。

• 所有的成员数据是否在构造函数中初始化了？

• 函数/类的名字是否可以清楚地表达它是做什么的？类/函数是否有恰当的注释？类/函数的改动是否已经使得它们不同于设计初衷了？那么命名和注释是否作了相应的改动了？

• new出来的内存都被释放了吗？

• 有没有哪些数值可以被定义为枚举、宏、const或者static const？

• #ifdef和#if define的使用是否恰当？

• 你要实现的新的类或者函数是否需要对new/delete重载？

• 两个库之间是否有不恰当的交叉耦合？有没有不合适的包含关系？

• 检查soft fails (e.g. if(pObject == NULL) return;)，考虑好并确定是否应该用断言来代替它？

• 代码和数据的跨平台性被考虑了吗？

参考

• Best Practices for Peer Code Review