LPWSTRのコピー

Question

新たに作ったLPWSTRの内容を、あらかじめ用意しておいたLPWSTRにコピーしたいのですが、うまくいきません。
どうするのが正解なのでしょうか。

簡易化したソースコードと失敗例を載せておきます。
（見やすくするために行頭に全角スペースを入れています。）

class LPWSTR_Copy{
public:
　　LPWSTR lpwstr;
public:
　　// lpwstrにNewStrをコピーしたい
　　void StrCopy( LPWSTR NewStr){
　　　　// lpwstr= NewStr; // 失敗
　　　　// lstrcpy( lpwstr, NewStr); // 失敗
　　　　/* TCHAR tchar[20];
　　　　　 lstrcpy( tchar, NewStr);
　　　　　 lpwstr= tchar;
　　　　*/ // 失敗
　　　　/* lpwstr= new TCHAR[sizeof(NewStr)];
　　　　　 lstrcpy( lpwstr, NewStr);
　　　　*/ // 失敗（実行時エラー）
　　}
};

void copy(LPWSTR_Copy *str){
　　LPWSTR newstr= TEXT("コピー元文字列");
　　str->StrCopy(newstr);
}
void main(){
　　LPWSTR_Copy strcpy;
　　copy(&strcpy);
　　LPWSTR str= strcpy.lpwstr;
　　// ↑ここでstr.lpwstrの中身が"コピー元文字列"となっていてほしい
}

hiro_knigh · Accepted Answer

まず、クラスの作り方で違うという意味については

C++で追加されたクラスという物は、データとそのデータを扱う関数をセットにしたものです。
よって、文字列をコピーするだけのクラスというものは意味がありません。

ということで、とりあえず動くものを添付します。

#include <windows.h>
#include <iostream>
#include <vector>

class cstring{
private:
LPWSTR lpwstr;
public:
cstring(){
lpwstr = NULL;
}
~cstring(){
lpwstr = NULL;
}
cstring( LPWSTR NewStr){
lpwstr = NULL;
lpwstr= new TCHAR[wcslen(NewStr)+1];
lstrcpy( lpwstr, NewStr);
}
const LPWSTR chr(){
return lpwstr;
}
void operator=( LPWSTR NewStr){
if (lpwstr != NULL) {
delete []lpwstr;
}
lpwstr= new TCHAR[wcslen(NewStr)+1];
lstrcpy( lpwstr, NewStr);
}
};

class UseLSC{
public:
std::vector<cstring> vlc;
void In(){
WCHAR wc[20];
for( int i=0; i<20; i++){
wsprintf( wc, L"a_%02d_", i);
vlc.push_back(wc);
}
}
void Make(){
In();
}
};

int main(){
int i;
setlocale( LC_ALL, "Japanese" );

UseLSC use;
use.Make();

for ( i=0; i<20; i++ ){
wprintf(L"%s\n", use.vlc[i].chr());
}

}

私も汎用機の世界でしか仕事をしていないので、TCHAR等のマクロやUnicodeプログラミングの世界については分からないので、少し不安な箇所もあります。。。

hiro_knigh · Answer

お疲れ様です。

まず、お伺いしたいのが、「lpwstr= NewStr; 」で失敗していると言う事ですが、どう失敗しているのでしょうか？
理論上はこれでうまく行くはずです。
（[Visual C++ Express Edition] + [Windows Visuta])でコンパイルしデバックモードで追っかけて問題が無い事も確認済み。
（copy関数内のnewstrはデータ領域に生成された文字列「コピー元文字列」の先頭アドレスを指しています。）
（LPWSTR_Copy::StrCopy関数は引数でもらってきた、スタック領域の先頭アドレスをきちんとlpwstrにコピーしています。）

ちなみに、こんなクラスの作り方はまずしません。
「静的領域・データ領域・スタック領域・ヒープ領域」等のメモリ区分についてご理解はありますか？

phoenix343 · Answer

えーと
こんなクラスの作り方しないのは#1の方と同意です。笑

他の視点から気付いたのでそれを挙げますね。


LPWSTRを使っているならそれに統一すべきです。
なぜTCHARやTEXTを使ってるのでしょうか？

_T、TEXT、TCHARマクロは
UNICODEを定義するかしないかでCHAR、TCHARまたはLPSTRとLPWSTRを切り替えてくれるマクロです。

LPWSTRにするのであれば
L"文字列"
とLをつけるだけです。

Program Tips - UNICODE
http://www.246.ne.jp/~y-ookubo/program/tips/unicode.html

hiro_knigh · Answer

>>「lpwstr= NewStr;」について補足します。
>>void copy(LPWSTR_Copy *str)を抜けたとき、copy()の内部で
>>作られたnewstrは消滅します。
>>するとnewstrのアドレスをコピーしたlpwstrも消えてしまう、
>>ということです。

>>このクラスを作った理由は、vector<LPWSTR>で失敗したから
>>です。
>>メモリ区分についてはよく知りません。ざっと調べた限りでは、
>>・プログラム領域：関数
>>・静的領域：グローバル変数、static変数用
>>・スタック領域：自動変数などの自動的に消える変数用
>>・ヒープ領域：ポインタにnewで割り当てた領域
>>・データ領域：？
>>という感じでしょうか。


補足します。

「データ領域」という言葉で正しかったかどうか調べてみましたが、ちょっと分かりませんでした。（申し訳ありません）
調べてみる限りでは「プログラム領域」で正しいような感じです。

これを踏まえて最初にご提示頂いたソースコートではcopy関数で記述されている「TEXT("コピー元文字列")」の文字列の実体は書換え不可能なプログラム領域に置かれます。
（[Visual C++]＋windowsの場合）
「newstrは消滅します」と記載されている内容は正しいのですが、消滅するまえに、lpwstrへ文字列の先頭アドレスをコピー出来ているため問題ありません。

以上。

LPWSTRのコピー

お疲れ様です。

この回答への補足

えーと

まず、クラスの作り方で違うという意味については

>>「lpwstr= NewStr;」について補足します。

似たような質問が見つかりました

このQ&Aを見た人はこんなQ&Aも見ています

関連するカテゴリからQ&Aを探す

このQ&Aを見た人がよく見るQ&A

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング