C++ "ошибка: передача 'Const станд :: Карта <Int, станд :: basic_string <char>>' как 'этого' аргумент ..."

Question

С помощью следующего кода (Выдержки для краткости):

цвета. ч:

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"}}; 
}; 

color.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: error: passing 'const std::map >' as 'this' argument of 'std::map<_Key, _Tp, _Compare, _Alloc>::mapped_type& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](std::map<_Key, _Tp, _Compare, _Alloc>::key_type&&) [with _Key = int; _Tp = std::basic_string; _Compare = 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]' discards qualifiers [-fpermissive]

Ошибка относится к «цветам возврата [cColortype]»; в getColorText.

Я пишу это для проекта класса, и я могу заставить его работать ради назначения, удалив объявление const в подписи getColorText, но я пытаюсь изучить/принять хорошие практики и придерживаться рекомендуется использовать const для функций-членов, которые не изменяют данные, поэтому я хочу знать, как справиться с этим.

Я обычно хорошо разбираюсь в отладке/поиске неисправностей, но сообщение об ошибке настолько запутано, что это не очень помогает.

Любая помощь приветствуется.

Answer 1

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

Проблема в том, что вы отметили функцию как const. operator[] на std::map отмечен как неконстантный и не может использоваться в такой функции 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 в строку вместо междунар:

map<cColorType, string> colors 

Во-вторых: Как некоторые люди уже ответили: map::operator[]() не const. Причина этого в том, что этот оператор возвращает ссылку, которая позволяет вам изменять ее значение.

Позвольте мне предложить следующее решение: вы можете создать второй частный атрибут: цвет в строчном формате. Поэтому у вас будут две функции Get (по одному для каждого типа цвета) и одна функция Set (которая изменит два атрибута цвета).