"Залипают" клавиши управления у главного героя

[ndiezel]

Всем привет.

Какой-то странный баг и даже и не знаю как его убрать. Управление главным героем сделано с помощью слушателей событий:

		public function gameKeysDown(e:KeyboardEvent):void
		{
			if (! gameMenuPaused)
			{
				if (e.keyCode == 65 || e.keyCode == 37)
				{
					leftBln = true;
				}
				if (e.keyCode == 68 || e.keyCode == 39)
				{
					rightBln = true;
				}
				if (e.keyCode == 87 || e.keyCode == 38)
				{
					upBln = true;
				}
				if (e.keyCode == 83 || e.keyCode == 40)
				{
					downBln = true;
				}
			}
		}
 
		public function gameKeysUp(e:KeyboardEvent):void
		{
			if(!gameMenuPaused)
			{
				if (e.keyCode == 65 || e.keyCode == 37)
				{
					leftBln = false;
				}
				if (e.keyCode == 68 || e.keyCode == 39)
				{
					rightBln = false;
				}
				if (e.keyCode == 87 || e.keyCode == 38)
				{
					upBln = false;
				}
				if (e.keyCode == 83 || e.keyCode == 40)
				{
					downBln = false;
				}
			}
		}

Ну и соотв, если true - двигаем, если false - не двигаем.
Но постоянно, при возрастании нагрузки на флешку, особенно на считыватели событий, кнопки начинают "залипать". Например управляю я героем и при этом постоянно клацаю кнопкой мыши - всё. Слушатель событий не успевает считать что кнопка была поднята и в результате герой начинает сам двигаться в какую-то сторону, что сильно портит игровой процесс.

Может кто сталкивался? Почему такое происходит?
И эту проблему можно решить с помощью dispatchEvent? (или idispatchEvent, я не разбирался ещё) Если я правильно понял - есть функции управления событиями. Хочу чтоб флеш больше ресурсов уделял на слушатели событий управления.

Или как можно эту проблему решить?

[Psycho Tiger]

Попробуй повесить слушатель события на stage.

А вообще (ничего, что я на ты?) – сложно ты всё придумал.
Заведи хэш

private var _pressedKeys:Object = {}

Слушатели сделай элегантными

private function gameKeysUp(e:KeyboardEvent):void{
  _pressedKeys[e.keyCode] = false;
}
private function gameKeysDown(e:KeyboardEvent):void{
  _pressedKeys[e.keyCode] = true;
}

Заведи метод для определения, нажата ли клавиша. Раз у тебя клавиши дублируются, не стесняйся создать "таблицу алиасов".

private static const ALIASES:Object = { }
/* я неуверен, можно ли сразу создать значения из чисел прямо в определении, поэтому определю ниже */
{
ALIASES[65] = 37;
ALIASES[68] = 39;
ALIASES[87] = 38;
ALIASES[83] = 40;
};
private function isKeyDown(keyCode:int):Boolean{
  //Все понятно, если кнопка зажата
  if (_pressedKeys[keyCode]) return true;
  //проверяем, есть ли такая кнопка в таблице алиасов. Если есть, то мы должны проверить ещё и её алиас  
  if (keyCode in ALIASES) return(_pressedKeys(ALIASES[keyCode]));
  return false; //здесь если совсем ничего не зажато
}

Теперь тебе стоит работать только с клавишами 65, 68, 87 и 83. Запросив isKeyDown(65) метод вернёт истину, если нажата клавиша с кодом 65 или с кодом 37.

QA-секция:
Q: но ты совсем забыл про !gameMenuPaused условие! Я вставлю его сам.
A: конечно, отсюда не слишком понятна цель этого условия, но мне кажется, это чтобы персонаж не воспринимал кнопки во время паузы. Это неправильно представь ситуацию, что игрок зажал "вправо" и включил меню паузы, а потом отжал вправо. Убрав паузу игрок продолжит бежать вправо, потому что флаг правой клавиши не снялся как раз из за этого условия.

P.S. фигачил код прямо с форума, за работоспособность не ручаюсь.

[ndiezel]

