Как правильно инициализировать объект в объекте?

Question

[Max Ba]

Ребят, подскажите. Нужно в первом классе, присвоить свойству $this->a массив из внешнего класса (Test2).

class Test{
  protected $a = array();

  public function __construct(){
    $tmp = new Test2();
    $this->a = $tmp->get();
  }
}

class Test2{
  private $data = array();

  public function __construct(){
    $this->data = range(1,100);
  }

  public function get(){
    return $this->data;
  }
}

Правильно я сделал в данном примере?
Сейчас нужно именно вот так вызвать один класс, внутри другого.

Comments

[antoo]

А что именно вас смущает? С кодом проблем не вижу.

[Max Ba]

antoo, ну это уже хорошо)
А нужно ли избавляться (обойтись без) $tmp ?

[antoo]

Макс Васильев, если $tmp не выходит за пределы scope, то счётчик ссылок он не увеличит и объект будет позже удалён garbage collector'ом, поэтому в данном случае можно не избавляться.

[Max Ba]

Спасибо

Answer 1

[Антон Шаманов]

почитай о Dependency Injection

Comments

[Max Ba]

Да, я обычно так и делаю. Просто в данном случае нужно было вызвать именно объект внутри другого и наполнить массив в первом.

[Алексей]

Макс Васильев, в этом случае у вас что-то не так в архитектуре приложения, если вы ни как не можете позволить передать уже созданный и инициализированный объект в конструктор. И если в вашем вопросе ставится акцент на - как правильно делать, то именно то о чем сказал Антон Шаманов и является right way. А так если не принимать это во внимание, то да, почему бы не создать объект прямо в классе. Чем это плохо? Высокая связанность и зависимость кода, невозможность его повторного использования.

[Max Ba]

Я лишь экспериментировал. Решил вынести часть кода (один из методов) в совсем отдельный класс. И результат работы этого вынесенного класса, нужно вернуть туда, откуда его переместил. В то же самое свойство.
Грубо говоря я лишь разделил один класс на два, чтобы редактировать его в будущем было удобней.
А так, изначально код был такой:

class Test{
  protected $a = array();

  public function __construct(){
    $this->set();
  }
  
  private function set(){
    $this->a = range(1,100);
  }
}

[Max Ba]

Не private function set, а init
:)

[Алексей]

Макс Васильев,

class Test
{
  /** @var array */
  protected $a;

  public function __construct(){
    $this->a = range(1,100);
  }
}

а зачем усложнять? Ты set(), еще как нибудь используешь внутри класса, кроме как в конструкторе?

[Max Ba]

Алексей, да это понятно)))
вопрос был больше теоретический.
И естественно код упрощен до уровня "примеров".
Но смысл вопроса так и остался, про разделение классов.

[Max Ba]

Есть у меня один класс, там около 20 Кб кода. Я решил, что править эту махину (делая мелкие правки в некоторых методах) удобней было бы в разделенных файлах.

[Алексей]

Макс Васильев, тогда все же, если хочешь писать правильный, масштабируемый и легко поддерживаемый код смотри в сторону Dependency Injection, о чем уже было выше сказано. Перестрой приложение так, чтобы у тебя конструктор принимал даже не объект как таковой, а некий интерфейс. И передавай в конструктор объект, имплементирующий его. Хотя бы вскользь прочитай про композицию кода.

[Max Ba]

Алексей, типо этого?

class Test {
  public function __construct(Test1 $tmp1, Test2 $tmp2, Test3 $tmp3...){
    $this->test1 = $tmp1;
      $this->test2 = $tmp2;
      $this->test3 = $tmp3;
    }
}

[Max Ba]

Алексей, кстати, я раньше делал с помощью свойства public.

$test1 = new Test1();

$obj = new Test();
$obj->test1 = $test1->init();

[Алексей]

Макс Васильев, да вроде этого

__construct(TestInterface1 $tmp1, TestInterface2 $tmp2, TestInterface3 $tmp3...)

и то нужно смотреть, 100500+ инстансов передавать в конструктор тоже не комильфо, начни с этого Структурные шаблоны проектирования

если через сеттеры например так:

$obj = new Test();
$obj->set(new Test1);

[Max Ba]

Алексей, всё понял. Спасибо!

Answer 2

[Данила Риджи]

https://ideone.com/S4VJFU
Должно работать в 5.4+