我用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
}

最佳答案

代码中存在多个问题:
不要在正确的位置测试文件的结尾:如果返回了cc,您应该立即退出循环,而不是在下一次迭代中处理退出。经典的c语言习惯用法是:

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

为了便于移植和可读,您应该使用getc()fromEOF检查字节是否为字母,并避免将撇号的值硬编码为EOF,而是使用isalpha()
在将字节存储到<ctype.h>数组中时,可能会出现缓冲区溢出。您应该将偏移量与缓冲区大小进行比较。
您不可以空终止39,因此'\''函数无法确定第一个字符串的长度。该行为未定义。
不检查是否提供了命令行参数。
第二个测试是多余的:您可以只使用tempWord子句。
在打印tempWord的内容时,由于缺少空终止符,您的行为未定义。这解释了意外的行为,但你可能会有更糟糕的后果。
compareWord()只打印一个c字符串,使用else
tempWord[]函数本质上与printWord相同。
以下是一个简化和更正版本:
#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正确地评论了上述代码不处理两种特殊情况:
太长的单词在输出中被截断。
如果最后一个字正好落在文件末尾,则省略它。
我还意识到程序不能处理在命令行上传递长单词的情况。
以下是一种没有缓冲区的不同方法,它可以修复这些缺点:
#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;
}

09-25 20:58