Является ли хорошим решением вывод результата из контролера напрямую в шаблоне blade?

Question

[Kerm]

Я новичек в Laravel, в процессе создания сайта возникла необходимость создания модуля отвечающего за вывод горизонтального, вертикального меню категорий и разделов сайта, включая подвал сайта. Отмечу, что под модулем я подразумеваю компонент сайта, который можно подключить и вывести в любом месте шаблона.

Я создал контроллер, класс для контроллера, свои шаблоны для контроллера.

И вывожу в шаблоне, в нужном мне месте, результат такой строчкой кода, к примеру в header.blade.php:

{{(new App\Http\Controllers\MenuController)->init('horizontal')}}

Я сначала хотел выводить через переменные типа {{$menu_horizontal}}, но меня смутило то что скажем в контроллере главной страницы IndexController.php я должен буду обращаться к другому контроллеру, когда гуглил нашел запись что обращение из одного контроллера к другому не является хорошей практикой и решением и не совсем понял как это реализовать тем более.

В связи с этим вопрос, нормально ли такое обращение из шаблона blade к контролеру или это все ерись и скажем мне стоит лучше в моем модуле отказаться от контроллера и сделать обычный класс+свой шаблон blade и вывод через переменные в нужном мне шаблоне?

Comments

[Константин Б.]

bltr, просто месье не читал документацию и поэтому не знает о существовании ViewComposer, inject, component и еще кучи всего удивительного, что мог узнать начав с теории

[Александр Ковальчук]

Константин Б., Да бывает, какую-то область пропустил, главное чтобы сейчас разобрался и все будет ок

[JhaoDa]

Александр Ковальчук, автор пропустил ВСЁ, это видно по его вопросам.

Answer 1

[Александр Ковальчук]

В таких случаях лучше использовать inject

@inject('metrics', 'App\Services\MetricsService')

<div>
    Monthly Revenue: {{ $metrics->monthlyRevenue() }}.
</div>

Также этот сервис сделай синглтоном где в конструкторе будет брать данные с бд, а в методах их отдавать таким образом не будет несколько запросов даже если один компонент подключиться на странице несколько раз

P.S. Здесь в ответах был еще вариант расшарить переменные в провайдере но автор почему-то удалил его, а вариант вполне неплохой когда код точно есть на каждой странице (например меню)
Так что лучше всего будет попробовать два варианта и потом решить какой удобнее в твоем случае

Comments

[Kerm]

Здравствуйте, Александр! Я хотел спросить Вас про синглтон и расширение переменных в провайдере, можете посмотреть я правильно Вас понял и сделал:

<?php

namespace App\Providers;

use App\Models\Modules\Menu;
use Illuminate\Support\ServiceProvider;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        $this->app->singleton('Menu', function ($app) {
            return new Menu();
        });
    }

    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        view()->composer('modules.menu.horizontal', function($view) {
            $view->with(['menu' => Menu::getMenuHorizontal()]);
        });

        view()->composer('modules.menu.vertical', function($view) {
            $view->with(['menu' => Menu::getMenuVertical()]);
        });

        view()->composer('modules.menu.footer', function($view) {
            $view->with(['menu' => Menu::getMenuFooter()]);
        });
    }
}

А в классе Menu я уже в __contstruct() делаю запрос к БД и сохраняю результат в переменную типа $this->sql?

[Александр Ковальчук]

Kerm, Все ок, но можно регистрировать чуть проще
не так

public function register()
    {
        $this->app->singleton('Menu', function ($app) {
            return new Menu();
        });
    }

а вот так

public $singletons = [
        Menu::class => Menu::class,
    ];

Answer 2

[Владимир Кохан]

А че не подходит вариант через сервис-провайдер? Я в таких случаях просто пишу в AppServiceProvider

view()->composer(['header', 'footer'],  function($view){
            $view->with('menus', Menu::all());
        });

и все прекрасно работает

Comments

[Константин Б.]

Может быть не подходит потому что он не знает о существовании такого способа?

[Kerm]

Вот мне стало не понятно, в Вашем примере указано что в шаблонах header и footer в переменной menus выводить данные из Menu::all(), но мне нужно чтобы для каждого шаблона выводилась своя информация!

И это мне тогда нужно:

1) Создать провайдера php artisan make:provider ComposerServiceProvider
2) Прописать его в config/app.php
3) Создать файл app/Providers/ComposerServiceProvider.php с кодом:

<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use App\Models\Modules\Menu;

class ComposerServiceProvider extends ServiceProvider
{
    public function boot()
    {
        view()->composer('modules.menu.horizontal', function($view) {
            $view->with(['menu' => Menu::getMenuHorizontal()]);
        });

        view()->composer('modules.menu.vertical', function($view) {
            $view->with(['menu' => Menu::getMenuVertical()]);
        });

        view()->composer('modules.menu.footer', function($view) {
            $view->with(['menu' => Menu::getMenuFooter()]);
        });
    }
}

Еще раз повторюсь что мне нужно в каждом шаблоне несколько разное выводить в переменных.

4) И в нужном месте скажем index.blade.php мне прописать инклудом вывод этих шаблонов modules.menu.horizontal, modules.menu.vertical, modules.menu.footer?

Кто нибудь может ответить почему это является лучшим решением? Создавать провайдера, писать весь этот код, когда можно вывести напрямую из класса:

{{ App\Models\Modules\Menu::getMenuHorizontal() }}

[Владимир Кохан]

Kerm, ну так и напиши все это дело

view()->composer('modules.menu.horizontal', function($view) {
            $view->with(['menu' => Menu::getMenuHorizontal()]);
        });

        view()->composer('modules.menu.vertical', function($view) {
            $view->with(['menu' => Menu::getMenuVertical()]);
        });

        view()->composer('modules.menu.footer', function($view) {
            $view->with(['menu' => Menu::getMenuFooter()]);
        });

в AppServiceProvider