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

[dashaaa]

Добрый день,
извините за назойливость, но мне необходимо понять - почему плохо создавать функции внутри функций?
И как правильно организовать следующий код?

public function loadImg():Void {
	var mclLoader:MovieClipLoader = new MovieClipLoader();
	var oLoadListener:Object = new Object();
	mclLoader.addListener(oLoadListener);

	function onLoadError(mcTarget:MovieClip, sErrorCode:String):Void {}
}

Поиск довольно внятно ответил мне, что это "*****код", но хочется обоснования...

[iNils]

class имя класса {
	var mclLoader:MovieClipLoader = new MovieClipLoader ();

	public function конструктор класса () {
		mclLoader.addListener(this);
	}

	public function loadImg():Void {

	}

	private function onLoadError (mcTarget:MovieClip, sErrorCode:String):Void {
	}
}

[etc]

dashaaa, ну во-первых, приведенный вами код не будет работать, просто потому что onLoadError не принадлежит объекту oLoadListener. А вложенные функции плохи тем, что во первых, плывет область видимости и начинаются проблемы с обращением к различным свойствам и объектам, во вторых, вместо того, что создать метод один раз в памяти, он постоянно пересоздается новый.

[Wolsh]

Дело в том, что функция просто выполняется при вызове, и выполнившись, возвращает результат или меняет чтото вне себя. После того, как она выполнилась, все ее внутренние переменные больше недоступны, точнее их уже нет. Все эти переменные - mclLoader, oLoadListener, и функция onLoadError() существуют лишь до тех пор, пока выполняется loadImg(), потому что это ее внутренние переменные. Они как пассажиры пролетающего вагона - могут разговаривать друг с другом, но Вы, на полустаночке), ни словом с ними не перекинетесь. Пролетая мимо, они Вам могут крикнуть "загрузи картинку и отдай Васе на пятом сиденье", но пока Вы сбегаете за картинкой - ни Васи, ни сиденья.
(из моего поста могло создаться впечатление, что дело во времени, но на самом деле конечно в областях видимости - т.е. нормальная организация подразумевает, что Вася стоит с Вами на полустаночке. Т.е. функция не должна быть внутри другой функции)

[dashaaa]

class имя класса {
	var mclLoader:MovieClipLoader = new MovieClipLoader ();

	public function конструктор класса () {
		mclLoader.addListener(this);
	}

	public function loadImg():Void {

	}

	private function onLoadError (mcTarget:MovieClip, sErrorCode:String):Void {
	}
}

Compiler Error, line 2 - A class's instance variables may only be initialized to compile-time constant expressions.

Но вообще-то идея ясна, спасибо.

__etc, спасибо, с областью видимости уже столкнулась, и то, что метод пересоздаётся - это аргумент.
Wolsh, спасибо, понятно =)

[Mnilionic]

создавайте экземпляр MovieClipLoader в конструкторе класса

[chingachgoog]

Функция внутри функции - это нормальное явление, просто надо знать для чего это нужно.
То, что "поезд уходит" тоже не совсем верно:

function A(){
	var x="123"
	B()
	function B(){
		link=this
		this.x
	}
}
A()

trace(link.x)

А вообще посмотрите тему Битва гигантов..., там как раз ближе к концу объекты активации исследуются.

[Mnilionic]

хы-хы
в делигаторе функция внутри функции, но там правда ссылка на неё живая

[Division]

Функция внутри функции - это нормальное явление, просто надо знать для чего это нужно.

А для чего нужно, если не секрет? Мне вообще кажется что это довольно удобно в некоторых случаях. Например, если большая функци выполняет много однотипных действий, их можно вынести во вложенную функцию. ИМХО.

[iNils]

А для чего нужно, если не секрет?

Только для реализации Delegate. Других оправданных примеров использования я не вижу.

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

Выше было сказано, что это приводит к неоправданному расходу памяти. Делайте приватный метод и выносите в него.