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

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

新規登録して質問してみよう
ただいま回答率
86.12%
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)です。

解決済

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

Taffy
Taffy

総合スコア32

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)です。

1回答

0グッド

1クリップ

1386閲覧

投稿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

以下のような質問にはグッドを送りましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

グッドが多くついた質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

下記のような質問は推奨されていません。

  • 間違っている
  • 質問になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

適切な質問に修正を依頼しましょう。

回答1

2

ベストアンサー

//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

総合スコア16005

Taffy, KoichiSugiyama👍を押しています

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

Taffy

2020/01/30 00:20

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

2020/01/30 00:57

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

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

ただいまの回答率
86.12%

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

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

質問する

関連した質問

同じタグがついた質問を見る

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)です。