C言語：scanf_s関数で文字を入力するとエラーが起こる

前提・実現したいこと

C言語で、文字を入力してその文字を表示するプログラムを作っていたのですが、
日本語を入力するとエラーが起こり、2バイト文字がダメなのかと思い英語を入力してもダメでした。
エラーメッセージは「例外がスローされました」です

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

例外がスローされました

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

該当のソースコード

C
1#include <stdio.h>
2
3int main(void)
4{
5	/* 変数の定義 */
6	char moji[1000];
7
8	/* 文字列の入力 */
9	printf("文字列を入力してください。\n");
10	scanf_s("%s", moji);
11 
12	/* 文字列の出力 */
13	printf("%s\n", moji);
14
15	return 0;
16}

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

VisualStudio2017を使用しています。

行動規範の内容に同意します

回答1件

ベストアンサー

scanf_sの定義を確認してください

フォーマット%c%C%s%Sおよび[]の場合はバッファの要素数が必要になります。

scanf_s("%s", moji, 1000);

投稿2018/02/12 05:54

asm

総合スコア15147

episteme

2018/02/12 05:58

そっかー、可変長引数だから足りなくてもエラーにならんもんなー

asm

2018/02/12 06:09

セキュリティ目的にscanf拡張しても、結局はC言語の仕様に阻まれてイマイチ堅牢とは言い切れないですね… %sの場合幅指定を必須化した方がマシだったのではないかと思ったりします。

退会済みユーザー

2018/02/12 06:12

なるほど、()内にも数を描く必要があるのですね。変数の定義の際の[]内の数とはどういった違いがあるのですか？

退会済みユーザー

2018/02/12 06:19

なんとなく理解しました []で要素空を指定する際は()の中でも指定する必要があるという認識で間違いないですか？

asm

2018/02/12 06:27

> 変数の定義の際の[]内の数とはどういった違いが同じ数値を入れてやると上手くいくように設計されています。読み取った文字列要素が指定数より大きければ書き込みを行わない事でバッファオーバーフローを防ぎます。 stdlib.hを用いて、_countofマクロを用いるのがベストでしょう。https://msdn.microsoft.com/ja-jp/library/ms175773.aspx

rubato6809

2018/02/12 06:46 編集

scan_f("%s", moji);とした時、”%s"は別にして、scan_f() に渡る情報は moji配列の先頭アドレスだけです。要素数1000という情報が自動的に渡ることは無いのです。それがCの引数の基本ルールです。なので、scan_f("%s", moji, 1000); と、明示的に渡す必要があります。付け加えると、char moji[1000]; と、配列を宣言した時の 1000 は、配列をメモリに割り当てる時にしか使われません。moji配列の「属性？」のように付いてまわることはない、のです。

退会済みユーザー

2018/02/12 06:45

>>asm様そんな大雑把な感じなんですね... まだ見たことないライブラリです...これから勉強していきます >>rubato6809様なるほど。そういうものとして覚えるしかなさそうですね... 皆様回答ありがとうございました。

asm

2018/02/12 06:51

> []で要素空を指定する際は()の中でも指定する必要があるという認識で間違いないですか？残念ながら何を言っているのかさっぱりわかりません。要素空とはなんでしょうか？要素数の指定が必要な場合については定義をご確認ください

行動規範の内容に同意します