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

[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();
	}
}

[dashaaa]

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

[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);
		}
	};

}

[chingachgoog]

А что вообще делегат делает? Возвращает функцию оболочку с нужной функцией вызова apply:

class Room {
	public function Room() {
		xmlDoc = new XML();
		xmlDoc.onLoad = function (){                    
			return onLoadXML.apply(this, arguments);
		}
	}
	private function onLoadXML (success){}
}

По-любому генерится лишняя функция.

[etc]

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

			return onLoadXML.apply(this, arguments);

Ага, только в этом коде область видимости не изменится.

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

По-любому генерится лишняя функция.

Так она генерится в обоих костылях.

[chingachgoog]

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

Ага, только в этом коде область видимости не изменится.

Да, это просто схема работы делегата. (код нерабочий, если кто не понял. Чтобы сделать его рабочим надо добавить ссылку на функцию onLoadXML. Например, так: xmlDoc.onLoadXML=onLoadXML)
Но суть в создании лишней функции-оболочки.

[etc]

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

Чтобы сделать его рабочим надо добавить ссылку на функцию onLoadXML. Например, так: xmlDoc.onLoadXML=onLoadXML)

Опять же, this внутри onLoadXML будет ссылаться на xmlDoc, а не на экземпляр Room. Снова область видимости будет неправильной.

[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. Нет конечно можно подключить дополнительный класс, чтобы повысить вес программы, но зачем.

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