带有自定义键编译器错误的unordered_map

时间:2016-01-19 13:02:26

标签: c++ c++11 unordered-map

我有一个关于std::unordered_map和自定义类的问题,因为它是关键 我认为首先需要一些背景知识:

自定义类是变体数据类型,它实现基本的数字类型和std::string类 最近我的一个兄弟告诉我,如果该类支持数组和哈希表,那将是很好的。 “别再说了”我想并开始实现数组功能(使用std::vector),这非常有用,然后我实现了hashmap功能(使用unordered_map<Variant, Variant>)。

如果我理解正确,operator()的哈希函数(或unordered_map)必须符合我size_t (*) (const Key_Type &k) const;对象的专用版本的签名std::hash<Variant>应该这样做,不应该吗? 另外,unordered_map需要检查Key_Type是否相等,这应该可以通过operator==()进行,我是否正确?

无论如何,我得到了很多漂亮的编译器错误,在我看来,这是最有帮助的: /usr/include/c++/4.9/bits/hashtable_policy.h:85:33: error: no match for call to ‘(const std::hash<Variant>) (const Variant&)’

我真的不明白发生了什么,并且非常感谢任何有关正在发生的事情的见解。

下面是班级Variant的精简标题,我希望包含足够的信息(说实话,我担心这些信息太多,但我不确定可以省略什么)。
但是我遗漏了大部分实现细节,因为问题似乎只发生在专门的哈希对象中。

这是Variant标题的精简版:

class Variant
{
    private:
        enum Type {NONE = 0, LONG, DOUBLE, STRING, ARRAY, HASH_MAP};
        using Var = struct Var
        {
            union
            {
                int64_t l;
                double d;
                std::string *s;
                std::vector<Variant> *v;
                std::unordered_map<Variant, Variant> *h;
            };
            Type type = NONE;
        };

    public:
        //constructors, destructor and clear function
        Variant() : var() {}
        Variant(long val): Variant(){var.type = LONG; var.l = val;}
        Variant(double val) : Variant(){var.type = DOUBLE; var.d = val;}
        Variant(const std::string &val) : Variant(){var.type = STRING; var.s = new std::string(val);}
        template<typename T, typename... Args>Variant(T val, Args... args) : Variant() {set(val, args...);} //constructs an array

        Variant(const Variant &val); //calls default constructor as well
        Variant(Variant &&val) : Variant() {swap(*this, val);}

        ~Variant(){clear();}
        void clear();

        //set functions
        template<typename T, typename... Args> void set(const T val, Args... args){if(var.type == ARRAY)var.v->clear();add(val, args...);}
        void set(long val);
        void set(double val);
        void set(const std::string &val);
        void set(const Variant &val);

        //add functions
        template<typename T> void add(const T val){add(Variant(val));}
        template<typename T, typename... Args> void add(const T val, Args... args){add(Variant(val)); add(args...);}
        void add(const std::string &val){add(Variant(val));}
        void add(const Variant &val);

        //array access and evaluation functions
        Variant& operator[](const Variant &idx);
        size_t size() const {if(var.type == ARRAY)return var.v->size(); return 0;}
        std::unordered_map<Variant, Variant>::iterator begin(){if(var.type == HASH_MAP)return var.h->begin(); throw Exception("The internal type does not support iterators");}

        //operator= definitions
        template<typename T> Variant& operator=(const T val){set(val); return *this;}
        Variant& operator=(const std::string &val){set(val); return *this;}
        Variant& operator=(Variant val){swap(*this, val); return *this;}

        //operator definitions
        Variant& operator+=(const Variant &right);
        //and operator-=, ^= etc etc...

        //friend definitions (mainly comparison operators)
        friend void swap(Variant &left, Variant &right); //simple swap function
        friend bool operator==(const Variant &left, const Variant &right);
        friend bool operator!=(const Variant &left, const Variant &right);    
        friend std::hash<Variant>;

    private:
        Var var;
};


template <typename T>
inline void hash_combine(std::size_t& seed, const T &v)
{
    std::hash<T> hasher;
    seed ^= hasher(v) + 0x9e3779b9 + (seed<<6) + (seed>>2);
}

namespace std
{
    template<> struct hash<Variant>
    {
        size_t operator()(const Variant &x) const
        {
            if(x.var.type == Variant::DOUBLE)
                return std::hash<double>()(x.var.d);
            else if(x.var.type == Variant::LONG)
                return std::hash<int64_t>()(x.var.l);
            else if(x.var.type == Variant::STRING)
                return std::hash<std::string>()(*x.var.s);
            else if(x.var.type == Variant::ARRAY)
            {
                size_t seed = 0;
                for(size_t i = 0; i < x.var.v->size(); ++i)
                    hash_combine(seed, x.var.v->operator[](i));
                return seed;
            }
            else if(x.var.type == Variant::HASH_MAP)
            {
                size_t seed = 0;
                for(auto it = x.var.h->begin(); it != x.var.h->end(); ++it)
                {
                    hash_combine(seed, it->first);
                    hash_combine(seed, it->second);
                }
                return seed;
            }
            else if(x.var.type == Variant::NONE)
                return 0;
            else
                throw std::runtime_error("This Variant cannot be hashed");
        }
    };
}

inline void swap(Variant &left, Variant &right){Variant::Var tmp(left.var); left.var = right.var; right.var = tmp;}

bool operator==(const Variant &left, const Variant &right);
bool operator!=(const Variant &left, const Variant &right);

1 个答案:

答案 0 :(得分:1)

这里的问题是您在unordered_map<Variant, Variant>本身的定义中使用Variant。此时,您的hash专业化尚不可用,这就是编译器产生错误的原因。您不能仅在Variant定义之前移动哈希定义,因为哈希需要访问Variant成员。您可以做的是分离hash的声明和定义:

class Variant;

namespace std
{
    template<> struct hash<Variant>
    {
        size_t operator()(const Variant & x) const;
    };
}

class Variant {
/* Variant definition goes here ... */
};

template <typename T>
inline void hash_combine(std::size_t& seed, const T &v)
{
    std::hash<T> hasher;
    seed ^= hasher(v) + 0x9e3779b9 + (seed<<6) + (seed>>2);
}

size_t std::hash<Variant>::operator()(const Variant &x) const
{
    /* hash function implementation here ... */
}

但是你有另一个问题:在Variant类定义中,Variant本身是不完整的类型。在你的联盟中,你只存储指向vector和unordered_map的指针,这没关系,但是begin方法(实际上已经指定了它的返回类型)需要unordered_map<Variant, Variant>的实例化,这在那个地方是不可能的。

(注意:Limited support for containers of incomplete types(只有vector,list和forward_list)将添加到C ++中17)

要解决第二个问题,您可以使用map成员函数而不是begin函数来访问内部地图:

std::unordered_map<Variant, Variant> & map()
{
    if (var.type == HASH_MAP)
        return *var.h; 
    throw Exception("The internal type does not support iterators");
}

然后代替

Variant v;
v.begin();

你会用

v.map().begin();