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

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

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

Регистрация: Feb 2009
Сообщений: 141
По умолчанию После removeEventListener обработчик продолжает вызываться

Собственно заголовком все сказано, исходник ниже. Из вне создается несколько таких объектов. Я не могу догнать вобще как такое случилось, что после removeEventListener у меня продолжается вызываться обработчик.

обратите внимание на блок с trace("before", death.id) по trace("after", death.id);

Код AS3:
package ***
{
	***
 
	import flash.display.Bitmap;
	import flash.events.Event;
	import flash.text.TextField;
	import flash.text.TextFieldAutoSize;
	import flash.text.TextFormat;
 
	public class DeathImage extends SpriteControl
	{
		private static const CLOSE_DEATH_BACKGROUND_IMAGE:String = "images/ui/death/closeDeath.png";
		private static const BRONZE_DEATH_BORDER_IMAGE:String = "images/ui/death/bronzeDeathBorder.png";
		private static const SILVER_DEATH_BORDER_IMAGE:String = "images/ui/death/silverDeathBorder.png";
		private static const GOLD_DEATH_BORDER_IMAGE:String = "images/ui/death/goldDeathBorder.png";
		private static const BRONZE_DEATH_NUMBER_BACKGROUND_IMAGE:String = "images/ui/death/bronzeDeathNumberBackground.png";
		private static const SILVER_DEATH_NUMBER_BACKGROUND_IMAGE:String = "images/ui/death/silverDeathNumberBackground.png";
		private static const GOLD_DEATH_NUMBER_BACKGROUND_IMAGE:String = "images/ui/death/goldDeathNumberBackground.png";
 
		private var __width:int = 157;
		private var __height:int = 150;
 
		private var __death:Death;
 
		private var __isLoaded:Boolean = false;
 
		public function DeathImage(controls:Controls)
		{
			super(controls);
 
			this.addEventListener(Event.ADDED_TO_STAGE, __addedToStage);
		}
 
		override public function set width(value:Number):void
		{
			__width = value;
			super.width = width;
		}
 
		override public function get width():Number
		{
			return __width;
		}
 
		override public function set height(value:Number):void
		{
			__height = value;
			super.height = height;
		}
 
		override public function get height():Number
		{
			return __height;
		}
 
		public function set death(value:Death):void
		{
			__death = value;
		}
 
		public function get death():Death
		{
			return __death;
		}
 
		private function __addedToStage(e:Event):void
		{
			this.removeEventListener(Event.ADDED_TO_STAGE, __addedToStage);
 
			if (!__death)
				return;
 
			var images:Array = [];
			switch (death.conquestLevel)
			{
				case 0:
					images.push(CLOSE_DEATH_BACKGROUND_IMAGE);
					break;
				case 1:
					images.push(BRONZE_DEATH_BORDER_IMAGE, BRONZE_DEATH_NUMBER_BACKGROUND_IMAGE);
					break;
				case 2:
					images.push(SILVER_DEATH_BORDER_IMAGE, SILVER_DEATH_NUMBER_BACKGROUND_IMAGE);
					break;
				case 3:
					images.push(GOLD_DEATH_BORDER_IMAGE, GOLD_DEATH_NUMBER_BACKGROUND_IMAGE);
					break;
			}
 
			if (death.conquestLevel > 0)
				images.push(death.url);
			trace("before", death.id)
			controls.images.addEventListener(ImagesEvent.IMAGES_LOAD_COMPLETE, __imagesLoadComplete);
			controls.images.loadImages(images);
		}
 
		private function __imagesLoadComplete(e:ImagesEvent):void
		{
			/*if (__isLoaded)
				return;*/
 
			__isLoaded = true;
			controls.images.removeEventListener(ImagesEvent.IMAGES_LOAD_COMPLETE, __imagesLoadComplete);
			trace("after", death.id);
			var deathImage:Bitmap;
			var borderImage:Bitmap;
			var titleImage:Bitmap;
			switch (death.conquestLevel)
			{
				case 0:
					var closeDeath:Bitmap = controls.images.getLoadedImage(CLOSE_DEATH_BACKGROUND_IMAGE);
					this.addChild(closeDeath);
					break;
				case 1:
					borderImage = controls.images.getLoadedImage(BRONZE_DEATH_BORDER_IMAGE);
					titleImage = controls.images.getLoadedImage(BRONZE_DEATH_NUMBER_BACKGROUND_IMAGE);
					break;
				case 2:
					borderImage = controls.images.getLoadedImage(SILVER_DEATH_BORDER_IMAGE);
					titleImage = controls.images.getLoadedImage(SILVER_DEATH_NUMBER_BACKGROUND_IMAGE);
					break;
				case 3:
					borderImage = controls.images.getLoadedImage(GOLD_DEATH_BORDER_IMAGE);
					titleImage = controls.images.getLoadedImage(GOLD_DEATH_NUMBER_BACKGROUND_IMAGE);
					break;
			}
 
			if (death.conquestLevel > 0)
			{
				deathImage = controls.images.getLoadedImage(death.url);
				deathImage.smoothing = true;
 
				var imageSize:int = width - 30;
				var koef:Number = deathImage.width / imageSize;
				deathImage.scaleX /= koef;
				deathImage.scaleY /= koef;
				deathImage.x = width / 2 - deathImage.width / 2;
				deathImage.y = height / 2 - deathImage.height / 2;
				this.addChild(deathImage);
 
				if (borderImage)
					this.addChild(borderImage);
 
				if (titleImage)
				{
					titleImage.x = width / 2 - titleImage.width / 2;
					titleImage.y = 10;
					this.addChild(titleImage);
 
					var textField:TextField = TextFormats.defaultTextField;
					textField.defaultTextFormat = new TextFormat(TextFormats.EMBED_FONT, 12);
					textField.text = death.id.toString();
					textField.autoSize = TextFieldAutoSize.LEFT;
					textField.x = width / 2 - textField.width / 2 - 1;
					textField.y = 11;
					this.addChild(textField);
				}
			}
		}
	}
}
Результаты трейсов:
before 10
before 13
before 99
before 2
before 51
before 127
before 126
before 128
after 10
after 13
after 99
after 2
after 51
after 127
after 126
after 128
after 128
after 126
after 128
after 127
after 126
after 128
after 51
after 127
after 126
after 128
after 2
after 51
after 127
after 126
after 128
after 99
after 2
after 51
after 127
after 126
after 128
after 13
after 99
after 2
after 51
after 127
after 126
after 128

