Показать сообщение отдельно
Старый 06.12.2012, 19:04
iNils вне форума Посмотреть профиль Отправить личное сообщение для iNils Посетить домашнюю страницу iNils Найти все сообщения от iNils
  № 2  
Ответить с цитированием
iNils
Негуру
 
Аватар для iNils

администратор
Регистрация: Jan 2000
Адрес: Кёнигсберг in Moscow
Сообщений: 21,883
Записей в блоге: 7
Цитата:
Код AS3:
var GUNS:Array = new Array
GUNS.push (weapons.gun.guni)
for(var f:int=0;f<GUNS.length;f++)
{
	GUNS[f].addEventListener(MouseEvent.MOUSE_DOWN, mDown);
    GUNS[f].addEventListener(MouseEvent.MOUSE_UP, mUp);
 
	function mDown(e:MouseEvent):void
	{
		GUNS[f].startDrag(false);
	}
	function mUp(e:MouseEvent):void
	{
		GUNS[f].stopDrag();
	}
}
Мама дорогая...
1. GUNS - так называют только константы. Переменная должна называться guns
2. Зачем вы засунули определение функций в цикл? С какой целью? Думаете они будут каждый раз создаваться? Нет, функция определяется один раз. Лучше всего в самом верху кода.
3. Внутри функций не надо писать GUNS[f], это все равно работать не будет. Знаете почему? Потому что когда сработает функция, от любого объекта, то переменная f будет равна своему последнему значению в цикле, то есть GUNS.length - 1.
4. Внутри функции нужно использовать e.target. И да, называть аргумент e - плохая практика. Нужно давать осмысленное название - event

Что имеем на выходе?
Код AS3:
function mDown (event:MouseEvent):void {
	event.target.startDrag (false);
}
function mUp (event:MouseEvent):void {
	event.target.stopDrag ();
}
var guns:Array = new Array
guns.push (weapons.gun.guni);
for (var i:int = 0; i < guns.length; i++) {
	guns[i].addEventListener(MouseEvent.MOUSE_DOWN, mDown);
    guns[i].addEventListener(MouseEvent.MOUSE_UP, mUp);
}
__________________
(и)Нильс.ru | Плагины для FlashDevelop