Форум Flasher.ru
Ближайшие курсы в Школе RealTime
Список интенсивных курсов: [см.]  
  
Специальные предложения: [см.]  
  
 
Блоги Правила Справка Пользователи Календарь Сообщения за день
 

Вернуться   Форум Flasher.ru > Flash > ActionScript 3.0

Версия для печати  Отправить по электронной почте    « Предыдущая тема | Следующая тема »  
Опции темы Опции просмотра
 
Создать новую тему Ответ
Старый 17.09.2010, 17:49
Dulea вне форума Посмотреть профиль Отправить личное сообщение для Dulea Найти все сообщения от Dulea
  № 1  
Ответить с цитированием
Dulea

Регистрация: Sep 2010
Сообщений: 2
По умолчанию Одно событие для нескольких спрайтов

Здравствуйте. Помогите пожалуйста оптимизировать код.

В примере у меня игра "Морской бой".

Код создания первого трех-палубника следующий:

Код AS3:
//3-палубник A
var shipCell3a:Field = new Field();
shipCell3a.gotoAndStop(23); 
shipCell3a.x = 100;
shipCell3a.y = 100;
addChild(shipCell3a);
 
shipCell3a.addEventListener(MouseEvent.MOUSE_DOWN, startShipDrag3a);
shipCell3a.addEventListener(MouseEvent.MOUSE_UP, stopShipDrag3a);
shipCell3a.addEventListener(Event.ENTER_FRAME, dragshipCell3a);
 
var clickOffSet3a:Point = null;
 
function startShipDrag3a(event:MouseEvent)
{
	setChildIndex(event.target as DisplayObject, numChildren - 1);
	clickOffSet3a = new Point(event.localX,event.localY);	
}
 
function stopShipDrag3a(event:MouseEvent)
{
	clickOffSet3a = null;
}
 
function dragshipCell3a(event:Event)
{
	if (clickOffSet3a != null)
	{
		shipCell3a.x = mouseX - clickOffSet3a.x;
		shipCell3a.y = mouseY - clickOffSet3a.y;
	}
}
Для второго трех-палубника аналогичный код, только индекс "3a" везде меняется на "3b".
Как я могу оптимизировать код, и использовать только одно событие для нескольких корабликов?

Спасибо.

Старый 17.09.2010, 21:12
expl вне форума Посмотреть профиль Отправить личное сообщение для expl Найти все сообщения от expl
  № 2  
Ответить с цитированием
expl

блогер
Регистрация: Feb 2006
Сообщений: 1,474
Записей в блоге: 3
Не надо путать оптимизацию с рефакторингом

Самое минимальное что можно сделать - заменить clickOffSet3a и clickOffset3b на словать:
Код AS3:
var clickOffSetByShipCell:Dictionary = new Dictionary();
 
function startShipDrag(event:MouseEvent)
{
        var shipCell:Field = event.target;
	setChildIndex(shipCell, numChildren - 1);
	clickOffSetByShipCell[shipCell] = new Point(event.localX,event.localY);	
}
 
function stopShipDrag(event:MouseEvent)
{
        var shipCell:Field = event.target;
	clickOffSetByShipCell[shipCell] = null;
}
 
function dragshipCell(event:Event)
{
        var shipCell:Field = event.target;
        var offset:Point = clickOffSetByShipCell[shipCell];
	if (offset != null)
	{
		shipCell3a.x = mouseX - offset.x;
		shipCell3a.y = mouseY - offset.y;
	}
}
После этого в голову должны прийти более светлые идеи, ну или идея "ну ё нафиг и так сойдет".

Старый 18.09.2010, 01:03
Dulea вне форума Посмотреть профиль Отправить личное сообщение для Dulea Найти все сообщения от Dulea
  № 3  
Ответить с цитированием
Dulea

Регистрация: Sep 2010
Сообщений: 2
Спасибо за дельный совет, буду пробовать.

Создать новую тему Ответ Часовой пояс GMT +4, время: 14:49.
Быстрый переход
  « Предыдущая тема | Следующая тема »  

Теги
события оптимизация

Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.


 


Часовой пояс GMT +4, время: 14:49.


Copyright © 1999-2008 Flasher.ru. All rights reserved.
Работает на vBulletin®. Copyright ©2000 - 2026, Jelsoft Enterprises Ltd. Перевод: zCarot
Администрация сайта не несёт ответственности за любую предоставленную посетителями информацию. Подробнее см. Правила.