質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

連結リスト

連結リストとは、データ構造のひとつであるリストの中で、要素が前後の要素の情報を持つことで、要素が連結(リンク)しているリストの事を呼びます。

Q&A

解決済

2回答

696閲覧

C言語 線形リスト/連結リストについて

taymiyk

総合スコア11

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

連結リスト

連結リストとは、データ構造のひとつであるリストの中で、要素が前後の要素の情報を持つことで、要素が連結(リンク)しているリストの事を呼びます。

0グッド

0クリップ

投稿2020/05/02 00:09

C言語の線形リストについての質問になります。
現在C言語を学び始めて2ヶ月目の者です。

文字配列を一文字ずつ順番に線形リストに挿入していきたいのですが、現在以下のコードでSegmentation Faultが出てしまっている状態です。

int main のwhile文の中が問題であるのではないかと睨んでいるのですが、どう書けば正常に動くのかが解らない状態です。

ご指導いただければ幸いです。
よろしくお願い致します。

C

1typedef struct node 2{ 3 char c; 4 struct node *next; 5}node_t; 6 7 8void printlist(node_t *head) 9{ 10 node_t *temp = head; 11 12 while(temp != NULL) 13 { 14 printf("%c - ", temp -> c); 15 temp = temp -> next; 16 } 17} 18 19node_t *create_new_node(char c) 20{ 21 node_t *result; 22 23 result = malloc(sizeof(node_t)); 24 result -> c = c; 25 result -> next = NULL; 26 return (result); 27} 28 29int main() 30{ 31 int i; 32 node_t *temp; 33 node_t *head; 34 node_t *prev; 35 36 i = 0; 37 char name[4] = "JOHN"; 38 head = NULL; 39 while (name[i]) 40 { 41 prev = head; 42 temp = create_new_node(name[i]); 43 while(prev != NULL) 44 { 45 prev = prev->next; 46 } 47 prev->next = temp; 48 i++; 49 } 50 51 printlist(head); 52}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

まず、今回のエラーを過ぎてからの話ですが、入門レベルの間違いとしては、

C

1 char name[4] = "JOHN";

と4バイトしか確保していないのに、5バイト入れようとしているところ。

エラーの原因としては、書き換えのために末尾ノードを探すところで、何が末尾かの判断がおかしい。
末尾ノードは、nextNULLのものです。

また、ヘッドは単なるポインターじゃなくてノードにしておかないと、先頭への追加と、先頭以外への追加で、判断して異なる処理をする必要があります。

C

1int main() 2{ 3 int i; 4 node_t *head; 5 node_t *prev; 6 7 char name[] = "JOHN"; 8 9 head = create_new_node(0); 10 for(i=0; name[i]; i++) 11 { 12 for(prev = head; prev -> next; prev = prev->next) ; 13 prev->next = create_new_node(name[i]); 14 15 } 16 17 printlist(head->next); 18}

繰り返しごとに変数を順次更新するようなループはforで書きます。

投稿2020/05/02 01:13

otn

総合スコア84423

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

C

1 while(prev != NULL) // このloopを抜けるとprev == NULL だから 2 { 3 prev = prev->next; 4 } 5 prev->next = temp; // NULL->next ???

投稿2020/05/02 00:21

episteme

総合スコア16614

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

taymiyk

2020/05/02 01:02

解けました。ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問