将 char 指针复制到 char 指针数组时发生 Seg 错误

Question

我正在从文件中读取一行文本，并尝试使用strtok() 将其拆分为空格，然后在名为tokenize 的函数中使用strcpy() 将每个标记存储在一个字符指针数组中。

代码：

/**
** Tokenizes contents of buffer into words
** Delimiter is a space character
*/
void tokenize(char *buffer, char *tokens[]){
    char *token = NULL;
    int i = 0;
    token = strtok(buffer, " ");
    while(token != NULL){
        strcpy(tokens[i], token);//seg fault line
        token = strtok(NULL, " ");
        i++;
    }
}

我假设，根据 K&R 中的函数描述，我的调用会起作用，因为我传递了一个 char 指针，这是 tokens[i] 应该取消引用的，以及另一个包含字符串内存地址的 char 指针复制，这就是token 应该是什么。但是，当我尝试strcpy 时，我遇到了段错误。

在通过调用fgets() 检索到一行后立即在main 中调用此函数。 buffer 被声明为 char buffer[MAXLINE_LEN - 1]，它的大小为 100。tokens 被声明为 char *tokens[MAXLINE_LEN - 1]。

在main 中致电tokenize：

while(fgets(buffer, MAXLINE_LEN, inputFile)){
    int choice;
    printf("%s", buffer);
    tokenize(buffer, tokens);
    /**
    ....
    */
}

我正在使用：gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609

编辑： tokens 的声明（这发生在上面显示的 while 循环之前，在 main 中）：

char *tokens[MAXLINE_LEN - 1];//MAXLINE_LEN - 1 = 100

int a;
for(a = 0; a < MAXLINE_LEN - 1; a++)
    tokens[a] = NULL;

Answer 1

我能想到四种可能

1. tokens 是 NULL 或未初始化
2. tokens 已初始化，但您没有分配足够的空间。如果您为四个令牌分配空间但strtok() 解析五个令牌怎么办？
3. tokens 的各个元素是 nNULL 或未初始化
4. 您没有在每个 elemant 中分配足够的空间来复制令牌字符串。

其中一项或多项几乎肯定适用于您的代码。解决它的一种方法是在函数内部完全分配 tokens 并在指针中返回它（以计数作为返回值）。

enum
{
    CHUNK_SIZE = 16; // A trick to get an int constant in C that works everywhere a #define works
};

size_t tokenize(char *buffer, char **tokenRet[])
{
    size_t capacity = 0;
    char** tokens = NULL;
    size_t count = 0;
    char *token = NULL;
    token = strtok(buffer, " ");
    while(token != NULL)
    {
        count++;
        if (count > capacity)
        {
            capacity += CHUNK_SIZE;
            tokens = realloc(tokens, capacity * sizeof *tokens);
        }
        tokens[count - 1] = strdup(token);//strdup mallocs space and copies the string into it
        token = strtok(NULL, " ");
    }
    *tokenRet = tokens;
    return count;
}

这样称呼

char** myTokens = NULL;
size_t tokenCount = tokenize(buffer, &myTokens);

然后整理到最后，free()myTokens的第一个tokenCount元素，然后释放myTokens。

Answer 2

“char *tokens[]”的大小是多少？ “char *buffer”中的令牌可能比令牌[] 可以容纳的更多。 另外，您是否为指针数组分配了内存？