使用 strtok 不同的字符串长度

Question

void redact_words(const char *text_filename, const char *redact_words_filename){
    FILE *fp = fopen(text_filename,"r");
    FILE *f2p = fopen(redact_words_filename,"r");
    FILE *f3p = fopen("result.txt", "w"); ;
    char buffer1[1000];
    char buffer2[1000];
    char *word;
    char *redact;
    

    
    char **the_words;
    
    //if ((fgets(buffer1, 1000 ,fp) == NULL) || (fgets(buffer2,1000 ,f2p) == NULL))
    
    fgets(buffer1,1000,fp);
    fgets(buffer2,1000,f2p);
    
    rewind(fp);
    rewind(f2p);
    
    int word_count = 0; 
    while (!feof(f2p)){
        char c = fgetc(f2p);
        if (c == ' '){
            word_count += 1;
        }
    }
    word_count += 1;
    
    the_words = malloc(3 * sizeof(char*));
    redact = strtok(buffer2, ", ");
    
    for (int i = 0; i < word_count; i++){
        the_words[i] = malloc(100);
        the_words[i] = redact;
        redact = strtok(NULL, ", ");
    }
    
    char result[256] = "";
    word = strtok(buffer1, " ");
    while (word != NULL){
        for (int i = 0; i < word_count; i++){
            if (strcasecmp(the_words[i],word) == 0){
                for (int i = 0; i < strlen(word); i++){
                    strcat(result,"*");
                    
                }
                strcat(result, " ");
                break; 
            }
            else{
                if (i==(word_count-1)){
                    strcat(result, word);
                    strcat(result, " ");
                }  
            } 
        }
        word = strtok(NULL," "); 
    }
    
    fputs(result, f3p);
    
    fclose(fp);
    fclose(f2p);
    fclose(f3p);
    free(the_words);
}

这是我的 C 代码，如果名为 redact_words_filename 的文件中存在该单词，则用星号替换名为 text_filename 的文件中的单词。然而，我注意到在与 2 个字符串的比较过程中

if (strcasecmp(the_words[i],word) == 0){
                for (int i = 0; i < strlen(word); i++){
                    strcat(result,"*");

                }

，当我在两个文本文件中都有“quick”一词时，the_words[i] 包含一个长度为 6 的字符串，而 word 中的一个包含一个长度为 5 的字符串，两者都包含值很快，因此它没有注册为同一个字符串。为什么其中一根弦比另一根长？

（Ps，我对糟糕的代码质量表示歉意）

编辑1：好的，所以我发现它与每行末尾的 \n 有关。试图找到一种方法来解决这个问题。

编辑2：我设法通过一个简单的for循环摆脱\n

for (int i = 0; i < word_count; i++){
        the_words[i] = malloc(100);
        the_words[i] = redact;
        for (int j = 0; j < strlen(redact); j++){
            if (redact[j] == '\n'){
                redact[j] = '\0';
            }
        }
        redact = strtok(NULL, ", ");
    }

Answer 1

    the_words = malloc(3 * sizeof(char*));
    redact = strtok(buffer2, ", ");
    
    for (int i = 0; i < word_count; i++){
        the_words[i] = malloc(100);
        the_words[i] = redact;
        redact = strtok(NULL, ", ");
    }

两个明显的问题就在这里，

• 您为 the_words 中的 3 个指针分配了空间，但随后您将 word_count 个单词放入其中。所以如果 word_count > 3、您将溢出并为每个单词获得未定义的行为
• ，您分配 100 个字节，然后丢弃该分配，而是将指针存储到 buffer2 中。缓冲区当前包含该单词，但下次您读入该单词时，该单词会发生变化。您应该只使用 the_words[i] = strdup(redact); 来分配适量的内存，并将字符串复制到分配的内存中。