unordered_set <foo>作为Foo的数据成员?

时间:2017-02-28 18:53:07

标签: c++ unordered-set

正如标题所说,我正在尝试使用包含Foo类对象的unordered_set作为类Foo的数据成员。这在C ++中是否可行?

我有这段代码:

#include <unordered_set>
using namespace std;

struct FooHash;

class Foo {
public:
    int id;
    unordered_set<Foo, FooHash> foos; // error here
    bool operator==(const Foo& foo) {
        return id == foo.id;
    }
};

struct FooHash {
    size_t operator()(const Foo& foo) const {
        return foo.id;
    }
};

int main() {
    Foo f;
    unordered_set<Foo, FooHash> foos;
    return 0;
}

但它会引发以下错误:

In file included from /usr/include/c++/6.3.1/bits/hashtable.h:35:0,
                 from /usr/include/c++/6.3.1/unordered_set:47,
                 from main.cpp:1:
/usr/include/c++/6.3.1/bits/hashtable_policy.h: In instantiation of ‘struct std::__detail::__is_noexcept_hash<Foo, FooHash>’:
/usr/include/c++/6.3.1/type_traits:143:12:   required from ‘struct std::__and_<std::__is_fast_hash<FooHash>, std::__detail::__is_noexcept_hash<Foo, FooHash> >’
/usr/include/c++/6.3.1/type_traits:154:38:   required from ‘struct std::__not_<std::__and_<std::__is_fast_hash<FooHash>, std::__detail::__is_noexcept_hash<Foo, FooHash> > >’
/usr/include/c++/6.3.1/bits/unordered_set.h:95:63:   required from ‘class std::unordered_set<Foo, FooHash>’
main.cpp:9:33:   required from here
/usr/include/c++/6.3.1/bits/hashtable_policy.h:85:34: error: no match for call to ‘(const FooHash) (const Foo&)’
  noexcept(declval<const _Hash&>()(declval<const _Key&>()))>
           ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/c++/6.3.1/bits/move.h:57:0,
                 from /usr/include/c++/6.3.1/bits/stl_pair.h:59,
                 from /usr/include/c++/6.3.1/utility:70,
                 from /usr/include/c++/6.3.1/unordered_set:38,
                 from main.cpp:1:
/usr/include/c++/6.3.1/type_traits: In instantiation of ‘struct std::__not_<std::__and_<std::__is_fast_hash<FooHash>, std::__detail::__is_noexcept_hash<Foo, FooHash> > >’:
/usr/include/c++/6.3.1/bits/unordered_set.h:95:63:   required from ‘class std::unordered_set<Foo, FooHash>’
main.cpp:9:33:   required from here
/usr/include/c++/6.3.1/type_traits:154:38: error: ‘value’ is not a member of ‘std::__and_<std::__is_fast_hash<FooHash>, std::__detail::__is_noexcept_hash<Foo, FooHash> >’
     : public integral_constant<bool, !_Pp::value>

向前声明这两个类并在声明这些方法之后给出这两个错误:

In file included from /usr/include/c++/6.3.1/unordered_set:44:0,
                 from main.cpp:1:
/usr/include/c++/6.3.1/ext/aligned_buffer.h: In instantiation of ‘struct __gnu_cxx::__aligned_buffer<Foo>’:
/usr/include/c++/6.3.1/bits/hashtable_policy.h:246:43:   required from ‘struct std::__detail::_Hash_node_value_base<Foo>’
/usr/include/c++/6.3.1/bits/hashtable_policy.h:277:12:   required from ‘struct std::__detail::_Hash_node<Foo, true>’
/usr/include/c++/6.3.1/bits/hashtable_policy.h:1894:60:   required from ‘struct std::__detail::_Hashtable_alloc<std::allocator<std::__detail::_Hash_node<Foo, true> > >’
/usr/include/c++/6.3.1/bits/hashtable.h:170:11:   required from ‘class std::_Hashtable<Foo, Foo, std::allocator<Foo>, std::__detail::_Identity, std::equal_to<Foo>, FooHash, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, true, true> >’
/usr/include/c++/6.3.1/bits/unordered_set.h:96:18:   required from ‘class std::unordered_set<Foo, FooHash>’
main.cpp:12:37:   required from here
/usr/include/c++/6.3.1/ext/aligned_buffer.h:85:34: error: invalid application of ‘sizeof’ to incomplete type ‘Foo’
     : std::aligned_storage<sizeof(_Tp), std::alignment_of<_Tp>::value>
                                  ^
