Показать сообщение отдельно
Старый 08.03.2011, 15:29
expl вне форума Посмотреть профиль Отправить личное сообщение для expl Найти все сообщения от expl
  № 3  
Ответить с цитированием
expl

блогер
Регистрация: Feb 2006
Сообщений: 1,474
Записей в блоге: 3
Цитата:
Задумайтесь. Пользы от ваших постов не вижу, аргументируйте пожалуйста свои претензии.
Не, претензии были бы если бы этот код достался мне в поддержу, а так это просто мнение.

Исходник, который является предметом топика, к сожалению, открыть не могу, поэтому на примитивных примерах:
Способ 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 { делаем общее; }