Как в операторе [] правильно вернуть указатель на внутренний vector/map?

Question

[Сергей Савостин]

Даже стыдно спрашивать. Хочется простого:

MyMap my;
my["one"] = 1;
my["second"] = 2;
int res = my["one"];

Вроде бы ничего сложного:

class MyMap
{
private:
   map<const char*, int> data;
public:
   MyMap() {}
   ~MyMap() {}
   int & operator [] (const char* name) { return data[name]; }
};

Но в результате получается странное:

my.count() : 3

one : 1
two : 2
one : 0

res : 0

Где этот чёртов детсадовский косяк? И вообще WTF?
P.S. К сожалению C++98

Comments

[Сергей Савостин]

Даже так:
my["one"] = 1;
my["one"] = 2;
my["one"] = 3;
в результате дает
my.count() : 3
!!!

Answer 1

[Sumor]

map это по сути словарик и он не допускает двух элементов с одним ключом. Если вы пытаетесь добавить значение с существующим ключом, то в вашем случае оно просто заменяется.
Возможно вам надо использовать set вместо map.

Comments

[Сергей Савостин]

Я как раз не пытаюсь добавить, я пытаюсь присвоить и получить значение.
Мне нужно, чтобы получилось:
my.count() : 2

one : 1
two : 2

res : 1

[Сергей Савостин]

Оно как раз почему-то не заменяется...

[Sumor]

MyMap map;

map["1"] = 1;
map["2"] = 2;
int res = map["1"];
std::cout << res;
map["1"] = 0;
res = map["1"];
std::cout << res;

У меня такой код возвращает 1 и 0. В конце размер map остаётся равен 2.

[Сергей Савостин]

Borland C++ 5.6 for Win32 :(((((((
Действительно, похоже дело в компиляторе.

[Сергей Савостин]

С
map<const string, int> data
все работает отлично.

[jcmvbkbc]

@savostin не в компиляторе дело. вы ж указатели сравнивали используя map<const char *, int>, а не строчки. Но можно было остановиться и на варианте с const char*, если гарантировать, что указатели всегда будут указывать на исходные строки, заменив map<const char *, int> на map<const char *, int, CompareString>, и определив

class CompareString {
public:
  bool operator ()(const char *a, const char *b) const {
    return strcmp(a, b) < 0;
  }
};

[Сергей Савостин]

Спасибо, интересное решение.

[AxisPod]

Да что гадать, дело тут в компиляторе, т.к. сохраняется адрес константы, а адреса сравнивать в данном случае нельзя, они могут быть разными. И проверять тут ничего не стоит.

map, тут const не нужен, const автоматом применяется для первого ключа pair.

[AxisPod]

На деле в данном примере обычное непредвиденное поведение.

Answer 2

[Sergei Borisov]

Странно. Код правильный. А чем компилируете? Я на gcc 4.6.3 с опцией -std=c++98 прогнал. все работает как ожидается.

Comments

[Сергей Савостин]

Borland C++ 5.6 for Win32 :(((((((
Действительно, похоже дело в компиляторе.

[tsarevfs]

@savostin не стоит раньше времени подозревать компилятор.

[Сергей Савостин]

Да, но на linux все работает как должно. В чем тогда фигня?

Answer 3

[tsarevfs]

Покажите код целиком, с выводом.

Answer 4

[ilejn]

А где у Вас при использовании
map<const char*, int>
хранятся ключи? Где под них память выделяется?

Если нигде (к примеру, Вы выходите за пределы функции), то
map<std::string, int> выглядит более здраво.

Comments

[Сергей Савостин]

Да, уже вижу. Спасибо.

[AxisPod]

В принципе можно юзать и как в источнике, т.к. константа разыменуется, она получит свой адрес. Где и будет лежать, проблема в другом, константы может быть 2, вот тогда естественно ничего и не полетит.