Как исправить ошибку?

Question

[rsvega]

Возникает ошибка Error Call to a member function goods() on null , при переходе на страницу, при этом возникает в той функции контроллера, которая никак не связана со страницей

public function getGoodsByCategory($slug){
        $categories = Category::orderBy('title')->get();
        $current_category = Category::where('slug',$slug)->first();

        return view('pages.index', [
            'goods' => $current_category->goods()->paginate(4),
            'categories' => $categories,
        ]);
    }

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

public function RequestCategory(){
        return view('pages.requestcategory');
    }

public function RequestCategoryCheck(Request $request){
        $valid = $request->validate([
            'title' => 'required',
            'slug' => 'required',
        ]);
        $review = new category();
        $review->title = $request->input('title');
        $review->slug = $request->input('slug');

        $review->save();

        return redirect()->route('RequestCategory');
    }

Comments

[JhaoDa]

Код оформи нормально.

Answer 1

[sl0]

А что в ошибке непонятного? По такому слагу нет категории, соответственно и получить продукты из нее не выходит.

Comments

[rsvega]

читай пожалуйста вопрос внимательнее, перед тем, как дать ответ)
я написал, что по идее эта функция не должна использоваться на этой странице, потому что я её вообще с ней никак не связывал, а значит и на этой странице она выводить ничего не должна
отсюда у меня и вопрос
почему он ругается на слаг, если там его нет и он вообще никак с той страницей не связан
Вот содержание web.php , которое относится к нужной странице

Route::get('/category/add', [CategoryGoodController::class, 'RequestCategory'])->name('RequestCategory');
Route::post('/category/add/check',[CategoryGoodController::class, 'RequestCategoryCheck']);

Вот содержание blade.php файла

@extends('layouts.main-layout')

@section('title', 'Add new categories')
<div class="card-header">
    <a href="/">All goods</a>
    <a href="/category/add">Add category</a>
    <a href="/good/add">Add good</a>
</div>

@section('content')
    <div class="row featurette" style="padding-top: 156px;">
    <div class="col-md-7">
        <h1 class="featurette-heading">Add new category</h1>
        <br>
        @if($errors->any())
            <div class="alert alert-danger">
                <ul>
                    @foreach($errors->all() as $error)
                        <li>{{$error}}</li>
                    @endforeach
                </ul>
            </div>
        @endif
        <br>
        <form method="post" action="/category/add/check">
            @csrf
            <input type="string" name="title" id="title" placeholder="Enter here name of product " class="form-control"><br>
            <input type="string" name="slug" id="slug" placeholder="Enter here slug " class="form-control"><br>
            <button type="submit" class="btn btn-success">Add category</button>
        </form>
    </div>
</div>
@endsection

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

[jazzus]

rsvega, роут с getGoodsByCategory нужно передвинуть за другие роуты категорий и использовать Model binding как написали иначе любой урл /category/sdefsfsdfsdfs из любого места будет вызывать ошибку

Answer 2

[Pavel Omelchenko]

$current_category = Category::where('slug',$slug)->first();

Воспользуйся таким способом

https://laravel.com/docs/8.x/routing#customizing-t...

Route::get('/posts/{post:slug}', function (Post $post) {
    return $post;
});

Если slug'а нет, то контроллер закончит работу через ответ с 404

Или же используй firstOrFail() что тоже приведет к 404 если ничего не найдет

Comments

[rsvega]

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

[Pavel Omelchenko]

Дело в том что твой запрос возвращает null в котором ты пытаешься использовать методы. Но раз решения, когда елоквент кинет исключение в случае если данных с требущимся слагом, не подходят. То я не знаю чем тебе помогать