セグメンテーションフォールトC

Question

で

のファイルを読み込むとき

int textdump(const char *filename) 
{ 
    int count = 0; 

    FILE *file; 

    file = fopen(filename,"r"); 
    char letter; 
    while(!feof(file)) { 
     letter = fgetc(file); 
     if (isprint(letter)) { 
      printf("%c",letter); 
      count++; 
     } 
     else { 
      printf("?"); 
     } 

    } 
    fclose(file); 
    return count; 
} 

Answer 1

オーケー...全く分からないなぜ、どのように私はそれをデバッグすることができ、私はCの文字でのファイルの文字を読み取ろうと私はセグメンテーションフォールトを取得するので、あなたのコードには、初心者のコードでよくある間違いがあります。ここで

は同じラインに沿ってそれを行う方法ですが、うまくいけば、エラーを修正：

size_t textdump(const char *filename) 
{ 
    size_t count = 0; 

    FILE * const file = fopen(filename, "rt"); 
    if(file == NULL) 
     return 0; 
    while (1) { 
     const int ch = fgetc(file); 
     if(ch == EOF) 
     break; 
     if(isprint(ch) { 
     putc(ch, stdout); 
     ++count; 
     } 
     else 
     putc('?', stdout); 
    } 
    fclose(file); 
    return count; 
} 

この：fopen()は有効であるファイルポインタに頼る前に、成功

• チェックが。
• intあるfgetc()の戻り値のための適切な型を使用します。
• while(!feof()アンチパターンを修正します。
• printf()よりも軽く、単一文字の出力用fputc()を使用します。
• size_tを使用してカウントを表します。

私は、これは最適で、それはまだ単一文字は、インスタンスのための読書ん（しかし、それはあまりにも悪いことではないはずですので、バッファリング）とは言いませんよ。しかしそれはより良いはずです。