1

我的代码最初看起来像这样:

int SomeObject::operator[]( const string& key ) const
{
    return ( *m_Map )[ key ];
}

int SomeObject::operator()( const string& key ) const
{
    return ( *m_Map2 )[ key ];
}

这两张地图都有以下签名:

std::map< std::string, int >

然后我读了一些关于 STL 容器真的不需要显式堆分配(即std::map< ... >* map = new std::map< ... >)的东西,这就是我正在做的事情。

一旦我将映射更改为堆栈分配并删除指针取消引用,它看起来像这样:

int SomeObject::operator[]( const string& key ) const
{
    return m_Map[ key ];
}

int SomeObject::operator()( const string& key ) const
{
    return m_Map2[ key ];
}

编译器抱怨以下错误(对于两个地图):

Error   1   error C2678: binary '[' : no operator found which takes a left-hand operand of type 'const std::map<_Kty,_Ty>' (or there is no acceptable conversion)

笏。

4

1 回答 1

5

问题是您已将函数标记为const修改映射(如果键operator[]()不在std::map映射中,则添加一个带有默认值的元素)。

使用指针时可以避免这种情况,因为const仅适用于成员指针,而不适用于指针所指的对象。

像下面这样的东西应该可以解决这个const问题:

int SomeObject::operator[]( const string& key ) const
{
    std::map<string,int>::const_iterator it(m_Map.find(key));

    if (it == m_Map.end()) {
        throw /* something */;
    }

    return it->second;
}
于 2012-12-04T08:17:11.537 回答