Как вынести из контроллера часть логики?

Question

[Mike Evstropov]

К примеру, у нас есть страничка main, на которой будут находится разные блоки (поиск, статьи, регистрация, etc). Разумеется, блоки будут повторяться в других контроллерах. Мне нужно как-то вынести формирование переменных/массивов передаваемых в шаблон для генерации этих блоков в отдельные контроллеры (или другие сущности, которых я не знаю). Что я пытался сделать:

Cоздаем MainController

<?php
// /src/AppBundle/Controller/MainController.php
namespace AppBundle\Controller;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Response;
class MainController extends Controller {
	/**
     * @Route("/", name="main")
     */
	public function mainAction () {
		$filter_form = $this->forward( 'app.test:testAction' );
		return new Response ( json_encode(Array( 'filter_form' => $filter_form )) );
	}
}

Пытаемся вынести формирование строки в другой "сервис", под названием TestController.

# app/config/services.yml
services: app.test: class: AppBundle\Controller\TestController

Создаем контроллер TestController, который должен вернуть строку.

<?php
// /src/AppBundle/Controller/TestController.php
namespace AppBundle\Controller;
class TestController {
	public function testAction () {
		return 'STRING';
	}
}

Но TestController возвращает Response объект, и в выводе мы видим:
{"filter_form":{"headers":{}}}

А нужно
{"filter_form":"STRING"}

Только начинаю разбираться с фреймворком. Подскажите, как сделать так, что бы контроллер возвращал то, что передается оператору return. Или как сделать правильно, что бы не дублировать код по контроллерам (то есть вынести логику из MainController)?

Comments

[Mike Evstropov]

Мне тут подсказывают, что вынести что-то из контроллера нельзя. Нужно формировать в модели, а потом уже из нее запрашивать данные в контроллере. Верно ли это?

[Артем Сошников]

Mike Evstropov: Не совсем, в symfony модели, а точнее сущности, не делают по сути ничего, кроме того, что существуют. Это такой ленивый объект, который говорит, что у него есть (свойства), и что с этим можно сделать (методы).
Выборка, сохранение, удаление лежат на плечах различных менеджеров.
P.S. Для выборки лучше всего использовать репозитории, вместо прямого построения запроса через Doctrine

Answer 1

[Артем Сошников]

1. Создайте класс-сервис mySuperManager
2. Запихните в него всю логику
3. В контроллере попросите mySuperManager отдать нужные данные и передавайте их во вьюху или json или еще куда)

Пример контроллера:


Пример метода класса mySuperService


В моем случае mySuperManager возвращает сразу json, но в Вашей ситуации возвращайте то, что Вам нужно: объект, массив, строка и т.д.

Comments

[Mike Evstropov]

С наскока ничего не понял, но похоже то что нужно =) Спасибо за пример! Буду разбираться.

[Mike Evstropov]

УРА! Разобрался! forward используется как раз что бы получить response, нужно $this->get('сервис')->метод и тогда можно получить что хочешь. =) Если я правильно понял, лучше создать разные сервисы, что бы не подключать ненужные методы лишний раз.

[Артем Сошников]

Да, все верно. Каждый менеджер будет отвечать за свои задачи. К примеру AlbumManager, PageManager и т.д. Так все будет хорошо структурировано и поделено на логические части.

[Артем Сошников]

Дополнение: бытует мнение, что называть классы как Manager, Helper и другими неоднозначными словами не очень хорошо. Критика сводится к тому, что таким образом нельзя легко и быстро понять из названия класса чем он занимается, и не является ли он одним из примеров god-object.