質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C++/CLI

C++/CLIは、.NET Frameworkの共通言語基盤であるCLI向けにC++を拡張したプログラム言語です。前身のC++マネージ拡張と比較するとシンプルで分かりやすい構文になっており、高い可読性を持ちます。

Visual C++

Microsoft Visual C++はWindowsのCとC++の統合開発環境(IDE)であり、コンパイラやデバッガを含んでいます。

Visual Studio 2010

Microsoft Visual Studio 2010はMicrosoftが提供している統合開発環境(IDE)です。

Q&A

解決済

1回答

3314閲覧

C++/CLI System::Stringをcharに変換

Taffy

総合スコア33

C++/CLI

C++/CLIは、.NET Frameworkの共通言語基盤であるCLI向けにC++を拡張したプログラム言語です。前身のC++マネージ拡張と比較するとシンプルで分かりやすい構文になっており、高い可読性を持ちます。

Visual C++

Microsoft Visual C++はWindowsのCとC++の統合開発環境(IDE)であり、コンパイラやデバッガを含んでいます。

Visual Studio 2010

Microsoft Visual Studio 2010はMicrosoftが提供している統合開発環境(IDE)です。

0グッド

1クリップ

投稿2020/01/29 05:29

編集2020/01/29 05:34

起きている現象

C++/CLI System::Stringをcharに変換したいのですが、
下記の[method1]のようなコードでchar変換を行い、
memcpyでchar型の変数にコピーしようとすると
プログラムがクラッシュしてしまいました。

自分で調べたこと、試したこと

##(1) char変換処理の変更
以下のサイトを参考に、std::stringを使わず、
System::Stringをcharに変換する処理を作ると
memcpyでエラーとならずコピーができました。[method2]

(参考) https://support.microsoft.com/ja-jp/help/311259/how-to-convert-from-system-string-to-char-in-visual-c

##(2) char変換結果の比較
下記のコードのように、method1とmethod2の結果を比較すると、
異なっていることがわかりました。(終端文字以降の部分で)

知りたいこと

method1を使用してchar型文字列に変換すると、
サイズ超過、メモリ破壊 など 起きうること、危険性について教えてください。

また、どのようなコードの書き方が望ましいのか、教えていただけないでしょうか。

#実験に使用したコード

C++/CLI

1#include "stdafx.h" 2#include <string> 3#include <msclr/marshal_cppstd.h> 4#include <time.h> 5 6using namespace System; 7using namespace System::Data; 8using namespace System::Runtime::InteropServices; 9 10// System::String^→std::stringへ変換 11std::string StringTostdstring(String^ StrData) 12{ 13 return msclr::interop::marshal_as<std::string>(StrData); 14} 15 16 17int main(array<System::String ^> ^args) 18{ 19 System::String^ strTemp = "SELECT * FROM OOO WHERE XXXXXXXXXX;"; // 長さは250文字程度 20 21 char szLog1[1024]; 22 char szLog2[1024]; 23 24 memset(szLog1,0x00,sizeof(szLog1)); 25 memset(szLog2,0x00,sizeof(szLog2)); 26 27 //method 1 28 memcpy(szLog1, StringTostdstring(strTemp).c_str(), 1024);  // 実際のコードは、ここで実行中エラーとなりクラッシュする 29 30 //method 2 31 char* str2 = (char*)Marshal::StringToHGlobalAnsi(strTemp).ToPointer(); 32 memcpy(szLog2, str2, 1024); 33 Marshal::FreeHGlobal((IntPtr)str2); 34 35 if (memcmp(szLog1, szLog2, 1024) == 0) 36 { 37 int nD1 = 0; 38 } 39 else 40 { 41 int nD2 = 0; //szLog1, szLog2は一致しない 42 } 43 44 return 0; 45} 46

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

//method 1
memcpy(szLog1, StringTostdstring(strTemp).c_str(), 1024);  // 実際のコードは、ここで実行中エラーとなりクラッシュする

StringTostdstringの結果(std::string)が1024byteあると決めつけちゃダメちゃうかな。

std::string result = StringTostdstring(strTemp);
memcpy(szLog1, result.c_str(), result.length());

ではいかがっスか?

投稿2020/01/29 07:57

episteme

総合スコア16614

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Taffy

2020/01/30 00:20

回答ありがとうございます。 std::stringは、文字数分しかメモリを確保しないということですよね。 第一引数szLog1のサイズを超えていないことを確認した上で、std::string のサイズ分だけコピーする処理に修正したところ、クラッシュしなくなりました。
episteme

2020/01/30 00:57

「少なくとも」文字数分の領域を確保する、でしょうね。 なんにせよ1024byte以上って保証はない。 あと、終端ナル文字がコピー対象でないなら、c_str()改めdata()でいい。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問