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