C言語で複数人の名前を入力させたい

Question

C言語で名前を読み取らせて最後に表示させるプログラムを作っています。

Tadao
Tomiaki
Syouji
と入力させると

1人目:Tadao
2人目:Tomiaki
3人目:Syouji
と出力させるのが期待している動作です。

以下のプログラムでは1人目の名前を入力した瞬間、

0x5238D3EC (ucrtbased.dll) で例外がスローされました (Cprogramming.c.exe 内): 0xC0000005: 場所 0x01100000 への書き込み中にアクセス違反が発生しました

という文章が表示されました。

#include<stdio.h>

int main(void)
{
	int i,j;
	char name[3][30];  //最大30文字の名前を3個格納する配列

	for (i = 0; i < 3; i++) {
		scanf_s("%s", name[i]);
	}

	for (j = 0; j < 3; j++) {
		printf("%d人目:%s", j + 1, name[j]);
	}

	return 0;
}

どのような点が間違っているのかをどなたか説明していただけないでしょうか。

また、私はC言語の学習を始めて2か月ほどの初心者なので拙い部分も多いかと思います。
質問以外で「ここをこうした方が良い」という改善点がございましたらご指摘していただけると大変嬉しく思います。

よろしくお願いします。

Comments

[y_waiwai]

エラーが出るなら、そのエラーメッセージをそのままで提示しましょう

[cateye]

>char name[3][30]; //最大30文字の名前を20個格納する配列・・・コメントが違うのですが？
ソースを修正する時はコメントの修正も忘れずに・・・・・どっちがバグかわからなくなります。

[Cpro]

申し訳ございません、今後気を付けます。
ご指摘ありがとうございました。

Answer 1

scanf_sをscanfの代わりに使うときのやりかた

ここにのそこらへんの話が載ってますね

そもそも、文字列入力にscanf使うのは禁忌だと思っておきましょう
fgetsつかえばどーです？

Comments

[Cpro]

回答ありがとうございました。
scanfに文字数を指定した場合、期待した動作をしてくれました。

一つだけ質問なんですが、fgetsの引数を
fgets(name[i][30],sizeof(name),stdin);
のようにいろいろと試してみたんですが、上手くいきませんでした。

解決策がありましたら教えていただけると嬉しいです。

[y_waiwai]

fgets(name[i],sizeof(name[0]),stdin);
ですね。
まあ、scanf_sはオーバーフローしないようにマイクロソフトが作ったもんですんで、そんなに嫌うこともないでしょうけど、既存の関数で同様なことができるんで、そっちを使いたいですね

[Cpro]

fgetsの第一引数が文字配列なので確かにそうなりますね。
解決しました！本当にありがとうございました。

Answer 2

scanf_s("%s", name[i]);

scanf_s() の書式ってこれで良いのでしょうか?
使った事無いので、調べたら、

scanf_sをscanfの代わりに使うときのやりかた
scanf_s、_scanf_s_l、wscanf_s、_wscanf_s_l

文字列の場合、文字数の指定が必要な様です。
例えば、 scanf_s("%s", name[i], 30);のように。

[追記]
元のコードを、Visual stdio 2019 でコンパイルしたら、エラーになりました。
で、 scanf_s("%s", name[i], 30); としたら、動きました。ただ、

printf("%d人目:%s", j + 1, name[j]);

最後に、改行コード無いので、一行で出力。 最後に、\n を付けましょう。

Comments

[Cpro]

回答ありがとうございました。
本当ですね、上手く動作しました！

一つだけ質問なんですが、scanfの第2引数は変数のアドレスを指定しているので
scanf_s("%c",name[i],30) = scanf_s("%c",&name[i][0],30)
という解釈でOKですか？？(実際に試してみたら後者の書き方でも上手く動作しました)

[cateye]

