cannot convert 'this' pointer from 'const CLayerEngine' to 'CLayerEngine &'

[USBLexus]

Описал функцию возврата указателя на класс CLayerEngine как const чтобы никто его повредить не смог

const CLayerEngine* COGraphEdit::GetLayerEngine(void)
{
return (CLayerEngine*)(&m_LayerEngine);
}

А при попытке вызова метода класса

GetLayerEngine()->AnyMethod();

происходит ошибка

cannot convert 'this' pointer from 'const CLayerEngine' to 'CLayerEngine &'

Что делать? Убрать const не хотелось бы Я хочу чтобы класс остался неизменным
можно сделать ((CLayerEngine*)GetLayerEngine())->AnyMethod();

Работает, но гарантирует ли эта запись сохранность класса?

[Alf]

Когда мне нужно гарантировать, чтобы клиент не мог изменить состояние класса, я обычно делаю следующее.

1. Описываю клиентский интерфейс, т.е. класс, состоящий из чисто виртуальных функций. Каждая из этих функций гарантированно оставляет состояние класса неизменным.

2. Делаю свой класс производным от интерфейса.

3. Передаю клиенту ссылку на интерфейс класса, а не на сам класс.

Такой подход полностью изолирует клиента от всех членов класса, которые могут изменить состояние класса.

[USBLexus]

Альф в принципе интерфейс у меня тоже описан
Пользователь может вызывать только функции public все опасные функции protected и private в том числе и данные Хотелось просто перестраховаться видимо не получится так сделать как я хотел?

[Alf]

Альф в принципе интерфейс у меня тоже описан

Однако в примере этого не видно. Функция GetLayerEngine возвращает указатель на класс CLayerEngine, а не на интерфейс. Поэтому проку от наличия интерфейса немного.

Пользователь может вызывать только функции public все опасные функции protected и private в том числе и данные Хотелось просто перестраховаться

Так это для любого класса справедливо. Только при этом получается, что после создания класса ты и сам не сможешь изменить его состояние. Иногда это хорошо, а иногда и не очень. Лучше все же разделить интерфейсы на интерфейс доступа к данным только на чтение (общедоступный) и интерфейс изменения состояния класса. Клиенту, который должен лишь читать состояние, передавать лишь ссылку на первый.

видимо не получится так сделать как я хотел?

Можно попробовать еще описать твой метод как

Код:

void AnyMethod(void) const;

Возможно, в этом случае сработает твой первый вариант (со ссылкой на константный класс).

P.S. И лучше все-таки не применять преобразование типов в стиле C (вроде (CLayerEngine*)(&m_LayerEngine)). Лучше применять стиль C++, он гораздо более предсказуем.