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

[chingachgoog]

А по-моему, если что и называть пренебрежительно "костылем", то как раз сам Delegate.

[etc]

И то и другое костыль, но self-ссылки — костыль в несколько раз больше, и не один.

[iNils]

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

И то и другое костыль, но self-ссылки — костыль в несколько раз больше, и не один.

Абсолютно согласен. Преимущество делагата в том, что мы не меняем область видимости this в любом месте класса.

[faust.d503]

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

self объявлено внутри конструктора, анонимная функция существует там же. Как они остануться жить после выполнения кода констуктора мне с трудом понятно.
На self ссылка после выполнения конструктора убивается, остается ссылка внутри безымянной функции, на которую ссылка отсутствует, по идее должна убиться.
Как и что убивается при использования Delegate мне неизвестно, если известно вам, поясните пожалуйста.
А вообще рассуждения "с большей вероятностью" чистая демагогия.

[iNils]

Сообщение от faust.d503

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

Все поубивались после конструктора. И куда же тогда придет сообщении о загрузке?

[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. Снова область видимости будет неправильной.

[chingachgoog]

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

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

Наоборот, правильной. Вместо вымученной делегатом.
Сразу видно, что обработчик onLoad принадлежит объекту xmlDoc и соответственно this внутри обработчика принадлежит xmlDoc.

Код без делегатов более понятный и логичный. Не зря же делегаты - пользовательский класс, а не встроенный.