>実際に試してみたら後者の書き方でも上手く動作しました
"%c"と1文字入力を指定しているのに、文字列が読めたということでしょうか？・・・そんなんバグじゃん＾＾；
MSのサイトに説明があった https://docs.microsoft.com/ja-jp/cpp/c-runtime-library/reference/scanf-s-scanf-s-l-wscanf-s-wscanf-s-l?view=vs-2019　OKらしい++？

[pepperleaf]

> scanf_s("%c",name[i],30) = scanf_s("%c",&name[i][0],30)
左右は同じですね。
ただ、この書き方は知らなかった。
[scanf 関数の文字幅指定](https://docs.microsoft.com/ja-jp/cpp/c-runtime-library/scanf-width-specification?view=vs-2019) の説明を見ると、最後のNULL('\0')が書き込まれないので、文字列として扱う場合、要注意のようです。

[Cpro]

ごめんなさい、%cではなく%sの間違いでした…
皆様ありがとうございました。

Answer 3

すでに問題自体は解決していますが、質問者にアドバイスを。
この手の質問をすると、scanf() は危険だから使うな、scanf_s() か fgets() を使え、と仰る有識者が何人か出てまいりますが、話半分に聞いとけば良いです。現に、scanf()より安全なはずの scanf_s() を使おうとしたことが例外の原因だった（らしい）という皮肉な結論じゃないですか。

C言語を学び始めたなら、scanf()でもscanf_s()でもfgets() でも、貴方が使いやすい関数を使えばよろしい。scanf() を使うことに遠慮は要らないと思います。scanf() は、入門時期に手ごろな入力関数だから、昔から使われてきたのです。テキストや参考例が scanf() を使っていることも多いので、まずはそのまま動かしてみることが先決。入力手段など、たいていは枝葉末節と割り切れば良い。そのうちもっと良い方法が見つかるでしょうし、scanf() の何が、どう危険なのか、どんな制約があるのかもわかってくるでしょう。それまでは失敗も勉強の内です。

ちなみに、私の手元には scanf_s() が無いので

    for (i = 0; i < 3; i++) {
        scanf("%s", name[i]);
    }
    for (i = 0; i < 3; i++) {
        printf("%d人目:%s\n", i + 1, name[i]);
    }

で、問題なく Tadao, Tomoaki, Syouji の3名を入力・表示できたことをお伝えします。念のために言うと、変数「j」は使いませんでした。

なお、

    scanf_s("%c",  name[i],    30);
    scanf_s("%c", &name[i][0], 30);

は、どちらも書き方間違ってますが、一文字入力に限れば問題なく動作するでしょうね。
今の段階で、正しい書き方は何かを確認しておくのは良いことでしょう。
さらに（ここが本題ですが）、間違いなのに動作するのは何故かという問いもあるわけです。まあ、でも、せっかくだから（笑）この手の問いは解決を先延ばしにしておくと後で楽しみが待ってるかもよ、と言っておきます。

Comments

[Cpro]

回答ありがとうございました。
%cではなく%sの間違いですね…ややこしいことを言って申し訳ないです。
そうなんですか。試行錯誤を繰り返しながら勉強していきます！

[pepperleaf]

確かに危険と言う話は多いですね。
自分も scanf()は使わないですが、単に書式確認が面倒なだけ。fgets()で読み込んで、その後、文字列分解。そうすれば、悩みは少ないと勝手に思っています。セキュリティ的に問題のあるところなら、ともかく、簡単な動作確認なら、入力バッファサイズを 1000バイト位にしておけば、大抵は問題無いと思う。C言語の場合、それ以外にもメモリ破壊の落とし穴は多いので、ご注意を。(昔はそういうトリックの好きな人も居たが)

Answer 4

y_waiwaiさんが仰っているようにscanf()危険な関数です。（そのためにMSはscanf_s()などという関数を作ってしまったｗ）
で、fgets()を使う場合、fgets(name[0],sizeof(name[0]),stdin)などとします。
・・・コンソールから文字列を1個取ってくる関数を作ってしまえばいいのですが・・・
参考：fgets() が改行文字を読み取ると仮定しない