Another tree-view problem.

**Queatrix** · 09-23-2005

Okay, this is my problem now:

When I run this:

Code:

 
	case DO_CREATE:
		{
		int a,b;
		a=1,b=2;
		//add some items to the the tree view common control
		tvi.mask=TVIF_TEXT|TVIF_IMAGE|TVIF_SELECTEDIMAGE|TVIF_DI_SETITEM;
		tvi.pszText=TEXT(tempSrc);			 //item label
		tvi.cchTextMax=lstrlen(tvi.pszText);	//length of item label
		tvi.iImage=a;						
		tvi.iSelectedImage=b; 
		tvis.item=tvi;
		tvis.item.mask = TVIF_TEXT;
		tvis.hInsertAfter=0;
		tvis.hParent=TVI_ROOT;				 //parent item of item about to be inserted 
		hPrev[CShPrev]=(HTREEITEM)SendMessage(g_hwndTreeView,TVM_INSERTITEM,0,(LPARAM)&tvis);
		return 0;
		}
//...
//...
//...
bool FindFiles_(int nType, char cFile[MAX_PATH])
{
MessageBox(hwndOtherUse,cFile,0,0);
 
if(nType == DIR)
PostMessage(hwndOtherUse, DO_CREATE, MAKEWPARAM(1, 0), 0); 
else if(nType == DOC)
PostMessage(hwndOtherUse, DO_CREATE, MAKEWPARAM(1, 0), 0);
 
}

It works fine.
But... when I take the line MessageBox(hwndOtherUse,cFile,0,0); out, all of the titles are the same on all of the items.

I have not the slightest idea why displaying some text in a message box makes it work fine, an without it, not work.

Please help, thanks, August.

**jmd15** · 09-23-2005

I don't think this is the problem but I don't think you need the "{" and "}" around your case DO_CREATE:. It's in a switch and it could just look like this:

Code:

case DO_CREATE:
//all your code here without those wrapping it up
return 0;    //the return 0; is what tells it that case is done
//break; works as well but I prefer return 0; for cases like that
//and break; for "sub-cases" like in WM_COMMAND:

Just a suggestion, doubt it's the problem though.

**Queatrix** · 09-23-2005

Nope, tried that. Didn't help at all.
Any other suggestions. Anyone at all?

**anonytmouse** · 09-23-2005

MessageBox runs an internal message loop. This means waiting messages will be dispatched while waiting for MessageBox to return. This fact is causing the behaviour you are seeing.

Consider the case without the message box call. You call PostMessage to add each item. This adds these messages to the message queue but they will not be dispatched until you pump the message queue. You only do this after you have added all the files. Therefore, since you are using a global variable for the title, it will be the same for all items.

When you call MessageBox and implicitly pump the messsage queue, the message is dispatched after each time and you get the correct titles.

This is one of the reasons you should try to avoid global variables. In this case you should use SendMessage instead of PostMessage. SendMessage will call the target window procedure directly and will not return until it is complete.

**novacain** · 09-25-2005

Or use the macros for this sort of thing

ie

TreeView_InsertItem() //in commctrl.h

**Queatrix** · 09-26-2005

Okay, even so, is there a way I can do what a MessageBox() does without displaying a message box?

Thread: Another tree-view problem.

Thread Tools

Search Thread

Display

Another tree-view problem.

Similar Threads

binary tree token problem

searching and insertion in a binary search tree

problem in storing data in a binary search tree

Binary tree problem

problem on tree implementation