Xорошо ли передавать в конструктор класса в качестве параметра this

[Nemez]

Возник спор, хорошо ли передавать в конструктор класса в качестве параметра this . Как альтернатива - передача нескольких пакаметров. Или это все равно как, или обусловленно какими-то причинами?

[dimarik]

Вполне нормальная практика. Это же просто ссылка на сам объект, в котором происходит инстанцирование другого объекта. Если конструктор этого "другого" объекта принимает ссылки на объекты, имеющие тип как у this, то почему бы не передать this параметром.

[Nemez]

Что делает коллега, упрощенная схема:

viewport = new Viewport(this);
public function setModus() {
	.............		
}

public function Viewport(main:GlavniiClass):void {			
	this.main = main;
	main.setModus();	
}

По-моему, это нехорошо.

[CROL]

непонятно что нехорошего то? this это такая же ссылка на объект как и все другие

[ChuwY]

GlavniiClass

=D
Вот это точно нехорошо.

[Котяра]

Ну и

main.setModus();

тоже не очень. Сильная связность получается. Хотя я не скажу что так вообще никогда нельзя делать. Могут быть исключения - просто на этот шаг надо идти осознано.
А передавать ли this в конструктор, даже не имеет смысла обсуждать - можно.

[Nemez]

Сообщение от Котяра

Ну и

main.setModus();

тоже не очень. Сильная связность получается. Хотя я не скажу что так вообще никогда нельзя делать. Могут быть исключения - просто на этот шаг надо идти осознано.

Из-за этого-то и возник спор.

[dimarik]

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

Что делает коллега, упрощенная схема:

public function Viewport(main:GlavniiClass):void {			
	this.main = main;
	main.setModus();	
}

По-моему, это нехорошо.

Это очень нехорошо!! Постарайтесь указывать аргументы (параметры передаем, аргументы принимаем!) в конструкторе как интерфейсы или как нативный тип классов флеш-плеера. Иначе образуется зависимость класса Viewport от указанного типа (GlavniiClass) и никакой другой тип, не считая унаследованных от GlavniiClass, Вы туда не передадите. А мы же стремимся сделать класс многоразовым (code reuse)? Если да, то прислушайтесь.

Добавлено через 1 минуту
Ну, если осознанно, то не вопрос.

[mikhailk]

Что делает коллега, упрощенная схема

Передача ссылки на объект как параметр - совершенно нормально.
Не суть, конструктор или какой другой метод.

А вот вызов методов друг друга таким образом как в примере - это, конечно, жуть. Честно говоря, я не могу себе представить ситуации, когда один объект создает другой и разрешает ему себя напрямую дергать...

[goodguy]

Честно говоря, я не могу себе представить ситуации, когда один объект создает другой и разрешает ему себя напрямую дергать...

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