c - assert == 中的函数调用不好吗？

Question

我刚刚在我们正在使用的一些代码中发现了一个错误*，由于以下情况而失败:

Assert(SomeVitalFunction(foo) == OK)

在 DEBUG 宏被 #defined 的时候这一切都很好:

#ifdef DEBUG
    #define Assert(x)  if((x) == 0){/*some error handling*/}
#else
    #define Assert(x)
#endif

但是当我们#undef'd DEBUG时，它具有从代码中删除重要函数调用的效果。

我终其一生都无法弄清楚它如何与 DEBUG #undef'd 一起工作，而且将任何类型的函数调用放在断言中似乎不是一个好主意像这样。

我错过了什么吗？

* = 编辑以澄清以下 Carpetsmoker 的评论:代码来自一个特别落后的 Elbonian 代码奴隶 cabal ，我们的任务是破解、削减、刮 mustache 、抛光、 sanitizer 和涂口红

Answer 1

您没有遗漏任何东西。

断言应该始终写得好像它们可以在编译器开关轻弹时消失。

您可以在断言中调用需要相对较长时间才能完成的函数(例如分析数据结构的完整性)，因为函数调用不会出现在发布版本中。不利的一面是您无法调用正确操作所必需的函数。