在 C++ 中，您可以在 if 语句中初始化变量，如下所示：
if (CThing* pThing = GetThing())
{
}
为什么人们会认为这种风格不好或好？有什么好处和坏处？
我个人喜欢这种风格，因为它限制了 pThing 变量的范围，所以当它为 NULL 时永远不会被意外使用。但是，我不喜欢你不能这样做：
if (CThing* pThing = GetThing() && pThing->IsReallySomeThing())
{
}
如果有办法使上述工作，请张贴。但如果那是不可能的，我还是想知道为什么。
【问题讨论】：
标签： c++ coding-style if-statement
这个应该在 C++ 中不起作用因为即使它支持short circuiting evaluation。 也许不要尝试以下操作：
if ((CThing* pThing = GetThing()) && (pThing->IsReallySomeThing()))
{
}
【讨论】：
我通常不这样做的一个原因是条件测试中缺少“=”的常见错误。我使用带有错误/警告设置的 lint 来捕获这些错误/警告。然后它会对条件句中的所有作业大喊大叫。
【讨论】：
关于优势：
始终建议您在第一次需要时定义变量，而不是之前的一行。这是为了提高代码的可读性，因为无需滚动和搜索 CThing 的定义位置即可知道它是什么。
还将范围缩小到循环/if 块，导致变量在代码块执行后未被引用，这使其成为垃圾收集的候选者（如果语言支持此功能）。
【讨论】：
仅供参考，一些较旧的 Microsoft C++ 编译器（我认为是 Visual Studios 6 和 .NET 2003）在某些情况下并不完全遵循范围规则。
for(int i = 0; i > 20; i++) {
// some code
}
cout << i << endl;
我应该超出范围，但那是/是有效代码。我相信它是作为一个功能被玩弄的，但在我看来，它只是不合规。不遵守标准是不好的。就像一个关于 IE 和 Firefox 的网络开发者一样。
有 VS 的人可以检查一下是否仍然有效吗？
【讨论】：
您还可以将分配包含在一组额外的 ( ) 中以防止出现警告消息。
【讨论】：
我认为这有点危险。下面的代码更安全，并且括号仍然会以您想要的方式限制 pThing 的范围。
我假设 GetThing() 有时会返回 NULL，这就是为什么我将那个有趣的子句放在 if() 语句中。它可以防止在 NULL 指针上调用 IsReallySomething()。
{
CThing *pThing = GetThing();
if(pThing ? pThing->IsReallySomeThing() : false)
{
// Do whatever
}
}
【讨论】：
重要的是 C++ 中的声明不是表达式。
bool a = (CThing* pThing = GetThing()); // not legit!!
您不能在 if 语句中同时进行声明和布尔逻辑，C++ 语言规范明确允许表达式或声明。
if(A *a = new A)
{
// this is legit and a is scoped here
}
我们如何知道表达式中的一个术语和另一个术语之间是否定义了a？
if((A *a = new A) && a->test())
{
// was a really declared before a->test?
}
咬紧牙关，使用内部 if。范围规则很有用，您的逻辑很明确：
if (CThing* pThing = GetThing())
{
if(pThing->IsReallySomeThing())
{
}
}
【讨论】：
还请注意，如果您正在编写 C++ 代码，您希望编译器在条件语句（不是声明的一部分）中警告“=”成为错误。
【讨论】：
这是可以接受的良好编码习惯。但是，不是来自低级编码背景的人可能会不同意。
【讨论】：
if (CThing* pThing = GetThing())
这是不好的风格，因为在 if 内部你没有提供布尔表达式。您提供的是CThing*。
CThing* pThing = GetThing();
if (pThing != NULL)
这种风格很好。
【讨论】：
很多东西。首先，裸指针。请务必避免使用它们。使用引用、可选、unique_ptr、shared_ptr。作为最后的手段，编写自己的类来处理指针所有权，而不是其他。
如果您可以要求 C++11（首选 C++14 以避免 C++11 缺陷），请使用统一初始化： - 它避免了 = vs == 混淆，并且在检查参数时更加严格。
if (CThing thing {})
{
}
确保实现 operator bool 以获得从 CThing 到 bool 的可预测转换。但是，请记住，阅读代码的其他人不会立即看到operator bool。显式方法调用通常更具可读性和令人放心。如果您需要 C++17，请使用初始化语法。
if (CThing thing {}; thing.is_good())
{
}
如果 C++17 不是一个选项，请按照其他人的建议使用上面的声明。
{
CThing thing {};
if (thing.is_good())
{
}
}
【讨论】：
您可以在if 和switch 中包含初始化语句，因为C++17。
您的代码现在是：
if (CThing* pThing = GetThing(); pThing->IsReallySomeThing())
{
// use pThing here
}
// pThing is out of scope here
【讨论】：