如果缺少 const char* 数组初始化逗号，则生成编译器警告

Question

我在我的 C 代码中经常使用字符串文字表。这些表或多或少都像这样：

static const char* const stateNames[STATE_AMOUNT] =
{
    "Init state",
    "Run state",
    "Pause state",
    "Error state",
};

上面代码的问题是，如果表格变长了，在开发过程中被修改了，我时不时会忘记逗号。代码编译时没有逗号，但我的程序最终崩溃，因为最后一个字符串设置为NULL。我使用了 MinGW 和 Keil 编译器来验证。

如果缺少逗号，是否有任何方法可以为我的初始化生成编译器警告？

Answer 1

将每个const char* 括在一对括号中应该可以解决问题，如下面的sn-p 所示：

static const char* const stateNames[5] =
{
    ("Init state"),
    ("Run state"),
    ("Pause state")     //comma missing
    ("Pause state3"),
    ("Error state")
};

如果你忘记了逗号，你会得到一个类似的编译错误：error: called object is not a function or function pointer

LIVE DEMO

---

请注意，如果您忘记了逗号，实际发生的情况是 C 将实际连接两个（或更多）字符串，直到下一个逗号或数组末尾。例如，假设您忘记了逗号，如下所示：

static const char* const stateNames[] =
{
    "Init state",
    "Run state",
    "Pause state" //comma missing
    "Pause state3" //comma missing
    "Error state"
};

int main(void)
{  
    printf("%s\n", stateNames[0]);
    return 0;    
}

这是gcc-9.2 generates（其他编译器生成类似代码）：

.LC0:
        .string "Init state"
        .string "Run state"
        .string "Pause statePause state3Error state" ; oooops look what happened
        .quad   .LC0
        .quad   .LC1
        .quad   .LC2
main:
        push    rbp
        mov     rbp, rsp
        mov     eax, OFFSET FLAT:.LC0
        mov     rdi, rax
        call    puts
        mov     eax, 0
        pop     rbp
        ret

很明显，最后三个字符串是连接在一起的，并且数组的长度不是您期望的。

Answer 2

如果出现意外结果，您可以让编译器对数组进行计数并生成错误消息：

enum { STATE_AMOUNT = 4 };

static const char* const stateNames[] =
{
    "Init state",
    "Run state",
    "Pause state"    // <--- missing comma
    "Error state",
};

_Static_assert( sizeof stateNames / sizeof *stateNames == STATE_AMOUNT,
        "oops, missed a comma" );

See this thread 如果您的编译器太旧且不支持它，请参考实现 _Static_assert 的想法。

另外，当您添加新状态但忘记更新字符串表时，这也有帮助。但你可能也想研究一下 X 宏。

Answer 3

这不会让编译器帮助您，但我发现像下面这样编写它可以让人类更容易不删除逗号：

static const char* const stateNames[STATE_AMOUNT] =
{
      "Init state"
    , "Run state"
    , "Pause state"
    , "Error state"
};

Answer 4

我一直使用对显式大小数组的引用来解决这个问题。

// no explicit size here
static const char* const stateNames[] =
{
    "Init state",
    "Run state",
    "Pause state",
    "Error state",
};
static const char* const (&stateNameVerifier)[STATE_AMOUNT] = stateNames;

http://coliru.stacked-crooked.com/a/593fc2eac80782a6

main.cpp:10:32: error: reference to type 'const char *const [5]' could not bind to an lvalue of type 'const char *const [4]'
static const char* const (&stateNameVerifier)[STATE_AMOUNT] = stateNames;