Code Review mojego rozwiązania w Laravelu

Question

Witam, chcialbym was poprosić o opinię mojego rozwiązania dot. widoków w laravelu.

Tutaj kod UserPagesController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
use App\posts;


class UserPagesController extends Controller
{
    public function index()
    {
      $this->setting("title", "Strona Główna");
      return $this->site()->withPosts(DB::table('posts')->get());
    }
}
?>

A tutaj Controller.php

<?php

namespace App\Http\Controllers;

use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Routing\Controller as BaseController;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;

class Controller extends BaseController
{
    use AuthorizesRequests, DispatchesJobs, ValidatesRequests;

    private $settings;
    public function __construct()
    {
      $this->settings["title"] = "Bloggers";
      $this->settings["head"] = "Default";
      $this->settings["navbar"] = "Default";
      $this->settings["content"] = "Main";
    }

    public function setting($key, $value)
    {
      $this->settings[$key] = $value;
    }

    public function site()
    {
      return view("User")->withSettings($this->settings);

    }
}

 

Answer 1

Dobrze by było abyś opisał jeszcze jaki problem ten kod rozwiązuje. Podejrzewam, że chcesz mieć na różnych stronach identyczne bloki różniące się tylko tytułem (lub coś podobnego). Jeśli tak, to w laravel'u 7 wprowadzono coś takiego jak komponenty. Myślę, że zastosowanie ich będzie dużo lepszym rozwiązaniem.

Comments

Na różnych podstronach ma być inny kontent i tytuł. sekcja head i navbar zazwyczaj są takie same, jednak na wszelki wypadek istnieje opcja ustawienia czegoś innego.

---

To tak jak mówiłem, możesz spróbować użyć komponentu w laravelu. 

A zamiast 

DB::table('posts')->get()

Możesz użyć eloquent na przykład (zakładam, że model nazywa się Post):

Post::all()

 

---

Powiem tak spojrzałem na te komponenty i jednak moje rozwiązanie wydaje mi się wyglądać lepiej, jednak rozwiązanie z modelem Posta na pewno wykorzystam, bo rzeczywiście jest to krótszy zapis