TUTORIAL02: 用于debug enum的宏

[randawn]

@miloyip
您好，
我在test.c里面加了两条用于DEBUG的宏。

test.c

#define EXPECT_EQ_RET(expect, actual) EXPECT_EQ_BASE((expect) == (actual), ret_e_name[expect], ret_e_name[actual], "%s")
#define EXPECT_EQ_TYPE(expect, actual) EXPECT_EQ_BASE((expect) == (actual), type_e_name[expect], type_e_name[actual], "%s")

然后把enum对应的名称数组放在了leptjson.h里面

leptjson.h

static const char* ret_e_name[] = {  
    "LEPT_PARSE_OK",
    "LEPT_PARSE_EXPECT_VALUE",
    "LEPT_PARSE_INVALID_VALUE",
    "LEPT_PARSE_ROOT_NOT_SINGULAR",
    "LEPT_PARSE_NUMBER_TOO_BIG"
};
static const char* type_e_name[] = {
    "LEPT_NULL",
    "LEPT_FALSE",
    "LEPT_TRUE",
    "LEPT_NUMBER",
    "LEPT_STRING",
    "LEPT_ARRAY",
    "LEPT_OBJECT"
};

我有一个疑问：

1. 如果我加了static char* ret_e_name[] = ...，编译器会warn: var defined but not used（虽然现在没用use，但是以后可能会）。
2. 如果直接写char* ret_e_name[] = ...，因为leptjson.h被两个文件同时include，LINK的时候会报错redefine。

我在想难道1就是正确的做法吗？通过编译选项把warning去掉？
我总感觉有其他漂亮的办法，希望您能解答。

[miloyip]

在头文件定义变量是不好的。

建议写成函数形式：

const char* lept_get_error_name(int error) {
    static const char* error_name[] = ... ;
    assert(error >= 0 && error < ...);
    return error_name[error]
}

[randawn]

哦 谢谢。
这应该就是best practice.
可以close掉了。