TreeView表示がおかしいです。

質問

Borland C++ (WindowsXP SP3)でWindowsAPIの練習中の人です。

メインウィンドウ(フレームウィンドウ)の中に子ウィンドウを作成してます。
その子ウィンドウの中でTreeView表示しているのですが、下の画像の様な表示になってしまいます。

何かお気づきになりましたら、ご教授お願いします。

一部、ソースを抜粋します。
----------------------------------------------------------------------------------
#include "header.h" // ヘッダーファイル

LRESULT CALLBACK TreeWndProc ( HWND, UINT, WPARAM, LPARAM );

BOOL Cls_OnTreeCreate(HWND, LPCREATESTRUCT);
void Cls_OnTreeSize(HWND, UINT, int, int );
void Cls_OnTreePaint(HWND);
static HWND hTree;

LRESULT CALLBACK TreeWndProc(HWND hWnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
switch ( msg ) {
HANDLE_MSG(hWnd, WM_CREATE, Cls_OnTreeCreate);
HANDLE_MSG(hWnd, WM_SIZE, Cls_OnTreeSize);
HANDLE_MSG(hWnd, WM_PAINT, Cls_OnTreePaint);
default:
return ( DefWindowProc ( hWnd, msg, wParam, lParam ) );
}
return 0L;
}
BOOL Cls_OnTreeCreate(HWND hwnd, LPCREATESTRUCT lpCreateStruct)
{
RECT rc;
HINSTANCE hInst;
HWND hwndTreeView;

HTREEITEM hParent1, hParent2, hParent3, hChild1, hChild2;
TV_INSERTSTRUCT tv;

InitCommonControls(); // コモンコントロールの初期化
GetClientRect (hwnd, &rc);

hInst = (HINSTANCE)GetWindowLong ( hwnd, GWL_HINSTANCE );
hwndTreeView = CreateWindowEx(
0,
WC_TREEVIEW,
"",
WS_CHILD | WS_BORDER | WS_VISIBLE | TVS_HASLINES | TVS_HASBUTTONS | TVS_LINESATROOT,
rc.left, rc.top, rc.right,rc.bottom,
// 0,0,0,0,
hwnd,
(HMENU)ID_MYTREE,
hInst, NULL );

TreeView_SetBkColor (hwndTreeView,RGB(0,0,0));
TreeView_SetTextColor (hwndTreeView,RGB(255,255,255));
TreeView_SetLineColor (hwndTreeView,RGB(125,125,255));

memset((char *)&tv, '\0', sizeof(tv));

tv.hInsertAfter = TVI_LAST;
tv.item.mask = TVIF_TEXT;
tv.hParent = TVI_ROOT;
tv.item.pszText = "粂井";
hParent1 = TreeView_InsertItem(hwndTreeView,
----------------------------------------------------------------------------------
文字数表示制限のため、ここまでです。

宜しくお願いします。

回答No.3

Wr5 · ベストアンサー

>子ウィンドウにSendMessage ( hTree, WM_PAINT, NULL, NULL );メッセージを送ります。

このhTreeはどのウィンドウです？
掲示されているコードでは未設定(グローバル変数なので0(NULL))…のようですが？

親ウィンドウのWM_PAINTに対して、実際の描画を行わず子ウィンドウに投げるだけ…だと、親ウィンドウの表示が不正になります。
(子ウィンドウですべて覆ってしまえば見えないでしょう…が)
また、BeginPaint()～EndPaint()で無効領域の解除を行っていませんので、OSから延々とWM_PAINTが飛んでくることになります。
# Borland C++ではSpy++は無いでしょうけど…Winspectorでメッセージの確認ができます。

子ウィンドウには「SendMessageでWM_PAINTを投げる」のではなく、InvalidateRect()などで無効領域を設定してOSに投げさせる…べきかと。
# もっとも、親ウィンドウにWM_PAINTが飛んできた時点でDefWindowProc()に任せればやってくれる…ハズですが。(子ウインドウのスタイルにも寄りますけど。)

……次は、コントロール(TreeView)がチラつく…でしょうかね？
# ちゃんと親ウィンドウに乗っかっているから、親のWM_ERASEBKGNDの影響はない…かな？？

お礼

おおお、まさしくその通りです。spy++で確認できました。
確かに永遠とWM_PAINTが飛んできています。
私の中では
Cls_OnPaint(HWND hwnd){
}
と親ウィンドウの中で設定すれば、それでWM_PAINT処理は終わりだと思っていました。

実際は無効領域の解除をしないと何度でもWM_PAINTが飛んできて永久ループ状態だったのですね。

親ウィンドウの処理
-----------------------------------------------------------------------
BOOL Cls_OnCreate(HWND hwnd, LPCREATESTRUCT lpCreateStruct)
{
RECT rc;
HINSTANCE hInst;

GetClientRect (hwnd, &rc);

hInst = (HINSTANCE)GetWindowLong ( hwnd, GWL_HINSTANCE );

hTree = CreateWindowEx (
NULL,
(LPCTSTR)"TreeWindow",
NULL,
WS_CHILD | WS_VISIBLE | WS_SIZEBOX,
rc.left, rc.top, rc.left+200, rc.bottom,
hwnd,NULL,hInst,NULL );
}

void Cls_OnPaint(HWND hwnd)
{
PAINTSTRUCT ps;
BeginPaint ( hwnd, &ps );
EndPaint ( hwnd, &ps );
}
-----------------------------------------------------------------------
子ウィンドウの処理
-----------------------------------------------------------------------
void Cls_OnTreeSize(HWND hwnd, UINT state, int cx, int cy)
{
MoveWindow (hwndTreeView, 0, 0, cx, cy, TRUE);
}

void Cls_OnTreePaint(HWND hwnd)
{
PAINTSTRUCT ps;
BeginPaint ( hwnd, &ps );
EndPaint ( hwnd, &ps );
}
-----------------------------------------------------------------------
とソースを変更して実行すると、希望していた通りの動きになりました。
「無効領域の解除」は大変、勉強になりました。
「Wr5」さん、何度も何度も的確なアドバイスありがとうございました。
今後とも、宜しくお願いいたします。

回答No.2

Wr5

>void Cls_OnTreeSize(HWND hwnd, UINT state, int cx, int cy)
>{
>　MoveWindow (hwndTreeView, 0, 0, cx, cy, TRUE);
>}

コンパイル通りますか？
hwndTreeViewの宣言がありませんけど？
グローバル変数なのでしょうか？
グローバル変数だとしても、この変数の中身は0(NULL)になっているかと思われますが…
# Cls_OnTreePaint()の方も同様。

「Cls_OnTreeCreate()で設定している。」というのであれば、残念ながらそれはCls_OnTreeSize()やCls_OnTreePaint()には引き継がれません。
Cls_OnTreeCreate()内のローカル変数に設定しただけで、抜けた後には失われてしまうモノですから。
# 「ローカル変数に設定しているが大丈夫か？」と回答したんですが…。
# 「大丈夫だ、問題ない。」なんですか？

Cls_OnTreeSize()で、ウィンドウサイズ全体にまでTreeViewを広げようとしていますが、実際に掲示された画像では広がっていませんよね？
その関係で、Cls_OnTreePaint()も正しく動作していないのではありませんか？

補足

失礼しました。No1さんのご指摘を受け
static HWND hwndTreeView;
として、再利用できるようにしました。

Cls_OnTreeSize()は子ウィンドウ画面の全体に広げています。
将来的には、ドッキング可能な子ウィンドウにしようと考えています。

理由がわからないのですが、親ウィンドウでWM_PAINTを受け取った時に、子ウィンドウにSendMessage ( hTree, WM_PAINT, NULL, NULL );メッセージを送ります。子ウィンドウでWM_PAINTを受け取った処理を
void Cls_OnTreePaint(HWND hwnd)
{
UpdateWindow (hwndTreeView);
}
と変更すると、不思議とすべての表示問題が解決されました。なぞです。
根本的な解決になっていないので、このまま、もう少し質問を継続としてご指摘を受けようと思います。

回答No.1

Wr5

解決には役立たないと思いますが、気になったので指摘を。

>InitCommonControls();// コモンコントロールの初期化

WinMain()あたりでやっておけば良いです。
WM_CREATEのたびに実行が必要なモノではありません。
# 私ならInitCommonControlsEx()で初期化実施しますけど。

>hwndTreeView = CreateWindowEx(

ここから抜けた後、TreeViewのウィンドウハンドルが不明になりますが大丈夫ですか？
ID(ID_MYTREE)から取得は可能ですから、回避策自体はありますけど…。

実際のアイテム設定(挿入)部分、および描画をおこなっている部分が不明ですので…なんとも。

補足

ご助言ありがとうございます。実際のアイテム設定部分等が不明ではお答えできませんよね。
失礼致しました。以下、前回のソースの続きです。
---------------------------------------------------------------------------------
memset((char *)&tv, '\0', sizeof(tv));

tv.hInsertAfter = TVI_LAST;
tv.item.mask = TVIF_TEXT;
tv.hParent = TVI_ROOT;
tv.item.pszText = "粂井";
hParent1 = TreeView_InsertItem(hwndTreeView, &tv);

tv.item.pszText = "田中";
hParent2 = TreeView_InsertItem(hwndTreeView, &tv);

tv.item.pszText = "佐藤";
hParent3 = TreeView_InsertItem(hwndTreeView, &tv);

tv.hParent = hParent1;
tv.item.pszText = "康孝";
hChild1 = TreeView_InsertItem(hwndTreeView, &tv);
tv.item.pszText = "ひとみ";
hChild2 = TreeView_InsertItem(hwndTreeView, &tv);
tv.hParent = hChild1;
tv.item.pszText = "志麻";
TreeView_InsertItem(hwndTreeView, &tv);
tv.hParent = hChild1;
tv.item.pszText = "櫻都";
TreeView_InsertItem(hwndTreeView, &tv);
tv.hParent = hParent2;
tv.item.pszText = "マイケル";
TreeView_InsertItem(hwndTreeView, &tv);
tv.hParent = hParent3;
tv.item.pszText = "パトリシア";
TreeView_InsertItem(hwndTreeView, &tv);
}

void Cls_OnTreeSize(HWND hwnd, UINT state, int cx, int cy)
{
MoveWindow (hwndTreeView, 0, 0, cx, cy, TRUE);

}

void Cls_OnTreePaint(HWND hwnd)
{
RECT rc;
GetClientRect (hwnd, &rc);
MoveWindow (hwndTreeView, rc.left, rc.top, rc.right, rc.bottom, TRUE);

}

/*
void Cls_OnTReeClose(HWND hwnd)
{
DestroyWindow ( hwnd );
}

void Cls_OnTReeDestroy(HWND hwnd)
{
PostQuitMessage ( 0 );
}
*/
--------------------------------------------------------------------------------
以上です。
何かおかしな所がありましたら、ご教授お願いします。