如何修复日志宏在if-else结构中触发的“else无对应前置if”编译错误？

这是宏定义里非常常见的语法陷阱！你的问题根源在于宏里的大括号{}会让if语句的作用域提前结束，导致else找不到对应的if。下面给你几种靠谱的修复方案：

方案1：用do-while(0)包裹宏内容（经典兼容方案） #

这是C/C++社区解决这类宏问题的标准操作，它能把宏内容包装成单个完整语句，不会破坏if-else的结构：

#define MY_LOG(s) do { \
    myMutex.lock(); \
    std::cout << __PRETTY_FUNCTION__ << " (" << __LINE__ << ")" << s << std::endl; \
    myMutex.unlock(); \
} while(0)

为什么有效？ #

当你写if(1 == 1) MY_LOG("1 == 1"); else c++;时，宏展开后会变成：

if(1 == 1) do { ... } while(0); else c++;

do-while(0)是一个独立的语句，后面的分号只是语句的结束符，不会让if的作用域提前闭合，else就能正确关联到前面的if了。

方案2：用C++11 lambda表达式（更现代的写法） #

如果你的项目支持C++11及以上，lambda的写法更优雅，还能避免宏的其他潜在陷阱（比如参数多次求值）：

#define MY_LOG(s) [&](){ \
    myMutex.lock(); \
    std::cout << __PRETTY_FUNCTION__ << " (" << __LINE__ << ")" << s << std::endl; \
    myMutex.unlock(); \
}()

为什么有效？ #

[&](){ ... }()是一个立即执行的lambda表达式，同样会被当作单个语句处理，和do-while的效果一致，但写法更贴近C++的现代风格，还能通过捕获列表灵活访问外部变量。

方案3：替换为函数模板（彻底抛弃宏，更安全） #

宏本身就容易带来各种语法陷阱和维护问题，用函数模板可以完全规避这些问题，还能提升代码的安全性：

#include <iostream>
#include <mutex>
#include <utility> // 用于std::forward

extern std::mutex myMutex;

template<typename T>
void MY_LOG(T&& s, const char* func = __PRETTY_FUNCTION__, int line = __LINE__) {
    std::lock_guard<std::mutex> lock(myMutex); // 自动解锁，避免异常导致死锁
    std::cout << func << " (" << line << ")" << std::forward<T>(s) << std::endl;
}

核心优势： #

1. 完全没有宏的语法问题，调用方式和原来的宏完全一致
2. 使用std::lock_guard代替手动lock/unlock，即使std::cout抛出异常（虽然概率极低），也能自动释放锁，避免死锁
3. 模板支持任意类型的输入，兼容性和扩展性更好

问题根源回顾 #

你原来的宏展开后，if(1 == 1) MY_LOG("1 == 1"); else c++;会变成：

if(1 == 1) { myMutex.lock(); ... myMutex.unlock(); }; else c++;

这里的{}已经结束了if的代码块，后面的分号是一个空语句，else自然找不到对应的if，所以编译器报错“else无前置if”。

内容的提问来源于stack exchange，提问作者zomega