Дабы хоть как-то решить вопрос пришлось сделать проверку на то, произошла ли уже загрузка в объекте или нет. Где я накосячил никак не могу понять

Старый 05.03.2013, 03:57
AlexLucas вне форума Посмотреть профиль Отправить личное сообщение для AlexLucas Найти все сообщения от AlexLucas
  № 2  
Ответить с цитированием
AlexLucas
[+1 05.11.12]
 
Аватар для AlexLucas

Регистрация: Feb 2011
Сообщений: 431
Видимо проблема в том что сам объект типа DeathImage несколько раз добавляется на сцену, или добавляется несколько объектов типа DeathImage подряд, и при каждом
Код AS3:
controls.images.loadImages(images);
несколько раз диспатчится ваш ImagesEvent, судя по тому что трэйсов 'after' больше чем 'before'.
Короче по ходу не в этом классе проблема, а в том что добавляет этот самый DeathImages на сцену.

Старый 05.03.2013, 04:11
Vreden вне форума Посмотреть профиль Отправить личное сообщение для Vreden Найти все сообщения от Vreden
  № 3  
Ответить с цитированием
Vreden
 
Аватар для Vreden

Регистрация: Feb 2009
Сообщений: 141
Да, таких объектов добавляется подряд несколько, и каждый объект подписывается на данное событие и при получении события отписывается.

В итоге что получается, после того как первый объект от события отписался, второй следом подписался, предыдущий тоже оказывается подписанным?

Я что-то запутался, видимо пора спать.

Старый 05.03.2013, 13:23
AlexLucas вне форума Посмотреть профиль Отправить личное сообщение для AlexLucas Найти все сообщения от AlexLucas
  № 4  
Ответить с цитированием
AlexLucas
[+1 05.11.12]
 
Аватар для AlexLucas

Регистрация: Feb 2011
Сообщений: 431
Для каждого объекта в отдельности, при его добавлении на сцену будет срабатывать его же __addedToStage.

