連結リストの表示関数を動かしたい

Question

前提・実現したいこと

連結リストの表示関数がうまく動かないです。

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

リスト表示がされません。

該当のソースコード

#include <stdio.h>
#include <stdlib.h>

struct CELL{
    int value;
    struct CELL *next;
};

struct CELL *header1;

/*先頭に追加*/
void insert(int n,struct CELL *header)
{
    struct CELL *new;
    if ((new=malloc(sizeof(struct CELL)))==NULL){
        printf("メモリ不足");
        exit(1);
    }
    new->next=header;
    new->value=n;
    header=new;
    
}

void print_list(struct CELL *header)
{
    struct CELL *p;
    
    printf("リスト：");
    for (p=header;p!=NULL;p=p->next){
        printf("%d,",p->value);
    }
}

int main(void)
{
	int n;
	int count=10;  
	while(count){
		printf("追加する要素："); scanf("%d",&n);
		insert(n,header1);
		print_list(header1); putchar('\n');
		count--;
	}
}

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

Xcode Version 11.2 (11B52)

Answer 1

void insert(int n,struct CELL *header)

この関数で、headerを更新しているようですが、呼び出し側に、更新された headerは、戻りません。引数のみで行うならば、2重ポインタにする、または、関数の戻り値としましょう。
例えば、関数の定義を CELL * insert(int n,struct CELL *header)とし、return header; とする。 呼び出し側も合わせて変更。

少し訂正: (Cだったので、 struct が必要だった)
関数宣言は、
struct CELL * insert(int n,struct CELL *header)
呼び出す方は、 header1 = insert(n,header1); ですね。

Comments

[VanS]

解決できました！ありがとうございます！
引数としてのポインタについてもう一度学習し直してみます。

Answer 2

表示関数print_list()がうまく動かない、のではなく、
追加の関数insert()が期待する動作をしていないようです。

実引数として渡されたheader1の「中身」を書き換えなければいけないのではないですか?

Comments

[VanS]

仮引数*headerにして、実引数header1にしてもinsert内のheaderはheader１として動かないのですか？

[VanS]

リストを二つ作成したい場合、header1、header2といった具合に実引数を変えたい場合はどうしたらいいのでしょうか？

[thkana]

> header１として動かないのですか
..として動く..とはどうなることを期待していますか?

そもそも、insert()関数によって「なにが」「どうなる」べきなのかはちゃんと把握していますか?

> header1、header2といった具合に実引数を変えたい場合は
特に悩む話じゃないです。ちゃんと作ってあれば、header1, header2を与えればそれなりに動く...はずです。