如何让 fgetc 在循环内打印文件的所有字符

Question

我正在使用 fgetc，想知道如何让我的循环打印该行部分一次，然后打印前 16 个字符。

这是我的代码：

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

    if((fp = fopen(argv[1], "rb+")) == '\0')
    {
        perror("fopen");
        return 0;
    }

    getchar(fp);

    return 0;
}

void getchar(FILE *fp)
{
    int c;
    size_t linecount = 0;

    while((c = fgetc(fp)) != EOF)
    {
        printf("line : ");
        printf("%c ", c);
        linecount++;

        if(linecount == 16)
        {
            printf("\n");
        }
    }
}

我希望我的输出类似于：

 line: ? 5 d s d a d 0 0 0 0 0 0 0 0 0 0 0
 line: a d s x c v d 0 0 0 0 0 0 0 0

我尝试使用此 for 循环打印字符：

for(i = 0; i < 16; i++)
{
   printf("%c ", c);
}

但这没有得到其他字符，因为循环仍然集中在第一个字符上

Answer 1

首先， 中已经定义了一个名为 getchar() 的标准函数 - 您必须更改函数的名称，否则您可能会得到一些非常糟糕的结果。奇怪的行为。

其次，在我看来，你想做的是：

• 打印出 "line: ";
• 读取一个字符并打印它，直到文件末尾或打印完 16 个字符；
• 打印出“\n”。

这将转换为代码形式：

printf("line : ");

while (linecount < 16 && (c = fgetc(fp)) != EOF)
{
    printf("%c ", c);
    linecount++;
}

printf("\n");

Answer 2

if((fp = fopen(argv[1], "rb+")) == '\0')

ASCII NUL 字符与 NULL 指针不同。这应该是 == NULL 或 if(!(fp = fopen(argv[1], "rb+"))) /* ... */，以两者为准你认为更容易阅读。此子句中的 return 0 可能应该是 return 1，因此在标准管道中使用您的程序会更容易。 （程序中的错误通过非零返回值来明确。是的，这有点倒退，但是有 充分的理由。）

void getchar(FILE *fp)

哎呀。 标头中已存在 getchar(3)。重新定义它并不是一个好主意。 （它现在可能有效，但将来的修改可能会由于不明显的原因而完全失败。）

我已经重新设计了 gc() 例程中的循环，它应该可以满足您的需要：

#include <stdio.h>

void gc(FILE * fp);

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

    if ((fp = fopen(argv[1], "rb+")) == NULL) {
        perror("fopen");
        return 1;
    }

    gc(fp);

    return 0;
}

void gc(FILE * fp)
{
    int c;
    size_t linecount = 0;

    while ((c = fgetc(fp)) != EOF) {
        if (linecount == 0)
            printf("line : ");

        printf("%c ", c);

        if (++linecount == 16) {
            printf("\n");
            linecount = 0;
        }
    }
}

自行运行时的输出：

$ ./printer printer.c | head
line : # i n c l u d e   < s t d i o . 
line : h > 

 v o i d   g c ( F I L E 
line :   *   f p ) ; 

 i n t   m a i 
line : n ( i n t   a r g c ,   c h a r 
line :   * a r g v [ ] ) 
 {