C#からc++の文字列を受け取ることができません

Question

C#からc++のコードを呼び出し、文字列を取得する処理を実装しています。
返却して受け取りたい文字コードはUnicode形式です。
その際、C#で定義したStringBuilderに格納される文字が文字化けしてしまいます。
何が原因と考えられますでしょうか？

C++とのインタフェース

        [DllImport(DllName, EntryPoint = "CppCode", CharSet =CharSet.Unicode)]
        [return: MarshalAs(UnmanagedType.LPWStr)]
        internal static extern void TestCode(safeHandle ptr, [Out,MarshalAs(UnmanagedType.LPWStr)]StringBuilder str);

上記コードの呼び出し元
このコードのsbの値が文字化けします。

sb = new StringBuilder("",1500);
TestCode(ptr, sb);

stringを返却するC++コード

extern "C" void CppCode(wchar_t **str)
{
        std::string text = "ABCDEFG";
		*str= new wchar_t[text.length() + 1];
		mbstowcs(*str, text.c_str(), text.length()+1);
}

Comments

[y_waiwai]

どういうふうに文字化けするんでしょう

[crew8573]

C++の処理を通すと、StringBuilderの変数に以下のような文字化けが発生します。
C++コード側をデバッグしてみたところ、正しい文字列が格納されていることは確認できました。
■文字化け
컜敶獲潩㵮ㄢ〮•湥潣楤杮∽瑵ⵦ∸㰾癳⁧敶獲潩≮⸱∰瘠敩䉷硯∽‰‰⸸〶

■正しい文字列
<?xml version=\"1.0\" encoding=\"utf-8\"><svg versio...

[BeatStar]

パターン的は『ファイルの文字コードが違う』系かなぁ。

[crew8573]

Unicode文字列なので、UnmanagedType.LPWStrでC#側で宣言してC++側ではwchar_tで扱っているんですけどね。。。なぜでしょうかね。

[退会済みユーザー]

C#側でバッファを確保してLPWStr(wchar_t*)を渡してるのに、C++側がwchar_t **でいいんですか？引数も2つから1つになってるし、色々おかしいです。

[BeatStar]

@ 質問者さん
> Unicode文字列なので、

私が言ってるのはその意味というより、ファイル自体のです。

OSが明記されていないのでわかりませんが、Windowsだと https://win10labo.info/win10-notepad-char-code/ のようなやつの方です。

[crew8573]

@BeatStarさん
文字列は内部で生成しているだけなので、ファイルの読み込みなどは行っていないのですが、そういう意味の指摘では無いでしょうか？

Answer 1

まずDllImportの定義自体が引数の数が違ってるし、呼び出し規約も書いてないし、voidなのにreturnのマーシャリング定義してるし、色々間違ってる気がします。あと、ネイティブ相互運用性のベスト プラクティスでは、StringBuilderでの受け渡しは回避すべき項目として記述されています。

あとは、なんだか関数の設計自体もおかしい気がします。

1. 呼び出し側でバッファ確保するのが前提
2. CppCode側でバッファ確保するのが前提

どちらなのでしょう？サンプルコードを見る限り、両方でバッファ確保しようとしてますが。
1であれば、CppCode側でバッファ確保する必要はないでしょうし、呼び出し側からバッファのサイズも引数で渡すべきです。
2であれば、C#側でLPWStrを指定してるのはおかしいです。また、CppCodeで確保したメモリ解放用の関数も用意する必要があります。
1、2どちらの場合も、コピーした文字列の長さを返したほうがいいでしょう。
個人的には、1の方が解放用の関数用意しなくて済みますし、Cの標準関数の実装に近いので好みです。

Comments

[crew8573]

@radianさん
色々とおかしなところがあり申し訳ございません。
私の実現したい実装は”１呼び出し側でバッファ確保するのが前提”です。

以下、質問させていただきたいです。
・StringBuilderを使用するのが非推奨とのことですが、代わりに使用する型はstringでしょうか？
・呼び出し側でバッファを確保する場合、どのような書き方にすべきでしょうか？
※StringのバッファでググるとStringBuilderばかり出てきてしまいました。
・c++内側でバッファを確保せずに、以下処理を実行すると怒られてしまいました。
C#で確保したバッファをc++で流用する場合はどのような書き方にすべきでしょうか？
※バッファ確保すれば怒られませんでした。
mbstowcs(str, text.c_str(), text.length() + 1);

質問が多くて申し訳ありませんが、よろしくお願いいたします。

[退会済みユーザー]

> ・StringBuilderを使用するのが非推奨とのことですが、代わりに使用する型はstringでしょうか？
リンク先のネイティブ相互運用性のドキュメントを読んでください。
> ✔️ 推奨: ArrayPool から char[] を使用するようにします。
なお、[Out]で文字列を受け取るの場合の話なので、C#からC++に渡す場合はstringで問題ありません。

>・呼び出し側でバッファを確保する場合、どのような書き方にすべきでしょうか？
> ・C#で確保したバッファをc++で流用する場合はどのような書き方にすべきでしょうか？
こちらを読んでください。
https://docs.microsoft.com/ja-jp/dotnet/fundamentals/code-analysis/quality-rules/ca1838

[crew8573]

ありがとうございます。
C++で確保した情報をchar型で受け渡しすることで適切に動作することが出来ました。

[退会済みユーザー]

解決したのであれば、質問をクローズしてください。
> C++で確保した情報をchar型で受け渡しすることで適切に動作することが出来ました。
呼び出し側で確保するのはやめたということですか？