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

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

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

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

Q&A

解決済

2回答

1897閲覧

リスト構造 strcpyでエラーが起きる

kotue_245

総合スコア12

C

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

0グッド

0クリップ

投稿2019/08/04 12:38

前提・実現したいこと

c言語でリストを利用して、入力された文字列の ポインタをセルに格納するプログラムを考えています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

strcpy_s(g->s, sizeof(g->s), buf); 0xC0000005: 場所 0xCDCDCDCD への書き込み中にアクセス違反が発生しました

該当のソースコード

include<stdio.h>
include<stdlib.h>
include<string.h>

struct LIST {
struct LIST * next;
char *s;
};

struct LIST * addList(struct LIST ** pstart) {
struct LIST *p, *ne;
p = *pstart;
if ((ne = (struct LIST *) malloc(sizeof(struct LIST))) == NULL) {
return NULL;
}
ne->next = NULL;
if (p != NULL) {
while (p->next != NULL) {
p = p->next;
}
p->next = ne;
}
else {
*pstart = ne;
}
return ne;
}

int main() {
char buf[512];
struct LIST *start = NULL;
struct LIST *g;
while(1) {
gets_s(buf, sizeof(buf));
g = addList(&start);
strcpy_s(g->s, sizeof(g->s), buf);
}
return 0;
}

試したこと

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

VS2017を使用
includeは文字が大きくなるため#を省いています。

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

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

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

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

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

cateye

2019/08/04 12:53

〉includeは文字が大きくなるため#を省いています。 <code>の中にソースを貼り付けて下さい。
guest

回答2

0

ベストアンサー

文字列を格納する領域がありません。
strcpy_s(g->s, sizeof(g->s), buf);のまえに
g->s= maloc(strlen(buf)+1);
また、
strcpy_s(g->s, sizeof(g->s), buf);は
strcpy_s(g->s, strlen(buf)+1, buf);

投稿2019/08/04 12:51

cateye

総合スコア6851

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

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

kotue_245

2019/08/04 12:56

ありがとうございます。 解決しました‼
kotue_245

2019/08/04 13:48

すみません 入力した回数をcountに入れておいたときに for (i = 0; i<count; i++) { printf("%s\n", ?); } このようにして格納したものを表示したいのですが ?に何を入れればよいでしょうか
cateye

2019/08/04 20:11 編集

g->sのこと? printf("%s\n", g->s); リスト操作だから、g=start;から初めてnextがNULLになるまでかな? while()の脱出が出来ない。 g = addList(&start);もおかしいよ。このままでは、リストが成長しません。・・・ちょっと考えてみましょう。 あと、最後には確保した領域の開放も必要。 参考:http://capm-network.com/?tag=C%E8%A8%80%E8%AA%9E%E3%82%A2%E3%83%AB%E3%82%B4%E3%83%AA%E3%82%BA%E3%83%A0-%E9%80%A3%E7%B5%90%E3%83%AA%E3%82%B9%E3%83%88
guest

0

ne = (struct LIST *) malloc(sizeof(struct LIST))

では、ne->sの内容、指し先はどうなっているのか考えてみよう

#んでついでに、sizeof(g->s)ってのはポインタのサイズでしかないですぜ

投稿2019/08/04 12:47

y_waiwai

総合スコア87719

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

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

kotue_245

2019/08/04 12:57

ありがとうございます。 よく考えればそうですね。 解決しました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問