C++ 文字列同士を結合させたいです。

Question

C++ でchar[] へ格納した文字列を、std::stringに渡し、
std::string同士を連結させたいです。
以下のコードのように文字列 += 文字列 と記述し、
OpenCVのimwrite関数で保存したいのですが、
文字列結合ができておらず、.jpgファイル名が結合できていない状態となります。
なぜ結合できないのかご教授いただきたく思います。

char disparityMapWriteFilePath[MAX_PATH] = "";
char disparityMapWriteDataName[64] = "";
//disparityMapWriteFilePathにexeディレクトリパス内にある指定ファイルパスを入力
//パスを確認しました。
sprintf_s(disparityMapWriteDataName, 64, "\dispMap_%d_%d_%d_%d_%d_%d_%d_%d.jpg", preType, preSize, preCap, URatio, speckRange, speckWin, blockSize, numDisp);
std::string filename( disparityMapWriteDataName, sizeof(disparityMapWriteDataName)/sizeof(disparityMapWriteDataName[0]));
std::string filepath( disparityMapWriteFilePath, sizeof(disparityMapWriteFilePath)/sizeof(disparityMapWriteFilePath[0]));
filepath += filename;  //結合できるはずだけど…
cv::imwrite(filepath, disparity_map); //exeディレクトリ指定ファイルパスしか入ってない

Comments

[ardin]

宣言時のfilepathに何が入っていますか？
投稿するためにそこは抜いているだけでしょうか？投稿部分だけ見ると空に見える

Answer 1

こんにちは。

std::stringはC言語文字列と異なり、null終端ではありません。（つまり、'\0'も格納できます。）
ですので、kaztoさんが回答されているように、filepathの長さはMAX_PATHになっていますので、その後ろにfilenameが結合されます。

そして、恐らくOpenCVはfilepathからC言語文字列（c_str）を取り出すので、結合後のfilepathの最初のnull文字で文字列が終わってしまうのだろうと思います。結果、結合できていないかのように見えます。

#include <iostream>
#include <cstdlib>

#define MAX_PATH 260

int main()
{
    char disparityMapWriteFilePath[MAX_PATH] = "/to/path/\0<AFTER NULL CHAR>";
    char disparityMapWriteDataName[64] = "filename";
    std::string filename( disparityMapWriteDataName, sizeof(disparityMapWriteDataName)/sizeof(disparityMapWriteDataName[0]));
    std::string filepath( disparityMapWriteFilePath, sizeof(disparityMapWriteFilePath)/sizeof(disparityMapWriteFilePath[0]));
    filepath += filename;  //結合できるはずだけど…}
    std::cout << "filepath.c_str():" << filepath.c_str() << "\n";
    std::cout << "filepath        :" << filepath << "\n";
}
// Wandboxでの実行結果
filepath.c_str():/to/path/
filepath        :/to/path/<AFTER NULL CHAR>filename

wandbox

上記ケースで、disparityMapWriteFilePathの初期化で指定した文字列より長い部分は0で初期化（ゼロ初期化）されます。そして、恐らくstd::coutはnull文字を単に出力しないだけだろうと思います。

Answer 2

std::string filename( disparityMapWriteDataName, sizeof(disparityMapWriteDataName)/sizeof(disparityMapWriteDataName[0]));
std::string filepath( disparityMapWriteFilePath, sizeof(disparityMapWriteFilePath)/sizeof(disparityMapWriteFilePath[0]));

ここでstd::stringのコンストラクタにサイズ指定しているのが意図しない動作に繋がっていそうです。
通常std::stringは与えたcharの配列がヌル文字で終端されていればそこまででサイズ確保してくれます。

実際動かしていないため確実ではありませんが、filepathはMAX_PATH分確保されてしまっています。
一方、MAX_PATH分を文字列として受け取ってしまっているため、後半は\0埋めされているのか不定な文字が入っているのか、想定した文字列より長いバッファが確保されてしまい、連結しても途中にヌル文字が登場するため、ファイルパスとしてはそこまでになってしまう、ということが想像できます。

対処としては、サイズを与えないこと、になります。

Comments

[mypace]

ありがとうございました。
文字列結合できました。

Answer 3

filepath宣言時に中身とサイズ(260)も確保しているが、その確保している中身はなんからの文字列(charに入れたもの)と終端文字相当。
厳密にはfilepathには追加されています。260以降に。
しかし、宣言時に終端文字が入っているため、それが見えません。

	std::string filepath(disparityMapWriteFilePath);

だけでよいかと。
同時に

	std::string filename(disparityMapWriteDataName);

でよいような。