Старый 05.03.2013, 13:56
Vreden вне форума Посмотреть профиль Отправить личное сообщение для Vreden Найти все сообщения от Vreden
  № 5  
Ответить с цитированием
Vreden
 
Аватар для Vreden

Регистрация: Feb 2009
Сообщений: 141
Ну, да, а при чем здесь __addedToStage?

Старый 05.03.2013, 13:59
caseyryan вне форума Посмотреть профиль Отправить личное сообщение для caseyryan Найти все сообщения от caseyryan
  № 6  
Ответить с цитированием
caseyryan
 
Аватар для caseyryan

Регистрация: Jun 2012
Адрес: Новосибирск
Сообщений: 6,644
Записей в блоге: 4
Сделайте вот так при добавлении слушателя:
Код AS3:
controls.images.removeEventListener(ImagesEvent.IMAGES_LOAD_COMPLETE, __imagesLoadComplete);
controls.images.addEventListener(ImagesEvent.IMAGES_LOAD_COMPLETE, __imagesLoadComplete);
или так:
Код AS3:
if (!controls.images.hasEventListener(ImagesEvent.IMAGES_LOAD_COMPLETE)) {
   controls.images.addEventListener(ImagesEvent.IMAGES_LOAD_COMPLETE, __imagesLoadComplete);
}
чтобы точно больше одного слушателя не добавлялось, и проверьте

Старый 05.03.2013, 19:36
Vreden вне форума Посмотреть профиль Отправить личное сообщение для Vreden Найти все сообщения от Vreden
  № 7  
Ответить с цитированием
Vreden
 
Аватар для Vreden

Регистрация: Feb 2009
Сообщений: 141
Добавление нескольких прослушивателей с одиниковым типом, обработчиком и целевой фазой не приводят к дополнительной регистрации прослушивателя, поэтому это не имеет смысла вовсе.

Старый 05.03.2013, 20:33
AlexLucas вне форума Посмотреть профиль Отправить личное сообщение для AlexLucas Найти все сообщения от AlexLucas
  № 8  
Ответить с цитированием
AlexLucas
[+1 05.11.12]
 
Аватар для AlexLucas

Регистрация: Feb 2011
Сообщений: 431
Либо в __imagesLoadComplete ссылка на controls.images это уже ссылка на другой объект и вы не того отписываете, либо вы его опять где-то подписываете на это событие.
В любом случае из этого куска кода не понять.

Старый 05.03.2013, 20:44
Vreden вне форума Посмотреть профиль Отправить личное сообщение для Vreden Найти все сообщения от Vreden
  № 9  
Ответить с цитированием
Vreden
 
Аватар для Vreden

Регистрация: Feb 2009
Сообщений: 141
Цитата:
Сообщение от AlexLucas Посмотреть сообщение
либо вы его опять где-то подписываете на это событие.
Да, я опять подписываюсь на событие, но подписывается другой экземпляр класса DeathImage. Этот-то по идее должен быть отписанным.

Вобщем пришлось решать проблему тем, чтобы каждый экземпляр класса юзал отдельный controls.images, а не один и тот же.
Ибо получается, что подпись на событие экземпляра controls.images подписывает также все другие экземпляры DeathImage, которые уже отписались от него. Честно говоря для меня это поведение неожиданность. Я считал что подпись осуществляется в пределах экземпляра класса, к которому принадлежит обработчик. Неужели это не так?


Последний раз редактировалось Vreden; 05.03.2013 в 21:02.
Старый 05.03.2013, 22:06
caseyryan вне форума Посмотреть профиль Отправить личное сообщение для caseyryan Найти все сообщения от caseyryan
  № 10  
Ответить с цитированием
caseyryan
 
Аватар для caseyryan

Регистрация: Jun 2012
Адрес: Новосибирск
Сообщений: 6,644
Записей в блоге: 4
Цитата:
Неужели это не так?
Нет, не так. По сути этому экземпляру вообще пофиг в каком классе его подписали. Если экземпляр класса, в котором находится обработчик, будет собран сборщиком мусора, то при прохожении события вылетит ошибка обращения к объекту null

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

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

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


 


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


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