Question

当我尝试在VS2010中编译以下代码时，提示错误C2678。

#include <string>
#include <map>
using namespace std;
class test
{
    private:
        map<string, string> data;
    public:
        test(){};
        ~test(){};
    public:
        const string & get(const string & key)const{return data[key];}; //error C2678
        bool set(const string & key, const string & value){data[key]=value;return true;};
};
void main()
{
    const string key="Hello world!";
    const string value="I'm coming!";
    test t;
    t.set(key,value);
    t.get(key);
}

但是当我把它留作像

这样的功能时

#include <string>
#include <map>
using namespace std;
bool set(const string & key, const string & value, map<string, string> & data)
{
    data[key]=value;
    return true;
}
const string & get(const string & key, map<string, string> & data)
{
    return data[key];
}
void main()
{
    const string key="Hello world!";
    const string value="I'm coming!";
    map<string, string> data;
    set(key, value, data);
    get(key;
}

它确实编译并运行。

有谁知道问题是什么？

Answer 1

您已将测试类的get成员函数声明为const。但std::map operator[]是非const函数，因此无法从const函数调用它。请改用find功能。

operator[]非const的原因是因为如果该键不存在，那么它会将其与默认的构造值一起插入到映射中。

Answer 2

要在const地图中查找对象，您需要使用find成员函数，不能使用operator[]：

    const string & get(const string & key)const {return data.find(key)->second;}

请注意，这假定密钥始终存在。

在类</string，>中映射<string，string =“”>

2 个答案: