Как правильнее подтянуть форму на главную?

Question

[Александр]

Имеется контроллер главной страницы HomeController
также, на главной должна быть форма

Форма - отдельная сущность(Book) со своим контроллером и классами(заполняется в несколько этапов, на каждый этап - отдельный класс)

Как правильнее было бы сделать, в HomeController вызвать формбилдер
или в Контроллере формы сделать метод и к нему обратиться из HomeController-а

Answer 1

[Алексей Скобкин]

Best Practice
Define your forms as PHP classes.

https://symfony.com/doc/current/best_practices/for...

Таким образом вы сможете использовать одну и ту же форму в разных местах выполняя DRY.

Comments

[Александр]

Алексей Скобкин Да, я так и делаю
создал несколько класс формы
и в коетроллере главной страницы могу вызвать формбилдер

class HomeController extends Controller
{
    /**
     * @Route("/")
     */
    public function indexAction(Request $request)
    {
 $book = new Book();
 $form = $this->createFormBuilder($book)
//.............
}
}

вопрос в том, не правильней ли формбилдер вызывать в BookController
а уже из HomeController вызывать метод из BookController ?

[Юрий]

Александр: нет.
разве что можно сделать отдельный экшен который рендерит эту форму и в твиге
{{ render(controller(... )) }}

[Александр]

Юрий: спасибо, попробую

[Алексей Скобкин]

Александр:

Да, я так и делаю

В коде, который вы привели ниже, вы делаете не так.
Я не вижу, где вы создаете форму. Я вижу, что вы создаёте сущность и инициализируете FormBuilder с нуля.

[Алексей Скобкин]

Александр: Почитайте этот раздел документации: https://symfony.com/doc/current/forms.html#creatin...

[Александр]

Алексей Скобкин: вроде читал и сделал по образцу

class HomeController extends Controller
{
    /**
     * @Route("/")
     */
    public function indexAction(Request $request)
    {
        $book = new Book();

        $form = $this->createForm(BookType::class, $book);
        $form->handleRequest($request);

        if ($form->isSubmitted() AND $form->isValid()) {
            $em = $this->getDoctrine()->getManager();
            $em->persist($book);
            $em->flush();

            // дописываем информацию про идентификатор клиента
            $this->get('session')->set('book_id', $book->getId());

            return $this->redirectToRoute('app_book_transfer', array(
                'form' => $form,
                'request' => $request
            ));
        }
        return $this->render('AppBundle:Book:find.html.twig', array(
            'add_selects' => $additionalSelects,
            'form' => $form->createView(),
        ));
    }
}

и код класса формы

class BookType extends AbstractType
{
    /**
     * @param FormBuilderInterface $builder
     * @param array $options
     */
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('date', DateTimeType::class, array('widget' => 'single_text'))
            ->add('fromBook')
            ->add('toBook')
            ->add('people');
    }

    /**
     * @param OptionsResolver $resolver
     */
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults(array(
            'data_class' => 'AppBundle\Entity\Book'
        ));
    }
}

я создаю новый объект Book
Это новое бронирование
и передаю его в формбилдер

[Алексей Скобкин]

Александр: Теперь больше похоже на правду, чем то, что было указано в вопросе.