《C++编程规范》
1、使用编译器的最高警告级别,成功的构建应该是无声无息的(没有警告的)。
如果确定是无害警告,且是无法修改的第三方头文件引起的,可以用自己的头文件包装起来,并有选择性的关闭警告,然后项目中使用该头文件。
#pragma warning(push) //仅禁用此头文件 #pragma warning(disable:4512) #pragma warning(disable:4180) #pragma warning(pop) //恢复最初的警告级别
"未使用的函数参数": 如果是为未来预留的,则删除参数名,只保留参数类型即可去警告。如:
void print(int i, void * /* v */ = 0);
"定义了从未使用过的变量": 可以通过插入一个变量本身的求值表达式,使编译器不再报警,这种求值不会影响运行时速度,如:
void fun() { Lock lock; lock; }
"有符号数/无符号数不匹配": 其实不管怎么样,编译器都会为你插入一个强制转换,同时还会发出警告,因此还不如显式强转一下。
有时编译器会发出烦人甚至虚拟的警告,当遇到这种罕见情形时,作为团队决定可以单独禁用这个警告,但是要尽可能在局部禁用,且编写一个清晰的注释说明为什么禁用。
2、审查代码:更多的关注有助于提高质量。
既能通过同伴的良性压力提高代码质量,又是内部培训的一种极佳方法。