C++ wstringのreplace()が上手く置換できない

Question

visual studio 2015 C++で以下の関数を作成しました。

wstring Replace(wstring String1, wstring String2, wstring String3)
{
wstring::size_type Pos(String1.find(String2));

while (Pos != std::string::npos)
{
	String1.replace(Pos, String2.size(), String3);
	Pos = String1.find(String2, Pos + String3.size());
}

return String1;

}

この関数の
String1.replace(Pos, String2.size(), String3);
の一文が上手く動作してくれません。

デバッガで確認すると、この一文の実行前は、
String1 = L" 見積ID int(11) DEFAULT NULL,\n"
String2 = L"見積ID"
String3 = L"estimateID"
となっているのですが、この一文を実行すると、
String1 = L" estimateID"
となってしまいます。

どこが、おかしいのでしょうか。

Comments

[PineMatsu]

細かいことですが、タグは「C++」ですよね。
また、コードは```で囲うと見やすくなります。ブラウザ上では</>ボタンを押せばブロックを表示してくれます。

Answer 1

質問内容に書いてない部分で問題がありそうですね。
ほかにReplace関数があったりしませんか？

呼び出し側のコードも含めて書いてみてください。
その際に、’’’で囲むと変に文字が消えたりしないと思います。

C4717は再起のエラーですね。
wstring String4 = Replace(String1, String2, String3);の部分をReplaceの中に書いているとか。

Visual Studio Community 2015で問題なく置換されるので、コンパイラは気にしなくて大丈夫です。

#include <string>
using namespace std;

wstring Replace(wstring String1, wstring String2, wstring String3)
{
	wstring::size_type  Pos(String1.find(String2));

	while (Pos != string::npos)
	{
		String1.replace(Pos, String2.size(), String3);
		Pos = String1.find(String2, Pos + String3.size());
	}

	return String1;
}


int main()
{
	wstring String1 = L" '見積ID' int(11) DEFAULT NULL,\n";
	wstring String2 = L"'見積ID'";
	wstring String3 = L"'estimateID'";
	wstring String4 = Replace(String1, String2, String3);

    return 0;
}

Comments

[shinyaYS]

確かに上記コードだけだと、上手くいきますね。
日本語は、UTF-8なので、ソースファイルをUTF-8にしてみましたが、
上手くいきました。

何故、私が作成したプログラムが上手くいかないのか、全く分かりません。
一度、ソース全体を書かせて下さい。

#include "stdafx.h"
#include <string>
#include <sys/types.h>
#include <sys/stat.h>
#include <direct.h>
#include <fstream>
#include <iostream>
#include <vector>
#include <sstream>

using namespace std;

// 定義ファイルの構造
typedef struct _DefineData {
wstring beforeData;
wstring afterData;
} DefineData;

void GetInputFile(string & inputFile, string & fileName, string & folderName)
{
int nIndex;
nIndex = inputFile.rfind("\\");

// ファイル名を取得する
fileName = inputFile.substr(nIndex + 1);

// フォルダ名を取得する
folderName = inputFile.substr(0, nIndex);

}

vector<wstring> split(const wstring &str, char sep)
{
vector<wstring> v;

int nIndex = str.find(sep);
wstring before = str.substr(0, nIndex);
wstring after = str.substr(nIndex + 1);
v.push_back(before);
v.push_back(after);

return v;
}

int ReadDefineFile(vector<DefineData> & vDefine)
{
FILE * fp;
wstring str;
TCHAR data[128];

const errno_t er = fopen_s(&fp, "define.txt", "rt,ccs=utf-8");
while (_fgetts(data, 128, fp) != NULL)
{
str = data;
int nIndex = str.length();
str[nIndex - 1] = NULL;
vector<wstring> vData;
vData = split(str, '/');

DefineData stDefine;
stDefine.beforeData = vData[0];
stDefine.afterData = vData[1];
vDefine.push_back(stDefine);
}

return 0;

}
//
wstring Replace(wstring String1, wstring String2, wstring String3)
{

wstring::size_type Pos(String1.find(String2));

while (Pos != std::string::npos)
{
String1.replace(Pos, String2.size(), String3);
Pos = String1.find(String2, Pos + String3.size());
}

return String1;
}

int ReplaceData(string inputFile, vector<DefineData> vDefine, string outputFile)
{

FILE* fpr = NULL;
FILE * fpw = NULL;
TCHAR data[128];
wstring str;

const errno_t er = fopen_s(&fpr, inputFile.c_str(), "rt,ccs=utf-8");
const errno_t er1 = fopen_s(&fpw, outputFile.c_str(), "w,ccs=utf-8");
while (_fgetts(data, 128, fpr) != NULL)
{
str = data;

for (int i = 0; i < (int)vDefine.size(); ++i)
{
str = Replace(str, vDefine[i].beforeData, vDefine[i].afterData);
}
_fputts(str.c_str(), fpw);
}

fclose(fpr);
fclose(fpw);

return 0;

}

int main(int argc, char * argv[])
{
int rtn;
string inputFile = argv[1];
vector<DefineData> vDefine;

setlocale(LC_ALL, "");
// 入力ファイルのフォルダとファイル名を取得する
string fileName;
string folderName;
GetInputFile(inputFile, fileName, folderName);

// 出力フォルダがなければ、作成する
string outputFolder = folderName + "\\" + "(置換済)";
struct _stat stStat;
if (_stat(outputFolder.c_str(), &stStat))
{
_mkdir(outputFolder.c_str());
}
string outputFile = outputFolder + "\\" + fileName;

// 定義ファイルの読み込み
rtn = ReadDefineFile(vDefine);
if (rtn != 0)
{
exit(1);
}

// 入力ファイルの読み込みと出力ファイルの作成
rtn = ReplaceData(inputFile, vDefine, outputFile);
if (rtn != 0)
{
exit(1);
}

return 0;
}

[kopio]

パッと見て思いつくのはfopen_sの引数で、
ccs=utf-8となっているのをccs=UTF-8と
大文字にしてみるぐらいでしょうか。

[shinyaYS]

utf-8大文字にしてみましたが、変わりませんでした。
プログラムソースファイルの文字セットがSJISだったので、それをUTF-8に変わりませんでした。
プロジェクトプロパティの文字セットは"Unicodeを使用する"にしているのですが、これを"マルチバイト文字セット"を使用するに変える必要はあるでしょうか？
（今、変えるとビルドでたくさんのエラーが発生してしまうのですが）

[kopio]

文字コードが混乱していますね。

読み込むファイルはUTF-8という認識であっていますか？
fopen_sのオプションは大文字小文字で解釈変わるので要注意です。
https://msdn.microsoft.com/ja-jp/library/z5hh6ee9.aspx

他に思いつく点としては、fopen_sを_tfopen_sに変えてみてらどうなりますか？

[shinyaYS]

読み込むファイルはUTF-8です。
fopen_sを_tfopen_sに変えましたが、結果は変わりませんでした。

もうこのプログラムは諦めて、他の手段を検討しようかと思っています。

Answer 2

こんにちは。

MSVC 2015 update 2にてやってみましたが、手元では正常に動作しました。

wstring String1 = L"  見積ID int(11) DEFAULT NULL,\n";
wstring String2 = L"見積ID";
wstring String3 = L"estimateID";
wstring String4 = Replace(String1, String2, String3);

デバッガで止めて、String4を確認したら、L" estimateID int(11) DEFAULT NULL,\n"となってます。

String1 = L" estimateID"

となってしまいます。

String2が実はL"見積ID\0"のようになってないでしょうか?

---

【追記】
私が動作検証したソースです。

#include <iostream>
#include <string>

using namespace std;

wstring Replace(wstring String1, wstring String2, wstring String3)
{
    wstring::size_type  Pos(String1.find(String2));

    while (Pos != std::string::npos)
    {
        String1.replace(Pos, String2.size(), String3);
        Pos = String1.find(String2, Pos + String3.size());
    }

    return String1;
}

int main(int argc, char* argv[])
{
    wstring String1 = L" `見積ID` int(11) DEFAULT NULL,\n";
    wstring String2 = L"`見積ID`";
    wstring String3 = L"`estimateID`";
    wstring String4 = Replace(String1, String2, String3);

    wcout << String4 << endl;

    return 0;
}

Comments

[shinyaYS]

申し訳ありません。
少し、情報に誤りがありました。

実行前は
String1 = L" `見積ID` int(11) DEFAULT NULL,\n"
String2 = L"`見積ID`"
String3 = L"`estimateID`"
でした。
（値をコピーして、貼り付けた時に"`"の文字が落ちてしまったようです）

