C++“错误：将'const std::map<int, std::basic_string<char>>'作为...的'this'参数传递”

Question

使用以下代码（为简洁起见）：

颜色.h：

class color {
public:
    color();

    enum colorType {
        black, blue, green, cyan, red,
        magenta, brown, lightgray, nocolor
    };

    colorType getColorType();
    void setColorType(colorType cColortype);

    string getColorText() const;

private:
    colorType cColortype = nocolor;
    map<int, string> colors = {
        {black, "black"},
        {blue, "blue"},
        {green, "green"},
        {cyan, "cyan"},
        {red, "red"},
        {magenta, "magenta"},
        {brown, "brown"},
        {lightgray, "lightgray"},
        {nocolor, "nocolor"}};
};

颜色.cpp：

color::color() {
}

color::colorType color::getColorType() {
    return cColortype;
}

void color::setColorType(colorType cColortype) {
    this->cColortype = cColortype;
}

string color::getColorText() const {
    return colors[cColortype];
}

我收到以下错误：

color.cpp:16:29: 错误：将 'const std::map >' 作为 'std::map<_key _tp _compare _alloc>::mapped_type& std::map::operator[](std::map<_key _tp _compare _alloc>::key_type&&) [with _Key = int; _Tp = std::basic_string; _比较 = std::less; _Alloc = std::allocator >>; std::map<_key _tp _compare _alloc>::mapped_type = std::basic_string; std::map<_key _tp _compare _alloc>::key_type = int]' 丢弃限定符 [-fpermissive]

错误是指“返回颜色[cColortype];”在 getColorText 中。

我正在为一个班级项目编写此内容，我可以通过删除 getColorText 签名中的 const 声明来使其工作，但我正在尝试学习/采用良好做法并遵守建议将 const 用于不修改数据的成员函数，所以我想知道如何处理这个问题。

我通常非常擅长调试/故障排除，但错误消息太复杂以至于没有多大帮助。

感谢任何帮助。

Answer 1

string color::getColorText() const {
    return colors[cColortype];
}

问题是您已将函数标记为const。 std::map 上的 operator[] 被标记为非常量，并且不能在这样的 const 函数中使用。您需要手动使用std::map::find（或其他机制）搜索输入类型并处理未找到的情况。

如果您使用的是 C++11，则可以改用 std::map::at，它允许在常量映射上使用，如果请求的元素不存在则抛出异常。

Answer 2

关键是接近尾声：“丢弃限定符”。 getColorText 是 const 成员函数，所以 colors 是 const。但是map::operator[]() 不是const。

Answer 3

首先：映射map<int, string> colors 必须是从 cColorType 到 string 而不是 int 的映射：

map<cColorType, string> colors

第二：正如一些人已经回答的那样：map::operator[]() 不是const。原因是这个操作符返回一个引用，它允许你修改它的值。

让我建议以下解决方案：您可以创建第二个私有属性：字符串格式的颜色。因此，您将拥有 2 个 Get 函数（每种颜色一个）和一个 Set 函数（将修改 2 个颜色属性）。