Question

我用C编写了一个程序，预期的结果应该是：

$ cat poem.txt

Said Hamlet to Ophelia,
I'll draw a sketch of thee,
What kind of pencil shall I use?
2B or not 2B? 

$ ./censor Ophelia < poem.txt

Said Hamlet to CENSORED,
I'll draw a sketch of thee,
What kind of pencil shall I use?
2B or not 2B?

但我得到了这个：

$ ./censor Ophelia < poem.txt

Said Hamlet tomlet CENSORED,
I'lllia drawlia arawlia sketcha ofetcha theecha,
Whatcha kindcha ofndcha pencila shallla Ihallla usellla?
2Bsellla orellla notllla 2Botllla?

我使用tempWord存储每个单词并将其与需要审查的单词进行比较。然后我使用tempWord[0]='\0'重置临时字符串，以便我可以进行另一次比较。但它似乎不起作用。有人可以帮忙吗？

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

int compareWord(char *list1, char *list2);
int printWord(char *list);

int main(int argc, char *argv[]) {

    int character = 0;

    char tempWord[128];
    int count = 0;

    while (character != EOF) {
        character = getchar();

        if ((character <= 'z' && character >= 'a') ||
            (character <= 'Z' && character >= 'A') ||
            character == 39) {              
            tempWord[count] = character;
            count++;
        } else {
            if (count != 0 && compareWord(tempWord, argv[1])) {
                printf("CENSORED");
                count = 0;
                tempWord[0] = '\0';
            }

            if (count != 0 && !compareWord(tempWord, argv[1])) {
                printWord(tempWord);
                count = 0;
                tempWord[0] = '\0';
            }

            if (count == 0) {
                printf("%c", character);
            }
        }
    }
    return 0;
}

int printWord(char *list) {

    // print function
}

int compareWord(char *list1, char *list2) {
         // compareWord function
}

Answer 1

您的代码中存在多个问题：

您没有在正确的位置测试文件结尾：如果getc()返回EOF，您应该立即退出循环，而不是处理EOF并退出下一次迭代。这样做的经典C语言是：
```
while ((character = getchar()) != EOF) {
    ...
```
为了便于携带和阅读，您应该使用isalpha()中的<ctype.h>检查字节是否为字母，并避免将撇号值的值硬编码为39 ，请改用'\''。
将字节存储到tempWord数组时，可能存在缓冲区溢出。您应该将偏移量与缓冲区大小进行比较。
您没有空终止tempWord，因此compareWord()函数无法确定第一个字符串的长度。行为未定义。
您不检查是否提供了命令行参数。
第二项测试是多余的：你可以使用else条款。
由于缺少空终止，因此在打印tempWord[]的内容时会有未定义的行为。这解释了意外行为，但可能会产生更糟糕的后果。
printWord只打印一个C字符串，使用fputs()。
compWord功能与strcmp(a, b) == 0基本相同。

以下是简化版本：

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

int main(int argc, char *argv[]) {
    char tempWord[128];
    size_t count = 0;
    int c;

    while ((c = getchar()) != EOF) {
        if (isalpha(c) || c == '\'') {
            if (count < sizeof(tempWord) - 1) {
                tempWord[count++] = c;
            }
        } else {
            tempWord[count] = '\0';
            if (argc > 1 && strcmp(tempWord, argv[1]) == 0) {
                printf("CENSORED");
            } else {
                fputs(tempWord, stdout);
            }
            count = 0;
            putchar(c);
        }
    }
    return 0;
}

编辑： chux正确地评论说上面的代码没有处理2个特殊情况：

太长的单词会在输出中被截断。
如果它完全落在文件的末尾，则省略最后一个单词。

我也意识到程序不处理命令行传递的长字的情况。

这是一种不同的缓冲方法，可以解决这些缺点：

#include <ctype.h>
#include <stdio.h>

int main(int argc, char *argv[]) {
    const char *word = (argc > 1) ? argv[1] : "";
    int count = 0;
    int c;

    for (;;) {
        c = getchar();
        if (isalpha(c) || c == '\'') {
            if (count >= 0 && (unsigned char)word[count] == c) {
                count++;
            } else {
                if (count > 0) {
                    printf("%.*s", count, word);
                }
                count = -1;
                putchar(c);
            }
        } else {
            if (count > 0) {
                if (word[count] == '\0') {
                    printf("CENSORED");
                } else {
                    printf("%.*s", count, word);
                }
            }
            if (c == EOF)
                break;
            count = 0;
            putchar(c);
        }
    }
    return 0;
}

Answer 2

tempWord[0] = '\0';

它不会将变量重置为null。它只是分配了＆＃39; \ 0＆＃39;到第一个位置。但是分配的值仍然只在内存中。只有第一个位置被分配给＆＃39; \ 0＆＃39;。因此，要重置字符数组，请尝试以下操作。

memset(tempWord, 0, 128);

添加以上行而不是tempWord[0] = '\0'。

这也将解决您不需要添加＆＃39; \ 0＆＃39;在每个单词的结尾处。这本身就行。但是你第一次必须使用相同的memset函数重置字符数组。在进入循环之前，您必须使用memset函数将tempWord设置为null。

Answer 3

使用tempWord [0] =＆＃39; \ 0＆＃39;不会重置整个数组，只是第一个元素。查看代码，有两种方法可以继续，使用memset重置整个数组：

memset(tempWord, 0, sizeof tempWord);

或

memset(tempWord, 0, 128);

（或者你只能用最后一个单词的大小来清除它，也需要你已经包含的string.h），

或者您可以在“当前单词”的长度之后设置元素。成为＆＃39; \ 0＆＃39; （例如，如果当前单词为the，则设置tempWord[3]='\0'，因为strlen检查字符串直到null char），这可以放在那些2 ifs之前检查字符串是否相等，你的新while循环将如下所示：

{
        character = getchar();

        if((character<='z' && character>='a')||(character<='Z' && character>='A')||character == 39)
        {               
            tempWord[count]=character;
            count++;

        }else {
            tempWord[count]='\0';

            if(count!=0 && compareWord(tempWord, argv[1]))
            {

                printf("CENSORED");
                count=0;


            }

            if(count!=0 && !compareWord(tempWord, argv[1]))
            {

                printWord(tempWord);
                count=0;

            }

            if (count==0)
            {
                printf("%c", character);
            }
        }
    }

（它有效，经过测试）

tempWord [0] ='\ 0'不会以某种方式重置字符串

3 个答案: