Насколько плохо echo внутри метода?

Question

[dev400]

Это контроллер. Приходится делать echo лейаута. Насколько это криво?

public function index()
	{
		$layout = new View("/layouts/main");
		$view = new View("/site/index");
		$layout->set( 'content', $view->render() );
		$layout->set( 'title', $view->title );
		echo $layout->render();
	}

Планирую переделать вот так

public function index() {

        $news = new View( "/site/news/index", ['model' => $model, 'msg' => $this->msg] );
        $layout = new View( "/layouts/news", ['content' => $news->render()] );
        $layout->set( 'title', $news->title );
        Registry::set('layout', $layout->render() );
     
    }

И потом соответственно
echo Registry::get('layout');

Answer 1

[Антон Иванов]

Следуя канонам PHP, любой метод должен возвращать значение. Так что ваш вариант "кривоват". Другое дело, если вы используете какой-то фреймворк, в котором так положено делать. Например в Yii 1.0 экшены контроллера именно так и выглядели. Во второй же версии Yii 2.0 это было исправлено на правильное поведение. Любой экшн возвращал какое-то значение.

Answer 2

[Денис Кайназаров]

Если вы используете фреймворк то шаблон должен выводится средствами фреймворка.

Если самописка то в проекте должен быть только один финальный echo который выводит уже готовый шаблон.

echo на прямую в контролере - однозначно зло.

Answer 3

[Rou1997]

Самописный фреймворк, что ли?

Вот так сделайте:

public function index()
  {
    $layout = new View("/layouts/main");
    $view = new View("/site/index");
    $layout->set( 'content', $view->render() );
    $layout->set( 'title', $view->title );
    return $layout->render();
  }

Возвращаемое значение должен обрабатывать сам фреймворк, то есть он вызывает метод index (с помощью "рефлексии") и возвращаемый код выводит в echo.

Comments

[dev400]

да, рефлексию мне уже посоветовали, но я пока остановился на реестре. Спасибо!

[Rou1997]

Лично я одним из первых требований к любому фреймворку считаю лаконичность. Ну, то есть я за RAD (Rapid Application Development) и все такое.
Может быть, и вам стоит?
Зачем вообще в каждом Action вручную задавать layout? Этим фреймворк должен заниматься по умолчанию.
Получается так:
public function index()
{
return $this->render("/site/index");
}

[dev400]

Rou1997: В каждом экшене задавать его не нужно, просто в моем примере экшн с кастомным layout

[Rou1997]

Андрей: А почему так... хм... низкоуровнево? Какой-то content, какой-то title вручную задавать.
Уж лучше бы так:
public function index()
{
return $this->render("/site/index", [ "layout" => "main" ]);
}
Хотя, конечно, чем больше кода вы взвалите на приложение, тем проще и легче будет написать фреймворк :) Нужно на задачу ориентироваться, и на последующие задачи, вы же фреймворк не для одного проекта пишете.

[dev400]

Rou1997: тут выбор или в удобство, или в возможности. В Вашем случае как отрендерить письмо, или несколько виджетов?

[Rou1997]

Андрей: почему выбор? В хорошем фреймворке должно быть как можно больше возможных вариантов.
В отличие от "нехороших" фреймворков, таких, как API операционных систем, при разработке которых работают на количество функций, чтобы был доступ к максимум возможностей ОС, а о качестве не думают.

Answer 4

[shagguboy]

в Симфони экшн возвращает отрендеренный шаблон. выводится он позже.

Answer 5

[Ширшов Александр]

лучше сделайте в контроллере

return$layout;

А в месте где формируете ответ уже и выводите.