Цитата:
|
Задумайтесь. Пользы от ваших постов не вижу, аргументируйте пожалуйста свои претензии.
|
Не, претензии были бы если бы этот код достался мне в поддержу, а так это просто мнение.
Исходник, который является предметом топика, к сожалению, открыть не могу, поэтому на примитивных примерах:
Способ 1

Код AS3:
button0.addEventListener(MouseEvent.CLICK, onSomeButtonClick)
button1.addEventListener(MouseEvent.CLICK, onSomeButtonClick)
function onSomeButtonClick(event:Event):void {
делаемОбщее;
switch (event.target)
{
case button0:
делаем одно; break;
case button1:
делаем другое; break;
}
}
Достоинтва:
- одна функция вместо 2-х
Недостатки:
- появилась услованая логика - источник багов (например очень просто вместо button1 написать button3, на которую вообще этот слушатель не навешан)
- допустим мы хотим расширить класс и добавить еще одну кнопку, по нажатии на которую должно "делаться общее" - придется лезть в свитч базового класса (или, еще веселее - копипастить его весь в клас-потомок и дописывать недостающее);
- допустим мы хотим поменять слушатель button1 на другой, если продолжать следовать этому подходу - мы что флаг будем делать? и в свиче еще ифом разбирать?
(утрирую конечно, можно так же отписать и подписать другой слушатель с другими свичами)
- нельзя перегрузить действие одной кнопки
(на самом деле можно, если кусок свича в функцию вынести)
Способ 2:
Недостатки:
- 2 функции вместо одной
- ?

Код AS3:
button0.addEventListener(MouseEvent.CLICK, onButton0Click)
button1.addEventListener(MouseEvent.CLICK, onButton1Click)
function onButton0Click(event:Event):void {
doCommon();
делаем одно
}
function onButton2Click(event:Event):void {
doCommon();
делаем другое
}
function doCommon():void { делаем общее; }