C言語 visual studio 2015 動作停止について

Question

C言語の勉強をしている学生です
visual studio2015を使って実行しているのですが、

#include <stdio.h>

int main(void) {

	int a, b;
	char op;
	scanf_s("%d %c %d", &a, &op, &b);
	if (op == '+') {
		printf("%d", a + b);
	}
	else if (op == '-') {
		printf("%d", a - b);
	}
	else if (op == '*') {
		printf("%d", a * b);
	}
	else if (op == '/') {
		printf("%d", a / b);
	}
	else {
	}
	return 0;
}

といった単純計算機をつくって実行したところ
数字 演算子 数字 を入力し、エンターを押したところで
動作が停止します。プログラムに問題があるのかと思いますがどこに問題があるかわかりません。
初歩的な質問ですが、教えていただけたらありがたいです。

Comments

[archiver]

「動作が停止します」とありますが、具体的にはどのようになりますか？

[Takumi0106]

すいません、返信おそくなりました。
デバッグなしで開始をして、数字　演算子　数字を入力してエンターを押すと
動作を停止しました。と表示されます。

Answer 1

scanf と scanf_sは実は引数の仕様が違います。
違うんですが引数の指定の仕様上、間違っててもコンパイル時にエラーになりません。

scanf_s、_scanf_s_l、wscanf_s、_wscanf_s_l

scanf および wscanf とは異なり、scanf_s および wscanf_s では、c、C、s、S の各型、
または [] で囲まれた文字列コントロール セットのすべての入力パラメーターに対して
バッファー サイズを指定する必要があります。
バッファー サイズ (文字単位) は、バッファーまたは変数のポインターの直後に
追加パラメーターとして渡されます。

つまり
scanf_s("%d %c %d", &a, &op, &b);
ではなく、
scanf_s("%d %c %d", &a, &op,1, &b);
です

Comments

[Takumi0106]

ありがとうございます。回答のとおりに訂正したところ、動きました。
引数の仕様が違うとはしらなかったです。勉強になりました。

Answer 2

下記いずれかに修正すると期待通り動きますか？

scanf("%d %c %d", &a, &op, &b);

scanf_s("%d %c %d", &a, &op, 1, &b);

Comments

[Takumi0106]

scanf_sのほうは、その通りで修正するとうごきました。
scanfを使うと、コンパイルエラーになり、
エラー C4996 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
と表示されます。
それがよくわからないのですが、よければ教えていただいてもよろしいでしょうか？

[yohhoy]

エラーメッセージの通り、ソースコード冒頭（#includeより前）に _CRT_SECURE_NO_WARNINGS マクロを定義すればコンパイル可能です。

scanf関数ファミリは、誤使用によるバッファオーバーランなどの脆弱性を作りこみやすく、Microsoft社の判断により"危険な"関数としてマークされています。エラーメッセージはその旨を指摘するとともに、代替案(scanf_s)を使うか自己責任(マクロ定義)でそのまま使うかをプログラマに伝えています。