自定义 STL 容器

Question

我编写的代码允许按照输入的顺序遍历映射数据。

我编写了几次的解决方案是：

给定键类型 K 和数据类型 D， 标准::地图 标准::向量

如果想随机查找数据条目，请使用map.find(K)。如果想按入口顺序遍历地图，请使用std::vector::iterator (begin(), end()]。

这很好，但作为一个练习，我想把这个'OrderedMap' 写成一个符合 STL 的容器。我也有（精简到这个讨论）：

template <typename K, typename D>
class OrderedMapValue
{
private:
    K first_ref;
    std::map<K,size_t>& m;
    std::vector<D>& v;
public:
    const K& first
    D& second
    assignment operator=(const D& data) 
    {
        std::map<K,size_t>::const_iterator iter = m.find(first_ref);
        v[iter.second] = data;  // error checking of iter stripped
    }
};

进一步假设

template <typename K, typename D>
class OrderedMap
{
public:
     typename OrderedMapValue<K,D>& OrderedMap<K,D>::operator[](const K&);
     // snip...
};

class MyClass
{
public:
    MyClass(std::string s) : _my_data(s) {}
private:
    std::string _my_data;
};

以下代码有效：

OrderedMap<std::string,MyClass*> omap;
omap["MyKey"] = new MyClass("dummy");

但是，这段代码没有：

OrderedMap::iterator iter = omap.find("MyKey");
MyClass * obj = iter->second;
delete obj;
iter->second = new MyClass("dummy");

假设我做了什么 a) 结构愚蠢或 b) 不必要的复杂，应该怎么做？

我意识到我很可能在这里重新发明轮子，但同样，这项工作主要是为了增加我对 STL 容器、它们的设计模式和正确使用的知识。

提前感谢您提供任何见解，

Answer 1

我现在没有编译器来测试这个，所以可能会出现错误，但我认为你希望它更像：

template <typename K, typename D>
class OrderedMap
{
private:
        std::map<K,size_t> &m;
        std::vector<D> &v;
public:
    typename pair<K,D> TYPE;

        TYPE& operator[](const K &k)
    {
        return v[ m[ k ]];
    }

    TYPE& operator[](size_t idx)
    {
        return v[ idx ];
    }

    pair<iterator,bool> insert( const TYPE& pair )
    {
        map<K, size_t>::const_iterator iter;
        iter = m.find( pair.first );

        if( iter != m.end() )
            return make_pair( v[ iter.second], false );

        m.insert( make_pair( pair->first, v.size() ));
        v.push_back( pair->second );

        return make_pair(  v.last() , inserted );
    }

    iterator &begin()
    {
        return v.begin();
    }
    // etc
};

Answer 2

在OrderedMapValue::operator=，你有：

    std::map<K,size_t>::const_iterator iter = m.find(first_ref);

first_ref 是什么？该代码没有在其他地方引用它（没有双关语）。在我看来，它可能是旧实现的痕迹，在其他地方被公共成员取代

const K& first。

这可能是问题吗？

EDIT 来自 cmets：代码没有显示 first_ref 在任何地方初始化；所以据我所知，对m.find(first_ref) 的调用正在搜索一个空字符串，而不是 OrderedMapValue 的键。