Question 16060 in Laravel Question posted in
The official Laravel Documentation can be found here.

Attempt to read property "menus" on null – Laravel

when i run laravel project on my computer after doing php artisan serve as soon as it login it shows error "Attempt to read property "menus" on null"

Please help me

 @foreach ($specials->menus as $menu)
                    <div class="max-w-xs mx-4 mb-2 rounded-lg shadow-lg">
                        <img class="w-full h-48" src="{{ Storage::url($menu->image) }}" alt="Image" />
                        <div class="px-6 py-4">
                            <h4 class="mb-3 text-xl font-semibold tracking-tight text-green-600 uppercase">
                                {{ $menu->name }}</h4>
                            <p class="leading-normal text-gray-700">{{ $menu->description }}.</p>
                        </div>
                        <div class="flex items-center justify-between p-4">
                            <span class="text-xl text-green-600">${{ $menu->price }}</span>
                        </div>
                    </div>
                @endforeach

welcomecontroller.php

<?php

namespace AppModels;

use IlluminateDatabaseEloquentFactoriesHasFactory;
use IlluminateDatabaseEloquentModel;

class Menu extends Model
{
    use HasFactory;

    protected $fillable = ['name', 'price', 'description', 'image'];

    public function categories()
    {
        return $this->belongsToMany(Category::class, 'category_menu');
    }
}
Tags:

2

Answers


  1. 0

    I suppose you forgot to initialize $specials, it is null, that’s why you get the error.

    Could you try to use dd before foreach statement, so we can see it’s content?

    {{ dd($specials) }}

    UPDATE:

    @if(!is_null($specials))
 @foreach ($specials->menus as $menu)
                    <div class="max-w-xs mx-4 mb-2 rounded-lg shadow-lg">
                        <img class="w-full h-48" src="{{ Storage::url($menu->image) }}" alt="Image" />
                        <div class="px-6 py-4">
                            <h4 class="mb-3 text-xl font-semibold tracking-tight text-green-600 uppercase">
                                {{ $menu->name }}</h4>
                            <p class="leading-normal text-gray-700">{{ $menu->description }}.</p>
                        </div>
                        <div class="flex items-center justify-between p-4">
                            <span class="text-xl text-green-600">${{ $menu->price }}</span>
                        </div>
                    </div>
                @endforeach
@else
<div>no item was found</div>
@endif
    Login or Signup to reply.
  2. 0

    Your issue is with this line of code (from the comments):

    $specials = Category::where('name', 'specials')->first();

    ->first(); can return null, so when you use it later as $specials->menus, it can be null->menus, which is not valid.

    You can use ->firstOrFail() to trigger a 404 in the event $specials results in null, or @foreach($specials->menus ?? [] as $menu) to short-circuit your foreach() with an empty array if $specials is null:

    In your Controller:

    $specials = Category::where('name', 'specials')->firstOrFail();

return view('welcome', compact('specials'));

    OR

    In your View:

    @foreach($specials->menus ?? [] as $menu)
  ...
@endforeach

    Either case will properly handle your "unsafe" code (unsafe meaning functional, but possible for unhandled errors, like null->menus, etc.) and either trigger a 404 before the view is rendered, or perform a foreach() on an empty array, which does nothing.

    Login or Signup to reply.
Please signup or login to give your own answer.

Back To Top
Search