Функции внутри функций - почему это зло?

[dashaaa]

Подскажите, как в такой ситуации правильно организовать функции, чтобы не было проблем с вложенными функциями и с областями видимости:

class Room {
	static var xmlName:String = "1.xml";
	var xmlDoc:XML;
	var xml_rootNode:XMLNode;
 
	public function parsXML():Void {
		trace(xml_rootNode);
	}
	public function Init():Void {
		xmlDoc.load(xmlName);
           // ошибка - обработчик внутри функции.
           // хотя может и нет - обработчик нужен 1 раз и удаляется после использования
           // ?
		xmlDoc.onLoad = function():Void  {
			if (this.status == 0) {
				trace("Успешно");
				xml_rootNode = this.firstChild;
                                // !!!!! ошибка - эта функция "не видна" отсюда
				parsXML();
			} else {
				trace("Ошибка при чтении xml "+this.status);
			}
		};
	}
	public function Room() {
		xmlDoc = new XML();
	}
}

[iNils]

import mx.utils.Delegate;
class Room {
	static var xmlName:String = "1.xml";
	var xmlDoc:XML;
	var xml_rootNode:XMLNode;
 
	public function parsXML():Void {
		trace(xml_rootNode);
	}
	public function Init():Void {
		xmlDoc.load(xmlName);
           // ошибка - обработчик внутри функции.
           // хотя может и нет - обработчик нужен 1 раз и удаляется после использования
           // ?
	}
	public function Room() {
		xmlDoc = new XML();
		xmlDoc.onLoad = Delegate.create (this, onLoadXML);
	}
	
	private function onLoadXML (success:Boolean):Void
		//// this теперь ссылается на экземпляр класса Room
		if (success) {
			trace("Успешно");
			xml_rootNode = xmlDoc.firstChild;
							// !!!!! ошибка - эта функция "не видна" отсюда
			parsXML();
		} else {
			trace("Ошибка при чтении xml "+xmlDoc.status);
		}
	};

}

[dashaaa]

Спасибо большое! Я до этого понятия не имела о Delegate.

[faust.d503]

А зачем использовать Delegate, по-моему есть более простой и удобный способ:

class Room {
 
	public function Room() {
		var self:Room = this
		xmlDoc = new XML();
		xmlDoc.onLoad = function(){
			self.onLoadXML();
		}
	}
 
	private function onLoadXML (success:Boolean):Void
 
	};
 
}

[etc]

faust.d503, это кривой и неудобный способ.

[faust.d503]

__etc
Расскажите, чем он кривой и неудобный и чем лучше Delegate. Нет конечно можно подключить дополнительный класс, чтобы повысить вес программы, но зачем.

Кстати написать функцию прямо внутри конструктора, тоже вполне логичный способ, если парсер используется один раз. Ибо размножать количество методов и переменных класса пусть, и приватных, не всегда хорошо.

[etc]

Такой обработчик нельзя удалить, он будет висеть в памяти до упора, плюс требует self-ссылки, которая тоже живет в памяти, потому что живет функция.

[faust.d503]

Такой обработчик нельзя удалить

Почему?

class Room {
 
	public function Room() {
		var self:Room = this;
		xmlDoc = new XML();
		xmlDoc.onLoad = function(){
			self.onLoadXML();
			//удаляем)
			delete this.onLoad;
		}
	}
 
	private function onLoadXML (success:Boolean):Void
 
	};
 
}

[etc]

Да, но тогда анонимная функция с большой вероятностью останется жить, потому что есть self-ссылка, используемая внутри.

[iNils]

О, уже костыли пошли, а было "более простой и удобный способ".