C言語での簡単な構造体リストの実現

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
C言語で一番シンプルな構造体リストを実現しようとしているのですが、できずに困っています。

発生している問題・エラーメッセージ

エラーメッセージ

Segmentation fault (core dumped)

該当のソースコード

#include <stdio.h>
#include <malloc.h>

typedef struct _Node {
int x;
struct _Node *next;
} Node;

Node *gen_node(void);

int main(void){
printf("flag0");//デバッガ
Node *head, *p;
// headはリストの先頭を，pは新規ノードを指す
printf("flag1"); //デバッガ

// リストの作成
head=NULL;
while(p=gen_node(),scanf("%d",p->x)!=EOF){
printf("flag2"); //デバッガ
p->next=head; // 新規ノードのnextは，これまでのリストの先頭を指す
head=p; // リストの先頭を新規ノードに更新
}
printf("flag3"); //デバッガ
// リストの表示
p=head;
while(p!=NULL){
printf("%d\n",p->x);
p=p->next;
}

return 0;
}

Node *gen_node(void)
{
return (Node *)malloc(sizeof(Node));
}

//実行時には、このプログラムをexeとおいて、
./exe < test.txt
でtest.txtを読み込ませた。
test.txtは以下の通り。

1
4
6
8
9
11
13
16
21
22
27
28
31
35
37
39
41
44
48

試したこと

printf("flag")を入れてまずどこがエラーの原因になっているか絞ろうとした。
結果 → mainの一番最初に入れたflag0も表示されないので、もはやどこに問題があるのかもわからない。

補足情報（FW/ツールのバージョンなど）

CentOS Linux 7を使っています。

Answer 1

while(p=gen_node(),scanf("%d",p->x)!=EOF){

scanfの第二引数以降はアドレスを渡すので、scanf("%d", &p->x) が正しいです。

コンパイラの警告オプションをオンにしておくと、ちゃんと叱って貰えます。
例えばこんなふうに。

prog.c:19:28: warning: format '%d' expects argument of type 'int *', but argument 2 has type 'int' [-Wformat=]
   19 | while(p=gen_node(),scanf("%d", p->x)!=EOF){
      |                           ~^   ~~~~
      |                            |    |
      |                            |    int
      |                            int *

---

printf("flag")を入れてまずどこがエラーの原因になっているか絞ろうとした。

結果 → mainの一番最初に入れたflag0も表示されないので、もはやどこに問題があるのかもわからない。

デバッグプリントを用いること自体は非常に良いアイデアです。
しかしこのようなケースでは、バッファに文字列を溜め込まないよう工夫しなければなりません。

普通は printf("flag\n") などと改行するだけでも改善するでしょう。

コードの貼り方について

teratailには、コードを見易く表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

Comments

[asm]

> 普通は printf("flag\n") などと改行するだけでも改善するでしょう。
putsでいいですし、デバッグプリントならばfprintf(stderr, "flag");を使うと
stderrは既定でバッファリングされない事も覚えておくと便利ですね。

[wataamenoarashi]

丁寧な解説本当にありがとうございます!

[LouiS0616]

@asm さん
なるほど、確かにこのようなときは標準エラーに吐く方がより適切ですね。
コメントありがとうございます。