Модификатор доступа const

@e-loki · Регистрация: 05.02.2015

Author24 — интернет-сервис помощи студентам

Здравсвуйте, прошу помочь с вопросом.

В общем имеется такой класс:

C++

class Animator : public sf::Drawable
{
private:
    std::map<std::string, Animation> m_animationList;
    std::string m_currentAnimation;
 
...
 
private:
    virtual void draw(sf::RenderTarget &target, sf::RenderStates states = sf::RenderStates::Default) const;
};

метод draw должен быть приблизительно таким:

C++

void Animator::draw(sf::RenderTarget &target, sf::RenderStates states) const
{
    target.draw(m_animationList[m_currentAnimation]);
}

тут возникает проблема. Когда я компилирую вылазит ошибка: "passing 'const std::map<std::basic_string<char>, Animation>' as 'this' argument...", ошибка, как понимаю, возникает из-за того, что метод operator [] не может гарантировать, что this не будет изменен. Можно конечно добавить указатель который будет хранить текущее состояние m_currentAnimation, но такой способ не сильно привлекателен.
Подскажите, если можете.
Спасибо!

Croessmah · 18.02.2016, 01:27

Сообщение от e-loki

ошибка, как понимаю, возникает из-за того, что метод operator [] не может гарантировать, что this не будет изменен.

Да, operator[] не гарантирует константность map'а,
т.к. в случае отсутствия элемента, он будет вставлен.

Сообщение от e-loki

Подскажите, если можете.

Что мешает использовать

C++

1	target.draw(*m_animationList.find(m_currentAnimation));//Конечно же, лучше проверить возвращаемое значение на валидность.

Ну и можно сделать

C++

1	mutable std::map<std::string, Animation> m_animationList;

@avgoor · 18.02.2016, 01:57

Сообщение от Croessmah

Ну и можно сделать

C++

1	mutable std::map<std::string, Animation> m_animationList;

ИМХО, плохая идея. Семантически m_animationList никак не может быть mutable. Если так сделать - хрен знает какие итераторы и где может порушить простая отрисовка, что ведет к неприятным, трудно воспроизводимым багам.

Croessmah · 18.02.2016, 02:17

Сообщение от avgoor

Если так сделать - хрен знает какие итераторы и где может порушить простая отрисовка

Если знаешь контекст, то вполне можно.
конкретно в данном случае,
лучше сделать через find,
но почему автор так не сделал сразу - не понятно,
ждемс ответа на поставленный вопрос.

@e-loki · 18.02.2016, 08:39 **[ТС]**

Croessmah,

Сообщение от Croessmah

но почему автор так не сделал сразу - не понятно,

Если честно, то просто не догадался - мало опыта. Вариант с find мне вполне по душе. Спасибо!

Croessmah Неэпический 18099 / 10685 / 2061 Регистрация: 27.09.2012 Сообщений: 26,895 Записей в блоге: 1
	18.02.2016, 02:17	4
	Сообщение от avgoor Если так сделать - хрен знает какие итераторы и где может порушить простая отрисовка Если знаешь контекст, то вполне можно. конкретно в данном случае, лучше сделать через find, но почему автор так не сделал сразу - не понятно, ждемс ответа на поставленный вопрос. 0

@e-loki 0 / 0 / 0 Регистрация: 05.02.2015 Сообщений: 23
	18.02.2016, 08:39 [ТС]	5
	Croessmah, Сообщение от Croessmah но почему автор так не сделал сразу - не понятно, Если честно, то просто не догадался - мало опыта. Вариант с find мне вполне по душе. Спасибо! 0