2012-02-04 2 views
6

У меня есть куча объектов в иерархии классов и хотелось бы сделать std::map, используя ссылки на эти объекты в качестве ключей на карте. Кажется, что std::reference_wrapper будет именно тем, что необходимо для этого, но я не могу заставить его работать. То, что я пытался до сих пор:Использование std :: reference_wrapper в качестве ключа в std :: map

class Object { // base class of my hierarchy 
    // most details unimportant 
public 
    virtual bool operator< (const Object &) const; // comparison operator 
}; 

std::map<std::reference_wrapper<const Object>, int> table; 

auto it = table.find(object); 

table[object] = 42; 

table[object]++ 

Однако, я всегда получаю несколько непонятные ошибки компилятора:

/usr/include/c++/4.5.3/bits/stl_function.h: In member function ‘bool std::less<_Tp>::operator()(const _Tp&, const _Tp&) const [with _Tp = std::reference_wrapper<const Object>]’: 
/usr/include/c++/4.5.3/bits/stl_tree.h:1522:38: instantiated from ‘std::_Rb_tree<_Key, _Val, _KeyOfValue, _Compare, _Alloc>::iterator std::_Rb_tree<_Key, _Val, _KeyOfValue, _Compare, _Alloc>::find(const _Key&) [with _Key = std::reference_wrapper<const Object>, _Val = std::pair<const std::reference_wrapper<const Object>, int>, _KeyOfValue = std::_Select1st<std::pair<const std::reference_wrapper<const Object>, int> >, _Compare = std::less<std::reference_wrapper<const Object> >, _Alloc = std::allocator<std::pair<const std::reference_wrapper<const Object>, int> >, std::_Rb_tree<_Key, _Val, _KeyOfValue, _Compare, _Alloc>::iterator = std::_Rb_tree_iterator<std::pair<const std::reference_wrapper<const Object>, int> >]’ 
/usr/include/c++/4.5.3/bits/stl_map.h:697:29: instantiated from ‘std::map<_Key, _Tp, _Compare, _Alloc>::iterator std::map<_Key, _Tp, _Compare, _Alloc>::find(const key_type&)[with _Key = std::reference_wrapper<const Object>, _Tp = int, _Compare = std::less<std::reference_wrapper<const Object> >, _Alloc = std::allocator<std::pair<const std::reference_wrapper<const Object>, int> >, std::map<_Key, _Tp, _Compare, _Alloc>::iterator = std::_Rb_tree_iterator<std::pair<const std::reference_wrapper<const Object>, int> >, key_type = std::reference_wrapper<const Object>]’ 
testfile.cpp:39:31: instantiated from here 
/include/c++/4.5.3/bits/stl_function.h:230:22: error: no match for ‘operator<’ in ‘__x < __y’ 

кажется ошибка, говорит, что не может сравнить два std::reference_wrapper<const Object> объектов, но кажется, что это должно быть возможно - std::reference_wrapper имеет оператора преобразования, который может неявно преобразовывать его в T& (const Object & здесь), а Object имеет operator <, так почему же он не работает?

Должно ли это работать, и это всего лишь ошибка в g ++? Или что-то еще происходит?

+0

Does 'std :: ref (object) dalle

ответ

5

Кажется, что это сработает, если вы сделали оператор сравнения свободной функцией (которая, возможно, вызывает функцию виртуального члена).

Если это функция-член, a < b действительно означает a.operator<(b);, а неявные преобразования не рассматриваются для аргумента левой стороны.

1

В Visual Studio 11 Beta у меня такая же проблема. Использование бесплатной версии, которая вызывает оператор <, решает проблему.

#include<map> 
#include<iostream> 
using namespace::std; 

class Object { 
    int _n1; 
public: 

    Object(int n = 0):_n1(n){}; 
    bool operator < (const Object& rhs) const {return this->_n1 < rhs._n1;} 
    friend ostream &operator << (ostream &stream, const Object& o) { stream << o._n1 << " "; return stream;} 
}; 

struct ObjectLess{ 

    bool operator()(const Object& lhs, const Object& rhs) const 
    { 
     return lhs<rhs; 
    } 
}; 

int main(int argc, char* argv[]) 
{ 
    //This does not compile 
    //std::map<std::reference_wrapper<const Object>, string> table; 

    //Using the free function works 
    std::map<std::reference_wrapper<const Object>, string, ObjectLess> table; 

    Object a(1); 
    Object b(2); 
    Object c(3); 


    table[a]="One"; 
    table[c]="Three"; 
    table[b]="Two"; 

    for(auto y: table){ 
    cout << y.first << " " << y.second.c_str() << std::endl; 
} 


    return 0; 
} 
3

По умолчанию std::less<std::reference_wrapper<const Object>> используется, но не вперед operator<() к базовому типу.

Самый простой и concisest вариант, чтобы решить вашу проблему, чтобы определить std::less<const Object> (или std::greater<const Object>) в определении карты, как это:

std::map<std::reference_wrapper<const Object>, int, std::less<const Object>> table; 

Он будет работать правильно и, как ожидается, из-за implicit conversion of std::reference_wrapper to T& и implicit constructor of std::reference_wrapper.

Example.