strcpyとstrcpy_sの扱い

Question

strcpy関数をどうすれば使えるようになるか教えて頂きたいです。C初心者でVisualStudio2013を使っています。

#include<stdio.h>
#include<string.h>

void main(){
char s[10];
int len;
strcpy(s,"ABC");
printf("s=%s\n",s);
strcat(s,"DEF");
printf("s=%s\n",s);
len = strlen(s);
printf("文字列の長さ：%d\n",len);
getchar();
}

というプログラムを実行したいです。コンパイルしようとすると、「error C4996: 'strcpy': This function or variable may be unsafe. Consider using strcpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.」と出てきたので、strcpyの代わりにstrcpy_sを用いた所、「IntelliSense: 関数呼び出しの引数が少なすぎます」「error C2198: 'strcpy_s' : 呼び出しに対する引数が少なすぎます。」と出てきてしまいます。

すごく初歩的な質問ですが答えて下されば光栄です。

Answer 1

こんにちは。

取り敢えず、strcpy_sについては、その名前で検索するとすぐに使い方がでてきますので、それを確認すると良いですよ。strcpy_s、wcscpy_s、_mbscpy_s

ただ、Visual Studio 2013なのにエラーになるのですね。普通は警告で済みますし、対策もあと１つ表示されます。会社の開発環境で特殊な設定がされているのでしょうか？
もし、YESならstrcpy_sを使うのが適正かも知れません。会社の方へ確認下さい。

そうでないなら、strcpyを使ってもC4996警告を消す方法があります。C4996ワーニングについてが参考になります。
私は、主にマルチプラット・フォーム向けに開発しているので、C4996は小さな親切大きなお世話なので#pragma warning(disable:4996)にて出ないようにしています。

Comments

[yusukee345]

#pragma warning(disable:4996)
だと動いたのですが、
#define _CRT_SECURE_NO_WARNINGS
だと先程と同一のエラー「error C4996: 'strcpy': This function or variable may be unsafe. Consider using strcpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.」が出てきます。

[Chironian]

#pragma warning(disable:4996)で解決したのであればそれで良いのでは?

Answer 2

プロジェクトのプロパティ画面で、[C/C++]-[コマンドライン]ツーリーの「追加のオプション」に

/D "_CRT_SECURE_NO_WARNINGS"

を追加してコンパイルしてみてください。エラーが消えるはずです。（このオプションはDebug/Releaseの両方に入れておく必要があります）
もしくは、このエラーをワーニングに変えるかです。同じプロパティページで[C/C++]-[全般]の中に「警告レベル」があると思いますが、ここがレベル4になっていませんか？レベル3にするとワーニングメッセーに切り替わるはずです。そうすれば、ワーニングは出ますがコンパイルは正常終了となります。ワーニングが出るのも嫌な場合は、上記の追加のオプションを加えるか、_s 付きの関数に置き換えるかになります。

strcpy_sを使うのであれば、

strcpy(s,_countof(s),"ABC");

と2つ目パラメータにコピー先バッファのサイズを入れるようにします。

Answer 3

まさにエラー文の通りです
strcpy_sになって引数が増えましたので、必要な引数を追加してください。
引数については
MSDN参照
をどうぞ

端的に言うと、コピー先の配列の大きさを引数に追加する必要があります

Comments

[yusukee345]

コピー先の配列の大きさを引数にどうやって追加するのですか？

[ardin]

MSDNのリンク読みましたか？(リンク先の例はよくないけれども)
PineMatsuさんが記載してくれていますが、今回ならば
strcpy_s(s,10,"ABC");
とする手もあります。綺麗ではないですけれども。

こちらがスマートですかね
strcpy_s(s,_countof(s),"ABC");

_countofマクロについては、以下を参考に
https://msdn.microsoft.com/ja-jp/library/ms175773.aspx

[yusukee345]

皆様、わざわざ具体的な回答ありがとうございます。strcpy_sを使っていきます。お陰様で勉強になりました。