9.2.3 将警告视为错误

亦凉 2022-06-18 04:54 188阅读 0赞

Google C++ Style Guide 中有这样一条规则  
*When compiling with Microsoft Visual C++, set the compiler to warning level 3 or higher, and treat all warnings as errors.  
使用 Microsoft Visual C++ 进行编译时, 将编译器的警告等级设置为 3 或更高, 并将所有警告视为错误.*

过去我并不是很在意编译器发出的警告，因为接手的代码总是会有上百的warnings  
但是，今天遇到的abs函数调用问题让我对此深有体会。  
先看3个例子：  
以下例子均使用Visual C++ 2005编译，警告等级为3

**例子一**

#include <stdio.h>
    int main(int argc, char* argv[])
    {
      float f = abs(-1.6f);
      printf("f = %f\n", f);
      return 0;
    }

编译结果

error C3861: “abs”: 找不到标识符
    1 个错误，0 个警告

**例子二**

#include <stdio.h>
    #include <stdlib.h>
    int main(int argc, char* argv[])
    {
      float f = abs(-1.6f);
      printf("f = %f\n", f);
      return 0;
    }

编译结果

warning C4244: “参数”: 从“float”转换到“int”，可能丢失数据
    warning C4244: “初始化”: 从“int”转换到“float”，可能丢失数据
    0 个错误，2 个警告

运行结果

f = 1.000000

**例子三**

#include <stdio.h>
    #include <math.h>
    int main(int argc, char* argv[])
    {
      float f = abs(-1.6f);
      printf("f = %f\n", f);
      return 0;
    }

编译结果

0 个错误，0 个警告

运行结果

f = 1.600000

由此可见，包含不同的头文件程序会有不同的表现，警告有时候比错误更严重。  
因为错误必须修改，而警告可能被你忽略。

但是取绝对值是个数学函数，谁会先想到包含<stdlib.h>呢？  
好吧，再来看一个例子。

**例子四**

#include <iostream>
    int main(int argc, char* argv[])
    {
      float f = abs(-1.6f);
      std::cout << "f = " << f << std::endl;
      return 0;
    }

编译结果

运行结果

f = 1

看到了吧，我们没有刻意包含<stdlib.h>文件，它是被隐式包含进来的。

**为了避免发生逻辑错误，现在就把警告等级提高，并将警告视为错误，你会因此受益的。**