C++ stl map: issues with BSTR

Question

I use a map in my code <BSTR,struct> bstr being the key and struct the value.

Will this work or do I have to redefine something?

I see no compile issues and I'm able to add elements too. However, map.find() does not work. Even though the element is present, it always returns map.end() (element not found).

I did a temporary workaround as follows - by looping from map.begin() to map.end() and doing a lstrcmpW for each element. This seems to work, but dont think this is too efficient.

Any suggestions/tips on what could be wrong? Is it ok to use BSTR as key for a map? I know maps do not support some of the non-native data types - structs or classes...u need to define a < operator for that.

Answer 1

Use ATL's CComBSTR as the key type instead of BSTR. CComBSTR overloads operator< to do actual string comparisson, rather than pointer (address) comparisson as you are currently doing.

CComBSTR also simplifies lifetime management. Using BSTR as the key type, you have to make sure that the BSTRs outlive the lifetime of the map (actually they have to be deallocated right before the map is destructed). CComBSTR follows the RAII principle so you don't have to do any manual deallocation.

Answer 2

BSTR type in C++ is a pointer. Map is comparing the pointers to each other and not the string. To use in map you should probably write a wrapper for BSTR or use a premade wrapper.