如何检查/检测没有花括号的“if”子句？

Question

在 C/C++ 代码项目中，我想找到所有不使用大括号的 if/else 语句。是否有任何编译器检查或实用程序可以找到它们？

主要问题是我想重新定义一些调试宏以避免在某些情况下进行跟踪评估（有谁知道其他解决方案吗？）。而且我需要找到它们以避免“dangling-else problem”，正如"if" block without curly braces makes subsequent "else if" nested中指出的那样

代码片段：

修改前：

DEBUG GET_DEBUG_DST(DEBUG_LEVEL_DEBUG).nospace() << DEBUG_PREFIX << __PRETTY_FUNCTION__

现在我们将使用没有大括号的if：

DEBUG if (getDebugmode(DEBUG_LEVEL_DEBUG)) GET_DEBUG_DST(DEBUG_LEVEL_DEBUG).nospace() << DEBUG_PREFIX << __PRETTY_FUNCTION__

有问题的代码示例：

if (my condition)
    DEBUG << "hi there";
else
    {some more code;}

Answer 1

坏主意。

用途：

#define debug(x) do { if (DEBUG_LEVEL) print_stuff(x); } 而（0）

这避免了这个问题。

Answer 2

我想在 c/c++ 代码项目中找到所有不使用大括号的 if/else 语句。是否有任何编译器检查或实用程序可以找到它们？

这听起来像x-y problem。如果您尝试使用不会使 if 语法无效的空操作替换 DEBUG 宏，您可以使用：

#ifdef DEBUG_ENABLED
#define DEBUG GET_DEBUG_DST(DEBUG_LEVEL_DEBUG).nospace() << DEBUG_PREFIX // ...
#else
#define DEBUG do {;} while(false); /##/
#endif

在这种情况下，DEBUG 将是一个有效操作（不会因为 if 块中的任何内容而使 else 无效），并且同一行上的任何内容（调试消息数据）都将评估为：

if (my condition)
    do {;} while(false); // << "hi there";
else
    {some more code;}

这有一个缺点，就是不允许多行调试 cmets：

if (my condition)
    DEBUG << "hi there"
        << "some more data on a different line"; // causes error if DEBUG is no-op
else
    {some more code;}

Answer 3

只需反转逻辑，将活动部分放在else中 分支：

#define DEBUG if( !getDebugmode( DEBUG_LEVEL_DEBUG ) )              \
        ;                                                           \
    else                                                            \
        GET_DEBUG_DST( DEBUG_LEVEL_DEBUG ).nospace()                \
            << DEBUG_PREFIX << __PRETTY_FUNCTION__

由于if 有匹配的else，因此无法接听 任何额外的else。