在 C 中实现拆分函数

Question

我正在尝试编写一个类似于 Java 中的split 的 C 函数。当我做我在 main 做的事情时，而不是添加功能，它完美地工作。但我不明白为什么它不适用于添加功能。

#include <stdio.h>
#include <string.h>
char *words[50] = { NULL };

void add(const char *word) {
    static int i = 0;
    words[i] = word;
    i++;
}


int main( void )
{
    char string[65];
    char *tokenPtr; 

    fgets(string, 65, stdin); 
    tokenPtr = strtok( string, " " ); 
    add(tokenPtr);

    int i = 0;
    while ( tokenPtr != NULL ) {
        add(tokenPtr);
        tokenPtr = strtok( NULL, " " ); 
    }

    int i;
    for(i = 0; words[i] != NULL; i++)
        puts(words[i]);

    return 0; 
} 

这只是我实际代码的一小部分，有一些原因我需要在另一个函数中这样做。

Answer 1

删除第一个add 调用。

fgets(string, 65, stdin); 
tokenPtr = strtok( string, " " ); 
// add(tokenPtr); // remove

因为您将在下一个 while 循环中添加第一个令牌。

另外，您应该删除重复的int i。

// int i; // <-- why is it there?
for(i = 0; words[i] != NULL; i++)
    puts(words[i]);

Answer 2

除了删除对add 的第一次调用和i 的重复声明之外，还有一些其他注意事项。虽然使用while 解析令牌并没有错，尤其是strtok，但for 循环很好地处理了初始调用和每个后续调用NULL。

同样，虽然 static 声明 add 中的 i 没有任何问题，但将索引作为参数传递给函数可以提供对 main() 中计数的访问。如果您以后决定动态分配/重新分配words，这会增加令牌的确定数量以及灵活性。

虽然在依赖指针上的NULL 测试时使用for 循环无限迭代并没有错，但如果您确实将令牌索引作为参数传递给add，您可以遍历确定范围0 < i < token index 并确保在您拥有49 令牌的情况下不会尝试读取超出words 末尾的内容。 （可以添加测试）

最后，（这只是一个小问题），当声明变量用作正计数器、索引等时，考虑使用size_t 而不是int。

解决这个问题有很多方法，只要正确，没有一种比另一种更正确。考虑到所有因素，对代码的轻微修改可能如下所示：

#include <stdio.h>
#include <string.h>

char *words[50] = { NULL };

void add (const char *word, size_t idx) {
    words[idx] = word;
}

int main( void )
{
    char string[65];
    char *tokenPtr; 
    size_t tokIdx = 0;      /* token index  */
    size_t i = 0;           /* use size_t if non-negative counter */

    fgets(string, 65, stdin); 

    for (tokenPtr = strtok (string, " "); tokenPtr && *tokenPtr; tokenPtr = strtok (NULL, " "))
        add (tokenPtr, tokIdx++);

    for (i = 0; i < tokIdx; i++)
        printf (" words[%zu]  %s\n", i, words[i]);

    return 0; 
}

输出

$ echo "how many tokens are in this line eight?" | ./bin/strtokadd
 words[0]  how
 words[1]  many
 words[2]  tokens
 words[3]  are
 words[4]  in
 words[5]  this
 words[6]  line
 words[7]  eight?