How do I speed up this treeview insert?

[Gunner]

Thanks everyone for the great tips, they all help speed up the tree loading from my original post!!!!  I have found a faster way..  When I insert a root node, I save the text to a listview item, and the handle to the node in the lparam of the lvitem, next time I go and load an item, instead of looping through ALL the tree nodes (which is where the bottleneck was)  I just do a LVM_FINDITEM and if the item is in the listview, get the lparam and and use that as the parent.   Got the loading down to about 2 and a half seconds for about 6000 items  :clap: 

I have a feeling I could remove the listview and do the same with code only... not sure though!  My quest continues for more speed!

[Tedd]

You can insert only the top-level nodes, and then respond to TVN_ITEMEXPANDING (it's delivered through WM_NOTIFY) and insert the expanded node's children at that point.
This will avoid the extra listview and searching.


TVN_GETDISPINFO isn't quite suitable - it's meant for items that change dynamically, so you can update the text/attributes whenever they're drawn - you'd still need to insert all of the items.

[oex]

You can insert only the top-level nodes, and then respond to TVN_ITEMEXPANDING (it's delivered through WM_NOTIFY) and insert the expanded node's children at that point.
This will avoid the extra listview and searching.

This isnt ideal, it is better to add 2 levels so the children are already there otherwise there is a delay on expand.... One level extra should suffice however it is just a pain to manage the items to add.... Long time since I did this in VB and this was easier than in ASM....

[sinsi]

An interesting read from Mr Chen
There's a default implementation for WM_SETREDRAW, but you might be able to do better