std::map probs

Hi,

i got following code:

Code:

---

typedef std::map<int, LPITEMIDLIST> IdIDListMap;
IdIDListMap map;

---

so far so good everything works expect that one:

Code:

---

int index = "SomeNumber";
LPITEMIDLIST lPIDL = map[index];

---

my compiler says:
Error 1 error C2678: binary '[' : no operator found which takes a left-hand operand of type 'const IdIDListMap' (or there is no acceptable conversion)

as far as i know the operator[] is declared that way:
TYPE& operator[]( key_type& key )
so what am i doing wrong.
Sorry if its horribly stupid or already answered, but i tried searching for sth similiar and gave up after two pages of wrong posts

Quote:

---

Code:

---

int index = "SomeNumber";

---

---

This is an error, you can't assign a string literal to an int.


For your map, I wouldn't use the [ ] operator, instead, prefer to use find(), which is a nonmodifying function.

Even if your map isn't declared const, I assume its declared as a member of a class (although perhaps you'd like to confirm where your 'map' comes from), and my guess is that you're probably trying to call this code from a const member function, which isn't allowed to call modifying functions of the map.

try this instead - (untested)

Code:

---

IdIDListMap::const_iterator iter = map.find(index);
LPITEMIDLIST lPIDL;
if( iter )
    lPIDL = iter->second;

---

When you use operator[], it automatically creates a new entry in the map if one doesn't exist. Sometimes that is what you want, in which case you have to make sure you aren't in a const member function or using a const map as Bench82 said.

If you don't want to create an element if none exists, then use find. The example for find should test it against end(), though, not null:

Code:

---

IdIDListMap::const_iterator iter = map.find(index);
LPITEMIDLIST lPIDL;
if( iter != map.end() )
    lPIDL = iter->second;

---

Thanks for that correction, Daved, I guess that's what they call a brain-fart.

Code:

---

int index = "SomeNumber";

---

was just an to show i get some number :D

//well i already figured out how to use find, but i would love to know why the compiler thinks it is illegal access.
// But thanks for the help.
Nevermind thanks Bench82 its exaclty what you say !

Quote:

---

Originally Posted by BeBu

Code:

---

int index = "SomeNumber";

---

was just an to show i get some number :D

well i already figured out how to use find, but i would love to know why the compiler thinks it is illegal access.
But thanks for the help.

---

somewhere you have made a 'const' promise to your compiler, and the [ ] operator breaks that promise, so the compiler complains. However, without seeing more of your code, there's no way to pinpoint where you've made that promise. (The promise doesn't have to be at the declaration of the map)

yes thanks
i figured it out:

its in a const class function.
dude i owe you one. next tiem you are in germany come and i sepnt you a good beer