Psycho Tiger, сначала я не очень понял твой код. Подумал - о! Это, наверное, решение!
Но когда разобрался... Это, по сути, тот же метод, что использую я. Нужны будут те же слушатели событий KeyboardEvent.KEY_DOWN и KeyboardEvent.KEY_UP и по EnterFrame - проверка зажата ли кнопка или нет. Только будут не переменные, а массив с переменными. Да, это чуть изящней, но по сути - ничего не меняет.

Вот привязывание слушателя к stage - вроде как помогло.
Ещё проверяю, но вроде - не залипает больше.

[Psycho Tiger]

Вот привязывание слушателя к stage - вроде как помогло.
Ещё проверяю, но вроде - не залипает больше.

Есть такая штука - stage.focus. Указывает на то, что сейчас "в фокусе". Например, если это текстовое поле – то символы будут туда вводиться, а если не текстовое поле – соответственно, не будут.
Идея в том, что кликая на какой-нибудь другой элемент прочие перестают иметь фокус (ну, грубо говоря). А так как по факту любой клик всегда будет кликом на stage (конкретней - на его ребёнка, не суть) – то фокус не потеряется. Тадаам!

[ndiezel]

Сообщение от Psycho Tiger

Есть такая штука - stage.focus. Указывает на то, что сейчас "в фокусе". Например, если это текстовое поле – то символы будут туда вводиться, а если не текстовое поле – соответственно, не будут.
Идея в том, что кликая на какой-нибудь другой элемент прочие перестают иметь фокус (ну, грубо говоря). А так как по факту любой клик всегда будет кликом на stage (конкретней - на его ребёнка, не суть) – то фокус не потеряется. Тадаам!

Всё бы ничего, да только, как оказалось, если повесить слушатели на stage, то они потом не убираются. Даже если сделать вот так:

			stage.focus = this;
			stage.removeEventListener(KeyboardEvent.KEY_DOWN,workScreen[0].gameKeysDown);
			stage.removeEventListener(KeyboardEvent.KEY_UP,workScreen[0].gameKeysUp);

И остаётся в памяти слушатели, которые потом повторяются и, в общем - жуть какая-то получается...

[caseyryan]

Всё бы ничего, да только, как оказалось, если повесить слушатели на stage, то они потом не убираются.

Не правда.
Тут сам подход кривой. Вот так workScreen[0].gameKeysDown делать не нужно. 99% что именно в этом месте и проблема

[ndiezel]

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

Не правда.
Тут сам подход кривой. Вот так workScreen[0].gameKeysDown делать не нужно. 99% что именно в этом месте и проблема

Вот так у меня сделано:

workScreen[0].addEventListener(KeyboardEvent.KEY_DOWN,workScreen[0].gameKeysDown,false,0,true);
workScreen[0].addEventListener(KeyboardEvent.KEY_UP,workScreen[0].gameKeysUp,false,0,true);

Чем криво?

[Psycho Tiger]

И остаётся в памяти слушатели, которые потом повторяются

Один слушатель на один тип события на один метод. Нельзя повесить его дважды.

Это значит что workScreen[0] указывает на разные места в разные моменты времени.
Другими словами, workScreen[0] во время подписки !== workScreen[0] во время отписки.

[ndiezel]

Я не совсем понял, что ты написал Psycho Tiger

Я постараюсь просто описать что происходит, чтоб прояснить ситуацию.
Если присваивать и снимать слушатели с workScreen[0] (там находится мувик с игрой), то всё работает. Ссылка в массиве - на один и тот же мувик. Всё прекрасно вешается и снимается.
Если же я вешаю и снимаю со stage, то со stage слушатели не снимаются. Во время игры stage меняется, там приходится stage обновлять, чтоб слушатели работали.

Так вот, если вешать слушатели на stage, после завершения игры мувик с игрой gc не удаляется, потому что на нём висят эти два слушателя (на помогает даже weak reference). А потом, когда создаётся новый мувик с игрой, его слушают уже два слушателя. И игра начинает жутко глючить.

И если бы придумать как найти эти слушатели и всё таки их снять со stage, то это очень бы помогло, потому что при вызове слушателей на stage кнопки не залипают, или залипают намного меньше, чем при присвоении workScreen[0].

[MikroAcse]

Ну так храни стейдж, на который повесил слушатель.