清理调试输出功能？

Question

我想编写一个函数，它将在我的程序中打印出错误消息/警告以及文件和文件。行号。 C中有这两个宏：

__FILE__
__LINE__

但是我认为有一个问题...当我编写这样的函数时：

#include <stdio.h>
#define STRINGIFY(x) #x
#define TOSTRING(x) STRINGIFY(x)
#define AT __FILE__ ":" TOSTRING(__LINE__)
void error(const char *location, const char *msg)
{
#ifdef DEBUG
  printf("Error at %s: %s\n", location, msg);
#endif
}
int main(int , char**)
{
  error(AT, "fake error");
  return 0;
}

仍然有很多无用的函数调用&二进制文件中的垃圾（每次调用时 __FILE__ 和 __LINE__ 的值），即使我为发布版本取消定义 DEBUG 也是如此。 那么我怎样才能更优雅地完成这个任务呢？我想要这样的东西：

#include <stdio.h>
#define STRINGIFY(x) #x
#define TOSTRING(x) STRINGIFY(x)
#define AT __FILE__ ":" TOSTRING(__LINE__)
void error(const char *location, const char *msg)
{
  printf("Error at %s: %s\n", location, msg);
}
int main(int , char**)
{
#ifdef DEBUG
  error(AT, "fake error");
#endif
  return 0;
}

但不要在每次函数调用之前和之后编写 #ifdef DEBUG 和 #endif - 这对于这样的任务来说太大了。并删除每个 error(AT, "fake error"); 手动调用也不是很优雅......

有什么想法吗？也许内联该函数（不会有帮助，不是吗）？一些宏或这个结构的改变？

Answer 1

只需将 error 变成一个宏：

#ifdef DEBUG
# define ERROR(l, m) error(l, m)
#else
# define ERROR(l, m)
#endif

然后在你的函数中编写

ERROR(AT, "fake error");

当然，你也可以简化 ERROR 并去掉 AT 作为第一个参数，直接指定这些信息在宏定义中。

Answer 2

将 error() 包装在宏中确实是正确的方法 - 我会这样写：

#ifdef DEBUG
static void error(const char *file, long line, const char *msg)
{
  fprintf(stderr, "Error (%s:%ld): %s\n", file, line, msg);
}
#define error(msg) (error)(__FILE__, __LINE__, msg)
#else
#define error(msg) ((void)0)
#endif

Answer 3

您可以使用此可变参数宏来模拟 printf ，它会使用所需的输出写入文件和行号。

#define eprintf(...) do {fprintf(stderr, "%s:%d: ", __FILE__, __LINE__);\
 fprintf(stderr, __VA_ARGS__);} while(0)

编辑：根据 jcsalomon 的建议修改了 eprintf

Answer 4

#ifdef DEBUG
#define ERROR(msg) error(AT, msg)
#else
#define ERROR(msg)
#endif

int main( int argc, char * argv[] )
{
   ERROR( "fake error");
   return 0;
}

Answer 5

将错误替换为#define，

例如：

#ifdef DEBUG
#define error(a,b) printf("Error at %s: %s\n", (a), (b))
#else
#define error(a,b) 
#endif

您还可以将 AT 移至错误中，而不是每次都传递它