/usr/include/c++/6.3.1/ext/aligned_buffer.h:85:34: error: invalid application of ‘sizeof’ to incomplete type ‘Foo’
/usr/include/c++/6.3.1/ext/aligned_buffer.h: In instantiation of ‘void* __gnu_cxx::__aligned_buffer<_Tp>::_M_addr() [with _Tp = Foo]’:
/usr/include/c++/6.3.1/ext/aligned_buffer.h:110:41:   required from ‘_Tp* __gnu_cxx::__aligned_buffer<_Tp>::_M_ptr() [with _Tp = Foo]’
/usr/include/c++/6.3.1/bits/hashtable_policy.h:250:34:   required from ‘_Value* std::__detail::_Hash_node_value_base<_Value>::_M_valptr() [with _Value = Foo]’
/usr/include/c++/6.3.1/bits/hashtable_policy.h:1971:36:   required from ‘void std::__detail::_Hashtable_alloc<_NodeAlloc>::_M_deallocate_node(std::__detail::_Hashtable_alloc<_NodeAlloc>::__node_type*) [with _NodeAlloc = std::allocator<std::__detail::_Hash_node<Foo, true> >; std::__detail::_Hashtable_alloc<_NodeAlloc>::__node_type = std::__detail::_Hash_node<Foo, true>]’
/usr/include/c++/6.3.1/bits/hashtable_policy.h:1984:22:   required from ‘void std::__detail::_Hashtable_alloc<_NodeAlloc>::_M_deallocate_nodes(std::__detail::_Hashtable_alloc<_NodeAlloc>::__node_type*) [with _NodeAlloc = std::allocator<std::__detail::_Hash_node<Foo, true> >; std::__detail::_Hashtable_alloc<_NodeAlloc>::__node_type = std::__detail::_Hash_node<Foo, true>]’
/usr/include/c++/6.3.1/bits/hashtable.h:1901:7:   required from ‘void std::_Hashtable<_Key, _Value, _Alloc, _ExtractKey, _Equal, _H1, _H2, _Hash, _RehashPolicy, _Traits>::clear() [with _Key = Foo; _Value = Foo; _Alloc = std::allocator<Foo>; _ExtractKey = std::__detail::_Identity; _Equal = std::equal_to<Foo>; _H1 = FooHash; _H2 = std::__detail::_Mod_range_hashing; _Hash = std::__detail::_Default_ranged_hash; _RehashPolicy = std::__detail::_Prime_rehash_policy; _Traits = std::__detail::_Hashtable_traits<true, true, true>]’
/usr/include/c++/6.3.1/bits/hashtable.h:1227:12:   required from ‘std::_Hashtable<_Key, _Value, _Alloc, _ExtractKey, _Equal, _H1, _H2, _Hash, _RehashPolicy, _Traits>::~_Hashtable() [with _Key = Foo; _Value = Foo; _Alloc = std::allocator<Foo>; _ExtractKey = std::__detail::_Identity; _Equal = std::equal_to<Foo>; _H1 = FooHash; _H2 = std::__detail::_Mod_range_hashing; _Hash = std::__detail::_Default_ranged_hash; _RehashPolicy = std::__detail::_Prime_rehash_policy; _Traits = std::__detail::_Hashtable_traits<true, true, true>]’
/usr/include/c++/6.3.1/bits/unordered_set.h:126:7:   required from here
/usr/include/c++/6.3.1/ext/aligned_buffer.h:99:36: error: using invalid field ‘__gnu_cxx::__aligned_buffer<_Tp>::_M_storage’
         return static_cast<void*>(&_M_storage);

标准中似乎不允许使用不完整类型的容器。我想尝试使用指针,但operator==不能为指针操作数重载。任何解决方法?

2 个答案:

答案 0 :(得分:3)

在将类用作unordered_set的模板参数之前,您需要完全定义类。虽然{<1}}在自己的定义期间无法完全定义,但它的指针类型是。您可以非常轻松地将Foo替换为Foo以解决此问题。

unique_ptr<Foo>

答案 1 :(得分:2)

正如FrançoisAndrieux已在his answer中指出的那样,不完整的类型不能与std :: unordered_set一起使用。他使用std::unique_ptr提供了一个可能的解决方案。

使用std::unique_ptr的一个缺点可能是您的班级现在需要显式动态内存分配才能插入Foo::foos。在我看来,这是一个应该隐藏的实现细节。

另一个(可能更大)问题可能是你松散值语义,因为std :: unique_ptr只能被移动而不能被复制。您必须实现自定义复制构造函数和赋值运算符才能获得值语义。

使用boost::recursive_wrapper,还有另一种可能隐藏指针的解决方案。虽然boost::recursive_wrapper是为boost::variant发明的,但它可用于其他不允许不完整类型的情况。

使用recursive_wrapper,Foo类变为:

class Foo {
public:
    using Wrapper = boost::recursive_wrapper<Foo>;

    Foo() = default;
    explicit Foo( int id ) : id( id ) {}

    struct Hash {
        size_t operator()(const Wrapper& foo) const { return foo.get().id; }
    };

    struct KeyEqual {
        bool operator()(const Wrapper& foo1, const Wrapper& foo2) const {
            return foo1.get().id == foo2.get().id; 
        }
    };

    using Set = std::unordered_set< Wrapper, Hash, KeyEqual >;
    Set foos;
    int id = 0;
};

我使用std::unordered_set的第3个模板参数 KeyEqual 进行了另一项更改,以独立于Foo::operator==定义密钥的相等性。现在Foo::operator==可以通过比较Foo的所有成员来实现,而不仅仅是 id 。这留给读者练习。

要在Foo::foos中插入和查找项目,您可以定期创建Foo实例,而无需使用动态内存分配。在幕后,boost::recursive_wrapper将动态分配内存。

当您通过Foo::Set的迭代器访问Foo时,您将注意到使用包装器的唯一地方是,因为您必须调用boost::reference_wrapper::get()。这并不比使用需要使用双重间接的std::unique_ptr更糟糕(即(*it)->id)。

使用示例:

int main() {
    Foo f1;

    // No explicit dynamic memory allocation here! 
    f1.foos.insert( Foo( 1 ) );
    f1.foos.insert( Foo( 1 ) );
    f1.foos.insert( Foo( 2 ) );

    // Value semantics are kept.
    Foo f2 = f1;

    f2.foos.erase( Foo( 1 ) );

    std::cout << "f1.size: " << f1.foos.size() << std::endl;
    std::cout << "f2.size: " << f2.foos.size() << std::endl;

    // Find a Foo, still clean syntax.
    auto it = f1.foos.find( Foo( 2 ) );
    if( it != f1.foos.end() )
    {
        // Only when accessing Foo through the iterator we notice
        // the recursive_wrapper because we must call its get() method. 
        std::cout << "found a Foo with id: " << it->get().id << std::endl;
    }    
    return 0;
}

完整演示:http://coliru.stacked-crooked.com/a/00d2a48106f2cc99