それで、実行結果は
String1 = L" `estimateID`"
です。

以上。よろしくお願いします。

[Chironian]

コピペしてやってみました。wcout << String4 << endl;にて下記が出力されました。
`estimateID` int(11) DEFAULT NULL,

う～ん、不思議ですね。ビルド方法等に差があるのでしょうか?
私はプロジェクトの新規作成でVisual C++ → Win32 → Win32コンソール アプリケーションで生成し、Win32のDebugモードでビルドし、IDEから実行してます。
後、私のMSVCはupdate2を当ててます。

[shinyaYS]

私は、Visual Studio Communityを1週間程前にインストールして、それを使用しています。
Communityだと何か差があるのでしょうか？

[shinyaYS]

私も、
wstring String4 = Replace(String1, String2, String3);
と変えてビルドした所、下記のわーにんぐが出ました。

warning C4717: 'Replace': すべてのコントロールのパス、関数を回帰するとランタイム スタック オーバーフローが発生します。

そして、実行すると、スタックオーバーフローのエラーが発生しました。
これは、どのように、回避されたのでしょうか？

[Chironian]

私が使っているのもVisual Studio 2015 ommunityです。
shinyaYSさんはwstring String4 = Replace(String1, String2, String3);をReplace()関数の中に入れたのではないでしょうか?
そうすると再帰定義となり、かつ、終了条件の指定がないのでスタックをひたすら消費するので、スタックオーバーフローになります。
私の試しているソースを回答へアップします。

[Chironian]

kopioさんへの回答のソースを確認しました。
String3の最後に\0が入ってます。
デバッガでは一見入っていないように見えますが、更にポイントして1文字単位で表示させると分かります。

原因は、ReadDefineFile()で文字列の最後にNULL文字を入れているからですね。
std::stringはNULL終端を仮定しないです。文字数をきっちり管理しているので途中にNULL文字があっても管理されます。それをc_str()で取り出すとC言語処理系はNULL終端で解釈するため、意図と異なる動作になるのだと思います。Visual StrudioもC言語文字列として解釈し表示しているようです。
resize()すれば大丈夫な筈です。

[shinyaYS]

ありがとうございました。
言われた通り、\0が入っていました。
resize()して正常に動作する事を確認しました。