C“错误：将'const std:：map'作为…的'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];
}

我得到以下错误：

颜色cpp:16:29:错误：传递“const std:：map”

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

我写这篇文章是为了一个类项目，我可以通过删除getColorText签名中的const声明来让它工作，但我正在尝试学习/采用良好的实践，并遵循建议，在不修改数据的成员函数中使用const，所以我想知道如何处理这个问题。

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

感谢您的帮助。

Answer 1

第一：地图地图

map<cColorType, string> colors

第二：正如一些人已经回答过的：map::运算符[]（）不是const。原因是这个运算符返回一个引用，它允许您修改它的值。

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

Answer 2

关键就在最后：“放弃资格赛”getColorText是一个const成员函数，因此colors是const。但是map:：operator[]（）不是const。

Answer 3

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

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

如果您使用的是C11，您可以使用std::map::at，它允许在常量映射上使用，如果请求的元素不存在，则会引发异常。