Laravel. Не понимаю логику кода?

Question

[Agent Gus]

Обучаюсь по урокам "Фреймворк Laravel с Нуля до Профи". В принципе, все понятно, кроме одного кода...

IndexController

class IndexController extends SiteController {
    public function __construct() {
        $menu = new \Corp\Repositories\MenusRepository(new \Corp\Menu);
        parent::__construct($menu);
        $this->bar = 'right';
        $this->template = env('THEME').'.index';
    }
}

MenusRepository

<?php
namespace Corp\Repositories;

use Corp\Menu;

class MenusRepository extends Repository {
	public function __construct(Menu $menu) {
		$this->model = $menu;
	}
}

?>

Так вот...
Я не понимаю функционал этого кода.

parent::__construct(new \Corp\Repositories\MenusRepository(new \Corp\Menu));

Я знаю, что это Вызов конструктора родительского класса.
Вопрос по Laravel.

$menu = new \Corp\Repositories\MenusRepository(new \Corp\Menu); Для чего?
В итоге получается:

public function __construct(Menu new \Corp\Repositories\MenusRepository(new \Corp\Menu)) 
{
	$this->model = $menu;
}

Comments

[Agent Gus]

parent::__construct(new \Corp\Repositories\MenusRepository(new \Corp\Menu));

class MenusRepository extends Repository {
  public function __construct(Menu $menu) {
    $this->model = $menu;
  }

///

Answer 1

[Роман Игошин]

Какому бреду вас учат. Лучше читайте доку и смотрите ларакасты(https://laracasts.com). Никто в ларе так не делает.
Функцию env вообще в коде юзать нельзя. Все переменные окружения нужно переносить в конфиги и желательно указывать дефолтные конфиги.
Но по факту если Вам непонятна сама логика то подучите OOP и SOLID

Comments

[Agent Gus]

Спасибо, адекватный ответ... Учту ваш ответ....

Answer 2

[bkosun]

Я не понимаю функционал этого кода, обЪесните пожалуйста.

parent::__construct(new \Corp\Repositories\MenusRepository(new \Corp\Menu));

Вызов конструктора родительского класса

php.net/manual/ru/language.oop5.decon.php

Comments

[Agent Gus]

Я имею ввиду new \Corp\Repositories\MenusRepository(new \Corp\Menu)

parent::__construct(new \Corp\Repositories\MenusRepository(new \Corp\Menu));

class MenusRepository extends Repository {
  public function __construct(Menu $menu) {
    $this->model = $menu;
  }

[bkosun]

yanminibaev: конструктор принимает в качестве параметра объект Menu $menu

public function __construct(Menu $menu) {}

Вы его передаете:

parent::__construct(new \Corp\Repositories\MenusRepository(new \Corp\Menu));

Или проще:

$menu = new \Corp\Repositories\MenusRepository(new \Corp\Menu);
parent::__construct($menu);

[Light Air]

yanminibaev: Псс... Парень... Смотри, что покажу.

Answer 3

[Вячеслав Плиско]

Автор этого кода не владеет ни php, ни laravel на должном уровне. Лучше читайте оригинальную документации и сотрите ларакасты. А ваш вопрос связан с php.
Этот код вызывает конструктор родительского класса, при этом интерфейс конструктора наследника отличает от родительского, что нарушает принципы SOLID.
php.net/manual/ru/language.oop5.decon.php

Comments

[Agent Gus]

У меня вопрос не про __construct();

[Вячеслав Плиско]

Ян Минибаев: ну там же обычный php
parent::__construct - вызывается родительский конструктор в который передаётся объект $menu
Это экземпляр класса MenusRepository в конструктор которого передаётся объект \Corp\Menu

[Василий Назаров]

Приплюсую к этому ответу.
За такие конструкторы имеет смысл автору в голову вбить гвоздь.

[Вячеслав Плиско]

Василий: Вы думаете что не понятна фантазия автора этого говнокода?

[Agent Gus]

Вячеслав Плиско: ну это не мой код, я теперь благодаря вам понял, что не стоит учиться у этого "гавнокодера". Спасибо, за ответ. А мне стоит учить документацию laravel.

[Вячеслав Плиско]

Ян Минибаев: только с документацией есть тонкий момент - примеры демонстрируют конкретный компонент, а не хорошие практики. Например, не использовать статические вызовы, а использовать внедрением зависимостей.

[Agent Gus]

Вячеслав Плиско: Есть ли толковый человек, который обучает laravel видеоуроки допустим? laracast на английском