0

роут:

Route::get('/page/{url}', 'PageController@page')->where('url', '[a-z]+');

контроллер:

<?php

namespace App\Http\Controllers;

use App\Category;
use App\Page;
use Illuminate\Http\Request;
use DB;

class PageController extends Controller
{

    protected $category;
    protected $page;

    public function __construct()
    {
       // Коллекция всех категорий
      $this->category = Category::all();
      // Коллекция всех страниц
      $this->page = Page::all();
    }

    public function page($url) {
        $page_info = DB::select("select * from pages where url ='". $url ."'");
        $title = $page_info[0]->title;
        $description = $page_info[0]->description;
        $catalog = $this->category;
        $page = $this->page;

        return view('page', ['page' => $page, 'catalog' => $catalog, 'title' => $title, 'description' => $description]);
    }

}

вьюшка:

@extends('layout')

@section('content')
  <h1>{{$title}}</h1>
  <p>{{$description}}</p>
@endsection

Подскажите плиз, такой способ рабочий, НО что-то подсказывает что так делать нельзя (не профессионально). Может подскажите как лучше?

1 ответ 1

3
  1. Используйте Eloquent или возможности Сервис контейнера, это куда проще и удобней.

    Если используете Laravel 7 можно сделать так.

    //Route.php
    Route::get('page/{page:slug}', 'PageController@show');
    
    |------------------------------------------------------------|
    
    //PageController.php
    public function show(Page $page)
    {
        return view('page', compact('page'));
    }
    
    |------------------------------------------------------------|
    
    //page.blade.php
    @section('content')
      <h1>{{$page->title}}</h1>
      <p>{{$page->description}}</p>
    @endsection
    
  2. Давайте осмысленные и запоминающиеся имена переменным и методам, у вас свойство $page вместо 1 страницы содержит коллекцию.

  3. Список всех страниц и категорий можно вынести View Composers или попробовать использовать новые компоненты

p.s. Посмотрите курсы от Laracast по Larvael.

6
  • Большое вам спасибо за ответ и советы. Но такой способ у меня не работает, 404 ошибка...
    – ilyaaa521
    8 мар 2020 в 2:07
  • @ilyaaa521, можете оставить свой роут, а в контроллер добавить show($url) { $page=Page::where('url', $url)->get();}.
    – E_K
    8 мар 2020 в 10:27
  • Большое спасибо еще раз, работает! $page_info = Page::where('url', $url)->get(); смотрится лучше чем $page_info = DB::select("select * from pages where url ='". $url ."'"); Правда не понимаю, чем лучше такой подход кроме удобства и читабильности кода.
    – ilyaaa521
    8 мар 2020 в 10:52
  • @ilyaaa521, Eloquent ORM предоставляет возможность работать с базами в ООП стиле. Это у вас сейчас простой SELECT, а дальше пойдут более сложные запросы с которыми Eloquent справляется лучше и быстрее, чем это будете делать вы. Вам нужно посмотреть реальные примеры работы с Eloquent и сразу поймете преимущества.
    – E_K
    8 мар 2020 в 11:19
  • Принял! А есть прмиеры стоящие? )
    – ilyaaa521
    8 мар 2020 в 11:36

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.