CString データの文字数カウントができない

質問

VusualStudioのMFCでクリップボードのデータの文字数をカウントしようとしています。

Webにあったように以下のようなクラスを作成し、

===========================================================
CString CClipboard::GetClipboardText()
{
CString strText;
strText.Empty();

// クリップボードにテキストデータが入っているかを調べる
if( !::IsClipboardFormatAvailable(CF_UNICODETEXT) )
return strText;

// クリップボードのオープン
if( !::OpenClipboard(NULL) )
return strText;

// クリップボードからデータを取得し、strTextへコピー
HANDLE hMem = ::GetClipboardData(CF_UNICODETEXT);

LPTSTR pMem = (LPTSTR)::GlobalLock(hMem);
::lstrcpy((LPTSTR)(LPCTSTR)strText, pMem);
::GlobalUnlock(hMem);
::CloseClipboard();

return strText;
}
===========================================================

適当にメモ帳などでコピー操作をしたのち、

CClipboard board;
CString a = board.GetClipboardText();
int b = a.GetLength();

上記コードが含まれた操作を実行するとaという文字列自体はデバッガ上から"test"のように
認識されているのですが、bの値が0になってしまいます。

ちょっとわけがわからず途方に暮れています。

アドバイスがあればよろしくお願いします。

回答No.4

m-take0220 · ベストアンサー

＞ HANDLE hMem = ::GetClipboardData(CF_UNICODETEXT);
＞ LPTSTR pMem = (LPTSTR)::GlobalLock(hMem);

UNICODEでとってきたなら、LPTSTRじゃなくてLPWSTRにしないとダメなのでは?
あとは

strText = pMem;

でいいような気もするけど。

お礼

ありがとうございます。No.1の方でも動きましたが、こちらでも
動きました。理屈的にはこちらでいいと思いました。

ただLPTSTRのままでいいとの私の理解です。
（LPTSTRはUNICODEの時に使用すると思っているのですが、間違っていますか）

回答No.5

m-take0220

＞ ただLPTSTRのままでいいとの私の理解です。
＞ （LPTSTRはUNICODEの時に使用すると思っているのですが、間違っていますか）

LPTSTRは、UNICODEが#defineなどで定義されている場合にLPWSTRとなり、それ以外の場合はLPSTRとなります。
現在のプロジェクトではUNICODEが定義されているのでしょう。そうであれば、コード的には同じになります。しかし、そのコードを別のプロジェクトで使用しようとした場合に、そのプロジェクトでUNICODEが定義されていないと、UNICODE文字列が入っているバッファの内容を、マルチバイト文字列として処理しようとしてしまい、正しく動作しなくなるでしょう。

TCHARやLPTSTRなどは、プロジェクトの設定によって使用する文字コードが違っても、同じソースで済むようにするためのものです。最初からUNICODE文字列とわかっているなら、wchar_tやLPWSTRなどを使うべきです。

お礼

なるほど、確かにそうですね。

ありがとうございます。色々すっきりしました。

回答No.3

επιστημη（@episteme）

> ClipboardText内のローカル変数だから、用がすめば破壊されるけど

破壊されるのは戻り値が確定した後だから無問題じゃね?

お礼

ありがとうございます。No.1の方のもので動きました！

回答No.2

noname#157863

CString CClipboard::GetClipboardText()
{
CString strText; <----
・・・
lstrcpy((LPTSTR)(LPCTSTR)strText, pMem);
・・・
return strText;
｝

GetClipboardText内のローカル変数だから、用がすめば破壊されるけど

お礼

ありがとうございます。

No.3の方がおっしゃっているように問題ないと理解しました。

回答No.1

επιστημη（@episteme）

> ::lstrcpy((LPTSTR)(LPCTSTR)strText, pMem);

ここがヘン。無理矢理感満載。

LPTSTR buf = strText.GetBuffer(文字列サイズ);
::lstrcpy(buf, pMem);
strText.ReleaseBuffer();

参考URL：

http://msdn.microsoft.com/ja-jp/library/kt26tkzx.aspx

お礼

お返事がすぐにいただけていることに気づかず(メールが飛ぶと勘違いしていました）、ご連絡遅れました。

ありがとうございます。
上記でうまくいきました！

ただ、GetBuffer, ReleaseBufの使い方の意味を完全に把握できていないのでこれは勉強しなおします。