新的c++警告会提高可靠性吗？

Question

因此，我在C++中工作了很长一段时间之后一直在使用C#，现在我注意到了所有这些警告。

虽然有些警告似乎是使代码更可靠的好建议，但似乎许多警告并不可靠，而且其中一些似乎完全错误。

下面是一个我觉得不值得的例子：

static TCHAR szGeneral[] = _T("General");
static TCHAR szSqlOdbcDriver[] = _T("SqlOdbcDriver");

WriteProfileString(szGeneral, szSqlOdbcDriver, CSqlServerDatabase::m_sOdbcDriver.GetString());

警告C26485表达式'szGeneral'：没有指向指针衰减的数组(界3)。警告C26485表达式'szSqlOdbcDriver'：没有指向指针衰减的数组(界3)。

我的问题是:这会使我的代码更好吗？

我认为我不得不关闭警告，因为我的项目中有将近800个警告。(十年前编写的代码被认为是完全有效的。)所以我会错过所有的好建议。结果就是我的代码更糟了。

Answer 1

请注意，此警告来自最近添加到Visual的核心指南检查器。只有当您从头开始编写新代码并准备遵循核心准则时，此检查器发出的警告才有意义。否则，请关闭检查程序。

检查器是静态代码分析器的一部分，您可能不应该在编译中包括分析器。

在您的项目属性中，/analyze在C/C++-->Command Line中。您应该去掉此标志(我不知道如何针对您的Visual版本这样做)。

不要关闭所有警告。

Answer 2

海事组织视情况而定。例如，编译器发出警告，因为当您将char[]传递给函数时，您就丢失了sizeof()信息。但是，这可能并不重要，因为字符串是空终止的。但是，考虑一下不知何故nullptr已经消失的情况。那么在函数中如何计算它的长度呢？

为了在这种情况下帮助编译器，我建议显式地告诉编译器您知道自己在做什么。这也使得意图显而易见。考虑-

long b = 50;
int a = b;

将生成编译器警告，但取决于使用情况可能是安全的。但是，您要对团队成员进行猜测(这是有意的还是错误的？)哪里

long b = 50;
int a = static_cast<int>(b);

清楚地表明这是有意的。