文字列を結合したい(VC++)

Question

str1とstr2の文字列を結合したい。
str1とstr2をstrcatを使って結合したいのですが、うまくいきません。
strcatを使うと、strcat_sを使用してくださいとエラーがでるため、strcatを使ったのですが、第2引数に何を入れればいいのかわからないので質問させてもらいました。
使い方を調べると、第二引数は省略可と書いてあったのですが、引数を渡さないとコンパイルエラーになります。

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

int main(void) {

	char str1[] = "aaa";
	char str2[] = "bbb";
	
	strcat_s(str1,256,str2);★ここでstack around the variable str was corrupted とRuntimeエラーが起きます。
	printf(str1);

	return 0;

}

Answer 1

ええと、C++なんですよね？
それならば、用意されている文字列型を使うのが手っ取り早いです。

#include <iostream>
#include <string>

int main() {
    std::string str1 = "aaa";
    std::string str2 = "bbb";
    
    str1 = str1 + str2;
    std::cout << str1 << "\n";
}

Answer 2

str1のサイズは４しかないので、

strcat_s(str1,256,str2);

第二引数は４としないとダメですね

使い方がわからない場合はぐぐりましょう。

---

このまんまにしとくとアレなので、追記しときます
strcat_s の第二引数は、格納するバッファのサイズを入れます。
で、この場合、格納するバッファサイズは、結合後の文字列分のサイズが必要となります。
質門のコードの場合は、3文字＋3文字＋'\0'ということで７以上ないとダメということになりますんで、
最小限動かすコードとしては

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

int main(void) {

    char str1[7] = "aaa";
    char str2[] = "bbb";

    // strcat_s(str1,7,str2);  // これでもいいけど、わざわざ数字書かなくてもいいように
    strcat_s(str1,sizeof(str1),str2);

    printf(str1);

    return 0;
}

ということになりますね。

Comments

[lleldigm]

4でも動かないですね。。ランタイムエラーです

[lleldigm]

ぐぐると、strcat_s(str1,sizeof(str1),str2);
こうなんですよね。https://gyazo.com/5d830129043905cb1564cbed7c4f49ca

[y_waiwai]

str1 に、str2の内容を追加するため、
str1のサイズは７以上ないとダメってことですね
char str1[7] = "aaa";
としといて、
strcat_s(str1,7,str2);

[lleldigm]

やはり同じエラーです・・

[y_waiwai]

strcat_s(str1,1024,str2);
としました？
第二引数は、格納するバッファのサイズがはいります

まあ、
strcat_s(str1,sizeof(str1),str2);
としとけばいいですね

[lleldigm]

なるほどありがとうございます。
今確認したところ無事に動きました。ありがとうございます！

[lleldigm]

追記ありがとうございます。追加先の配列要素数が足りないという点は盲点だったので、ためになりました。

[y_waiwai]

んで、も一つ言っときますと、
VisualStudioでは、scanf とかstrcpy,strcat もそうですが、危険な関数、ってことで使えないようにしています

が、プロジェクトのプロパティで、セキュリティのオプションを無効とすることで使えるようになります。
が、これらの関数を使うと、ちょっとしたバグでプログラムが暴走することになりますんで注意しましょう

[lleldigm]

そんな設定があるんですね・・
参考書に乗ってるscanf , strcpy 等すべて使えないみたいなので、疑問に思ってました。。