reference to temporary

[0...-5]

Проблема такая: в классе имеются в переменные следующего вида:
QMap<int, QList<int> > data_map
int data_ind

Далее объявляю метод:
const QList<int>& data() {return data_map[data_ind];}

на что получаю предупреждение при сборке:
returning reference to temporary.

Скажите, где тут временные переменные?
И самое интересное: если из метода убрать модификаторы const, то никаких предупреждений при сборке не выдается!

[alexman]

QMultiMap

[0...-5]

QMultiMap

Теперь еще и ошибка: оператор [] является закрытым для этого класса))

[minigo]

наверно вот так для начала const QList<int>& data() {return data_map[data_ind];}.

Но 2010 студия не выдаёт такой варнинг (сборка на варнинг левел 3). Ещё вот http://msdn.microsoft.com/en-us/library/a8kfxa78(VS.71).aspx

но конечно лучше увидеть код целиком.

[0...-5]

наверно вот так для начала const QList<int>& data() {return data_map[data_ind];}.

Но 2010 студия не выдаёт такой варнинг (сборка на варнинг левел 3). Ещё вот http://msdn.microsoft.com/en-us/library/a8kfxa78(VS.71).aspx

но конечно лучше увидеть код целиком.

Мне нужен именно метод const

[Fat-Zer]

1) как уже говорили нужен QMultiMap
2) используйте values() вместо operator[]
3) посмотрите на тип восвращаемого значения оператора [] и вам откроется, что было не так...

[mutineer]

QMultiMap

В чем смысл тут мультимап делать? Он же нигде не говорил что ключи повторяются

Мне нужен именно метод const

Ты амперсанд не там поставил

[Igors]

Далее объявляю метод:
const QList<int> data()& {return data_map[data_ind];}

на что получаю предупреждение при сборке:
returning reference to temporary.

Точнее с &, непонятно что Вы хотели

Так все нормально

Код

C++ (Qt)
const QList<int> & data() {return data_map[data_ind];} 
 

А так должны получить

Код

C++ (Qt)
const QList<int> & data() const {return data_map[data_ind];} 
 

[0...-5]

Ты амперсанд не там поставил

По поводу амперсанта все действительно так, я когда писал сообщение в тему поставил его не в нужное место, но от предупреждение это естественное не лечит.

1) как уже говорили нужен QMultiMap
2) используйте values() вместо operator[]
3) посмотрите на тип восвращаемого значения оператора [] и вам откроется, что было не так...

Насчет QMultiMap – это забавно, насколько я знаю это просто подкласс для хранения неуникальных пар ключ – значение.
Выдержка методов из документации:
QList<T> QMap::values (const key & key) const
То есть не возвращает ссылки, а возвращает список копий. Мне нужны именно ссылки.
Скорее всего проблема в том, что оператор [] возвращает неконстантную ссылку либо константную копию

Уточнение про амперсант:
Мне нужен метод, не модифицирующий класс при вызове и возвращающий немодифицируемую ссылку (в данном случае на список интов)

[Igors]

По поводу амперсанта все действительно так, я когда писал сообщение в тему поставил его не в нужное место, но от предупреждение это естественное не лечит.

Так подправьте свой первый пост чтобы не было конфузу

Мне нужен метод, не модифицирующий класс при вызове и возвращающий немодифицируемую ссылку (в данном случае на список интов)

Если метод const, то ссылку не получить по той причине что в ассоциативном контейнере ее может просто не быть. Но можно удовлетвориться константным указателем

Код

C++ (Qt)
const QList<int> * data( void ) const
{
  QMap<int, QList<int> >::const_iterator it = data_map.find(data_ind);
  return (it == data_map.end()) ? 0 : &it.value();
}

И затем обыграть возвращаемый 0

[0...-5]

По поводу амперсанта все действительно так, я когда писал сообщение в тему поставил его не в нужное место, но от предупреждение это естественное не лечит.

Так подправьте свой первый пост чтобы не было конфузу

Мне нужен метод, не модифицирующий класс при вызове и возвращающий немодифицируемую ссылку (в данном случае на список интов)

Если метод const, то ссылку не получить по той причине что в ассоциативном контейнере ее может просто не быть. Но можно удовлетвориться константным указателем

Код

C++ (Qt)
const QList<int> * data( void ) const
{
  QMap<int, QList<int> >::const_iterator it = data_map.find(data_ind);
  return (it == data_map.end()) ? 0 : &it.value();
}

И затем обыграть возвращаемый 0

Была такая мысля)))
Не смейтесь, просто мне приятнее работать со списком через [], а так придется пользовать at()
)))))

[Igors]

Была такая мысля)))
Не смейтесь, просто мне приятнее работать со списком через [], а так придется пользовать at()
)))))

Так приведите к ссылке

Код

C++ (Qt)
typedef QList<int> TIntList;
...
const TIntList * lstPtr = data();
if (!lstPtr) return;
const TIntList & lst = *lstPtr;
if (lst.size() && lst[0] == 1) ...
}

[0...-5]

Спасибо)))))Это лишние телодвижения, оставлю все в указателях