ファイル読み込みについて

前提・実現したいこと

ファイル読み込みを行っているコードを読み取りたいです。具体的には、以下のコードです。
アルファベットをつけている行のコードが理解できません。

A...ファイルポインタに、buff のアドレスを代入している。
B...line[line_no++] = buff; ではだめなのか、この行の意味が理解できません。
C...このif文の中で何を行っているのか、さっぱり理解できません。
D...なぜ最後に0 NULLを代入しているのか

どなたか教えていただけますでしょうか。

ソースコード

C
1int main(){
2   int c;
3   char buff[1000];
4   char *line[50];
5   int line_no=0;
6   char *p;
7   int i;
8   FILE *fp;
9
10   p　=　buff;                // A
11   line[line_no++] = p;       // B
12
13   fp=fopen("sample.txt","r");
14
15   while((c=fgetc(fp)) != -1){
16       *p++ = c;             
17       if(c == 0x0a){        // C
18            *p++  = 0;
19            line[line_no] = p;
20            line_no++;
21      }
22   }
23  
24    *p = 0;                  // D
25    fclose(fp);
26  
27    return 0;
28}

また、このプログラムは、バッファリングをしているといえるでしょうか。（バッファリングの意味が広すぎてわからないです。）

行動規範の内容に同意します

回答2件

あんまり、良いコードとは思えないですが、、、

B...line[line_no++] = buff; ではだめなのか、この行の意味が理解できません。

どちらでも同じ。

C...このif文の中で何を行っているのか、さっぱり理解できません。

入力した文字が改行 ('\n' とも) ならば、です。'\n' == '\x0a' で改行文字のこと。

D...なぜ最後に0 NULLを代入しているのか

C言語では、文字列は 0 ('\x0')で終わります。これが無いと、文字列の終わりが分かりません。

投稿2021/11/14 10:22

pepperleaf

総合スコア6385

_kei

2021/11/14 10:29

ご回答ありがとうございます。どのあたりを見て良くないコードだと思われたのでしょうか。このコードは、ファイルの中身を表示するコードのようです。改善の余地があれば教えていただけますでしょうか。また、質問文に追加した質問についてはどのようにお考えでしょうか。よろしくお願いいたします。

pepperleaf

2021/11/14 12:21 編集

既にコメントがついていますが、プログラムではバッファリングしてません。ただ、大抵の処理系では、バッファリングされているので、実動作してはバッファリングとなるでしょう。良くないと思った理由は、 - 一つのバッファ(char buff[1000];)を分割して使っている。 - ++の後置演算子が単独でない箇所があり、慣れない人が間違える。 - 0x0a で比較しているが、文字なので、'\x0A' または、'\n' の方が良いのでは? と言ったところです。まあ、間違いでもなく、意見の分かれるところですが。

_kei

2021/11/14 11:48

なるほど、納得することができました。ご教授いただきありがとうございます。ベストアンサーを差し上げたいところですが、申し訳ございません。

pepperleaf

2021/11/14 12:25

上記、コメント修正。 > ++の後置演算子が単独でない箇所これは、一つの式に複数の機能(意味)があると、間違えやすいし、修正ミスになりやすいという事です。特に、C言語で、マクロ(#define)が使われると、発見が困難です。

行動規範の内容に同意します