c++ - 让宏定义为函数返回一个值是不好的做法吗？

Question

使用定义为有条件地返回值的宏有一个缺点，即仅查看客户端代码可能会在宏点退出时并不明显。

我正在考虑的用例是编写值和错误检查，如下所示:

#define WRITE_CHK(file, param)\
if (!write_that_returns_zero_on_fail(file, param)) {\
   handle_error();\
   return false;\
}

客户端代码:

bool myfunc()
{
   ...
   WRITE_CHK(file, param) // function might return here
   ...
   return true;
}

我很好奇宏(将在我的代码中的许多地方使用)的好处是否会超过上述缺点。除了简单地扩展(不使用宏)之外，是否有首选的替代方案？

Answer 1

标准答案是“不要使用宏”；但这通常有点简单化。在某些情况下，它们可以大大减少样板文件的冗长程度，否则您会遇到这种情况。

那么，为什么不将事实编码到宏名称中呢？例如WRITE_OR_RETURN_ON_FAILURE。它可能有点冗长，但不太可能让您的代码的读者感到困惑。