Как правильно расширить класс?

[djken]

Привет всем.
Подскажите пожалуйста как сделать правильно.
Есть класс ImageBox - некий контейнер, которому можно задавать размеры, в себя умеет загружать картинку и пустое место заполнять выбранным фоном.

От этого класса будут производные классы. Например, класс ImageBoxExt - делает все тоже самое что и ImageBox, но еще при наведении должны внутри появляться кнопка корзины и чекбокс.




Наследоваться от ImageBox?

public class ImageBoxExt extends ImageBox

Проблема: в конструкторе после super добавляю кнопку корзины и чекбокс. Но после того как загружается основная картинка, она закрывает все новосозданные кнопки ) Следовательно, в этом классе нужно перекрывать родительские методы - а это неправильно.

Либо просто создавать класс, наследуясь от Sprite и внутри создавать экземпляр класса ImageBox?

public class ImageBoxExt extends Sprite
{
   private var _ib:ImageBox;
...
   _ib = new ImageBox(...);
   addChild(_ib);
   // и тут добавлять новые кнопки
...
}

[undefined]

Следовательно, в этом классе нужно перекрывать родительские методы - а это неправильно.

Почему?

[djken]

Undefined, почему нужно перекрывать? Или почему это неправильно?

Перекрывать нужно, т.к. надо ловить момент загрузки картинки, что бы потом остальное дорисовывать сверху. Либо менять слои местами. Следовательно, нужны изменения в методах родительского класса.

А неправильно, т.к. это нарушение инкапсуляции..

[caseyryan]

Следовательно, в этом классе нужно перекрывать родительские методы - а это неправильно.

Неправильно? Это один из принципов ООП, так сказать, один из трёх китов)

А неправильно, т.к. это нарушение инкапсуляции..

Нет, это неправильная архитектура изначально.

Если изначально планируется, что в базовом классе грузится какая-то картинка, а объекты дочерних классов должны знать об этом, то и архитектуру родительского класса надо делать сразу с учетом этого.
Если нужно дождаться загрузки картинки, то можно обработчику сделать модификатор protected и в дочернем классе просто его перезаписать

// в базовом классе
protected function onImageLoaded(e:Event):void {
      ... добавляется картинка
}
// в наследнике
override protected function onImageLoaded(e:Event):void {
    super.onImageLoaded(e);
    .. тут добавляешь свои чекбоксы
}

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

override protected function onImageLoaded(e:Event):void {
    super.onImageLoaded(e);
    addCheckboxesAndOtherCrap();
}

как-то так

[djken]

caseyryan, спасибо большое!
Для личного пользования всегда обходился только private и public. Но в данном случае не хотелось использовать public у родительского метода для перезаписи в наследнике, т.к. onImageLoaded не относится к интерфейсу данного класса.

Теперь буду знать про protected )

то и архитектуру родительского класса надо делать сразу с учетом этого

Но все же - какая в данном случае архитектура была бы правильная? Или имелось ввиду использование protected?

[caseyryan]

Но все же - какая в данном случае архитектура была бы правильная? Или имелось ввиду использование protected?

Правильно в данном случае - иметь возможность уведомить наследников о загрузке картинки.

Теперь буду знать про protected )

protected это всего лишь один из 4 штатных модификаторов доступа, означает доступ для наследников. Есть еще internal для свойств доступных внутри пакета. Можно и свое пространство имен объявить, но тут это избыточно

[djken]

А, ну у меня в родительском методе вконце стоит диспатч события после загрузки картинки.

dispatchEvent(new Event('COMPLETE'));

Получается я могу его так же в наследнике слушать.. я что-то не додумался сразу до этого )

super.addEventListener('COMPLETE', onCompleteHandler);
...
 
private function onComplete(e:Event):void
{
	var _cb:CheckBox = new CheckBox();
	addChild(_cb);
}

Значит можно без override обойтись )
Пасиб! )

[caseyryan]

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

А, ну у меня в родительском методе вконце стоит диспатч события после загрузки картинки.

dispatchEvent(new Event('COMPLETE'));

Получается я могу его так же в наследнике слушать.. я что-то не додумался сразу до этого )

super.addEventListener('COMPLETE', onCompleteHandler);
...
 
private function onComplete(e:Event):void
{
	var _cb:CheckBox = new CheckBox();
	addChild(_cb);
}

Значит можно без override обойтись )
Пасиб! )

Ну, во-первых писать типы событий хардкодом типа "COMPLETE" - это моветон. Можно легко опечататься и потом думать и гадать, почему не работает. Всегда используй для этого константы.
Во-вторых, для события complete есть штатная константа Event.COMPLETE, даже свою объявлять не надо
Ну и в третьих - отправка события - это всегда более ресурсоемкий способ, чем вызов метода. У тетя там и так событие приходит при загрузке картинки. Создавать еще одно - это оверхэд. Оверрайд в данном случае более правильный подход. Не нужно ни слушателей дополнительных ни событий и всё по ООПшному
А если событие все-таки нужно для чего-то ещё, то ты можешь не создавать новое, а просто передиспатчить то же

dispatchEvent(e);

[GBee]

Так может картинку на 0 уровень всегда пихать после загрузки?

[undefined]

Значит можно без override обойтись

Тебе явно кто-то вбил в голову,что override - это плохо.Это не так.