除了打印问题本身之外,您的代码中还有一些错误。我已经纠正了其中的大部分;一个简短的列表是:
-
总是测试
malloc是否成功。
-
fread(text, 100, 10000000, ..) 读取的文本太多。 100 * 10000000 = 1000000000,几乎是一个完整的千兆字节。您只为 10 Mb 分配了足够的内存。
- 您从文本文件中读取数据并将此数据视为字符串。因此,您必须确保数据以
0 结尾,否则printf 和strstr 等函数会在结束后尝试继续读取。
- 您的
rep 变量开始时未初始化,因此您将始终看到一个随机数。
- 始终释放您分配的内存。
也就是说,使用专用函数打印文本的效率稍高一些——如果只是不要在您的 main 中添加太多内容。而且由于它是一个函数,因此您可以根据需要向其中添加任意数量的有用参数;我添加了before 和after 变量,因此您可以改变显示的字符数。
为了更加美观,当在before 字符的最小数量之前找到短语时,此函数会打印正确数量的空格,因此结果排列得很好。另外,由于打印出诸如制表符和换行符之类的字符会弄乱您的输出,因此我将它们替换为?。
诚然,print_range 中有一些重复,但在这种情况下,我是为了清楚,而不是简洁。
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX_LENGTH 10000000
void print_range (char *source_text, int startindex, int before, int after, int phrase_length)
{
int i;
if (before > startindex)
{
for (i=0; i<before-startindex; i++)
printf (" ");
startindex = before;
}
for (i=0; i<before; i++)
{
if (strchr ("\t\r\n", source_text[startindex-before+i]))
printf ("?");
else
printf ("%c", source_text[startindex-before+i]);
}
for (i=0; i<phrase_length; i++)
{
if (strchr ("\t\r\n", source_text[startindex+i]))
printf ("?");
else
printf ("%c", source_text[startindex+i]);
}
for (i=0; i<after; i++)
{
if (!source_text[startindex+phrase_length+i])
break;
if (strchr ("\t\r\n", source_text[startindex+phrase_length+i]))
printf ("?");
else
printf ("%c", source_text[startindex+phrase_length+i]);
}
printf ("\n");
}
int main (int argc, char *argv[]){
char *text = (char *) malloc (MAX_LENGTH);
char *word = argv[1];
int rep = 0;
if (!text)
return -1;
if(argc < 2)
{
printf("Usage: GET <website> | ./word_counter <word>\n");
exit(1);
}
fread(text, 1, MAX_LENGTH, stdin);
text[MAX_LENGTH] = 0;
const char *tmp = text;
do
{
tmp = strstr(tmp, word);
if (!tmp)
break;
print_range (text, tmp-text, 16,16, strlen(word));
rep++;
tmp++;
} while (1);
free (text);
printf ("Word count: %d\n", rep);
return 0;
}
在自己的源代码上运行的结果:
~/Documents $ ./wordcounter printf < wordcounter.c
tindex; i++)????printf (" ");???starti
-before+i]))????printf ("?");???else??
"?");???else????printf ("%c", source_t
before+i]);??}??printf ("{");??for (i=
rtindex+i]))????printf ("?");???else??
"?");???else????printf ("%c", source_t
tindex+i]);??}??printf ("}");??for (i=
_length+i]))????printf ("?");???else??
"?");???else????printf ("%c", source_t
length+i]);??}??printf ("\n");?}??int
argc < 2)??{??? printf("Usage: GET <we
?free (text);???printf ("Word count: %
Word count: 12