const - как лучше?

[Igors]

Добрый день

Есть класс Texture и многие его методы const (не изменяют экземпляр). Но требуется отслеживать все созданные экземпляры. Я использую std::set (с тем же успехом можно QSet). Как мне его объявить: так

Код

C++ (Qt)
typedef std::set <Texture *> TextureSet;
 

Или так

Код

C++ (Qt)
typedef std::set <const Texture *> TextureSet;
 

В первом случае мне надо приводить const_cast всякий раз когда я добавляю/удаляю в/из set в константном методе (немало). Во втором всякий раз когда я итерирую TextureSet (меньше но тоже много).

Или есть лучшее решение чтобы избежать множества приведений?

Спасибо

[whirlwind]

можно сделать это поле mutable, тогда его можно будет менять из const методов

[pastor]

Непонятно, чем не устраивает первый способ хранения:

typedef std::set <Texture *> TextureSet;

Какие операции предполагается выполнять с объектами Texture? Вызывать его константные методы? Так способ храния здесь непричем. Если методы константные, то они не должены менять состояние объета, в противном случае: убираем "const" или ищем ошибку в проектировании класса.


При втором способе хранение весь объект константный. Нельзя менять его состояние, даже если он не содержит константных методов.

[Igors]

Непонятно, чем не устраивает первый способ хранения:

typedef std::set <Texture *> TextureSet;

Многим, напр нельзя так

Код

C++ (Qt)
// проверяем такая текстура уже есть в глобальном set
static bool LookupTexture( const Texture * iTexture )
{
 return theTextures.find(iTexture) != theTextures.end();
}
 

Если здесь убрать const - это потянет за собой др. методы.

Какие операции предполагается выполнять с объектами Texture? Вызывать его константные методы? Так способ храния здесь непричем. Если методы константные, то они не должены менять состояние объета, в противном случае: убираем "const" или ищем ошибку в проектировании класса.

При втором способе хранение весь объект константный. Нельзя менять его состояние, даже если он не содержит константных методов.

По книжке - да, а по жизни как?   Вот пример

Код

C++ (Qt)
if (!LookupTexture(iTexture))  {    // текстура  готова? 
 theTextures.insert(iTexture);       // нет, регистрируем ее 
 iTexture->CreateDataOpenGL();   // это не константный метод
|
 

можно сделать это поле mutable, тогда его можно будет менять из const методов

Полей тех много и они в очень многих др.  классах

[pastor]

Многим, напр нельзя так

Если здесь убрать const - это потянет за собой др. методы.

Что потянет? Какие методы?

Вот пример

Что не так с примером?

Код

C++ (Qt)
std::set <Texture *> theTextures;
 
Texture *iTexture = new Texture();
 
if (!LookupTexture(iTexture)) {
    theTextures.insert(iTexture);
    iTexture->CreateDataOpenGL();
}

В чем проблема то?

[twp]

typedef std::set <Texture *> TextureSet;

Многим, напр нельзя так

Код

C++ (Qt)
static bool LookupTexture( const Texture * iTexture )
{
 return theTextures.find(iTexture) != theTextures.end();
}
 

Если здесь убрать const - это потянет за собой др. методы.

Все верно, метод find не может автоматически привести const Texture * в Texture *.
В данном случае можно попробовать обявить так:
typedef std::set<Texture const *> TextureSet;
Хотя это может затронуть другие части кода в приложении.

[pastor]

Все верно, метод find не может автоматически привести const Texture * в Texture *.

Тут имеет место попытка приведения к Texture *const &, которое неможет быть выполнено. Это весьма очевидно если посомтреть как объявлен метод find для std::set или QSet


iterator find ( const key_type& x ) const;

iterator QSet::find ( const T & value );

[twp]

Все верно, метод find не может автоматически привести const Texture * в Texture *.

Тут имеет место попытка приведения к Texture *const &, которое неможет быть выполнено. Это весьма очевидно если посомтреть как объявлен метод find для std::set или QSet


iterator find ( const key_type& x ) const;

iterator QSet::find ( const T & value );

Может и так но gcc не согласен:
error: invalid conversion from 'const Texture*' to 'Texture*'

[Igors]

Что не так с примером?

Код

C++ (Qt)
std::set <Texture *> theTextures;
 
Texture *iTexture = new Texture();
 
if (!LookupTexture(iTexture)) {
    theTextures.insert(iTexture);
    iTexture->CreateDataOpenGL();
}

В чем проблема то?

В том что такие методы/ф-ции как LookupTexture очень часто нужны в константных методах