microsoft visual c++

Question

#include "stdafx.h"

int main()
{
char str[100];
int num;
printf("文字列を入力してください。\n");
scanf_s("%s",str);
for(num=0;str[num]!='\0';num++)
{}
printf("文字数は%dです。\n", num);
return 0;
}

という、簡単な文字数を測るプログラミングをしました。
ですが、
'scanf_s': not enough arguments passed for format string.
と「引数が足りない」という、おそらくそういう意味の文が出てきます。
自分なりに引数というのを調べてみたのですが分かりません。
このプログラミングでどこを修正すれば良いかと、できれば引数とは何かを教えていただきたいです。

Answer 1

こんにちは。

scanf_s()はscanf()と微妙にパラメータの与え方が異なります。
"%s"に対しては、バッファとバッファのサイズ（文字単位）を指定する必要が有ります。

scanf_s("%s",str, sizeof(str)/sizeof(str[0]));

とすれば良い筈です。（実は使ったことはないので外していたらごめんなさい。）

sizeof(str)/sizeof(str[0])はちょっと冗長です。が「文字数」であることを強調するためにこのようにしてみました。
お使いのソースでは該当しませんが、1文字が2バイトのような場合もあるからです。

Comments

[can110]

提示されたMicorosoftのページではchar, whar_tともに_countofマクロの利用を例示していますね。
（#include <stdlib.h>が必要ですが）

[Chironian]

can110さん。
おっと、本当ですね。見落としてました。フォローありがとうございます。

-tataさん。
can110さんの指摘通り、ソースの頭で#include <stdlib.h>しておいて、次のようにした方が良いです。

scanf_s("%s",str, _countof(str));

正規ドキュメントの推奨どおりに記述した方が無用なトラブルに巻き込まれにくいですから。

[-tata]

scanf_s("%s",str, sizeof(str)/sizeof(str[0]));
とすれば直りました。
ありがとうございます。
よろしければこの文はどういう処理がされるのか教えていただきたいです。。。m(_ _)m

[Chironian]

動作したとのこと、良かったですね。
sizeof(str)/sizeof(str[0])は配列の要素数を取り出す定番テクニックです。
http://kaworu.jpn.org/c/%E9%85%8D%E5%88%97%E3%81%AE%E8%A6%81%E7%B4%A0%E6%95%B0%E3%82%92%E8%AA%BF%E3%81%B9%E3%82%8B

C++11で多次元配列にも対応できる より強力なもの(std::extent)が出てますが、ちょっと難しいかも知れません。
https://cpprefjp.github.io/reference/type_traits/extent.html

Answer 2

先頭に

#pragma warning(disable:4996)

を入れてください。MS独自の仕様をキャンセルします＾＾
追記：
マイナス評価が付きましたが、VisualStudioでC,C++の勉強をするなら本でもネットでもMSの仕様でコードが書かれていない事が多いので私はやり易いと思っています＾＾

Comments

[can110]

先頭に#pragma warning(disable:4996)を追加したビルド結果。
--
1>c:\hoge\test.cpp(8): warning C4473: 'scanf_s': 書式文字列として渡された引数が不足しています
1> c:\hoge\test.cpp(8): note: プレースホルダーとそのパラメーターには 2 の可変個引数が必要ですが、1 が指定されています。
1> c:\hoge\test.cpp(8): note: 不足している可変個引数 2 が書式文字列 '%s' に必要です
1> c:\hoge\test.cpp(8): note: この引数はバッファー サイズとして使用されます
1> test.vcxproj -> c:\hoge\Debug\test.exe
1> test.vcxproj -> c:\hoge\Debug\test.pdb (Full PDB)
========== すべてリビルド: 1 正常終了、0 失敗、0 スキップ ==========
--
デバッグ実行すると以下のように異常終了します。
--
0x100AE224 (ucrtbased.dll) で例外がスローされました (test.exe 内): 0xC0000005: 場所 0x00B00000 への書き込み中にアクセス違反が発生しました
--
どうしたらよいのでしょうか？

[can110]

きつい言い方になったかもしれませんがすみません。お伝えしたいことは、むやみに警告を止めるのではなく一般的なC言語の説明ページで紹介されているscanfを使うように勧めたほうがまだましだということです。
ちなみにscanfを使った場合、100文字以上入力すると「Stack around the variable 'str' was corrupted.」となりますが、「範囲外アクセスしたからだよ（結果はどうなってもしりませんよ）」と説明しやすいです。

[kyunta]

scanf_sがMSの仕様ですので多くの所で使われていませんでした。(標準のscanfが使われているがVisualStudioでは4996を入れないとコンパイルが通りません。）
私が始めた時にこればかりではなく他もMSの仕様を強いられたために苦労しましたので余計な苦労はして欲しくないなという気持ちで回答させていただきました。

[kyunta]

前後してしまいましたが、can110さんの気持ちは分かりました。

[can110]

VS2015でC++コンソールアプリ作成した場合はscanfは「warning C4996」扱いでビルドできますが、ソリューション設定によってはエラー扱いになりえますね。
#pragma warning(disable:4996)を利用する意図について了解しました。レスありがとうございます。