Trying to get property 'articles' of non-objectというエラーを解決したい

Question

https://teratail.com/questions/313328の続きです。

エラー全文

Trying to get property 'articles' of non-object

articlesなんていうプロパティ無いよ～っていうエラーだと思うのですが
Article.phpあるいはUser.phpなどのモデルが悪さをしているのかなと思いますが皆目分かりません。
どうかよろしくお願いします。

環境
Laravel：Laravel Framework 6.18.20
OS: Windows10

UserController.php

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Http\Request;

class UserController extends Controller
{

  public function show(string $name)
    {
        $user = User::where('name', $name)->first()
        ->load(['articles.user', 'articles.likes', 'articles.tags']);  

         $articles = $user->articles->sortByDesc('created_at'); // ここがnon-objectになってしまう

        return view('users.show', [
            'user' => $user,
            'articles' => $articles,
        ]);
    }

  public function edit(string $name)
  {
    $user = User::where('name', $name)->first();

        // UserPolicyのupdateメソッドでアクセス制限
    $this->authorize('update', $user);

    return view('users.edit', ['user' => $user]);

  }

Article.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Database\Eloquent\Relations\BelongsToMany;

class Article extends Model
{
   protected $fillable = ['title', 'body'];

    public function user(): BelongsTo
    {
        return $this->belongsTo('App\User');
    }

    public function likes(): BelongsToMany
   {
       return $this->belongsToMany('App\User', 'likes')->withTimestamps();
   }

   public function isLikedBy(?User $user): bool
   {
       return $user
           ? (bool)$this->likes->where('id', $user->id)->count()
           : false;
   }

User.php

<?php

namespace App;

use App\Mail\BareMail;
use App\Notifications\PasswordResetNotification;
use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Database\Eloquent\Relations\BelongsToMany;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;

class User extends Authenticatable
{
    use Notifiable;


    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password', 'user_id', 'self_introduction',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];

    public function sendPasswordResetNotification($token)
    {
       $this->notify(new PasswordResetNotification($token, new BareMail()));
    }

    public function articles(): HasMany
    {
        return $this->hasMany('App\Article');

    }

追記

LoginController.php

<?php

namespace App\Http\Controllers\Auth;

use App\Http\Controllers\Controller;
use App\User;
use App\Providers\RouteServiceProvider;
use Illuminate\Foundation\Auth\AuthenticatesUsers;
use Illuminate\Http\Request;
use Laravel\Socialite\Facades\Socialite;

class LoginController extends Controller
{
    /*
    |--------------------------------------------------------------------------
    | Login Controller
    |--------------------------------------------------------------------------
    |
    | This controller handles authenticating users for the application and
    | redirecting them to your home screen. The controller uses a trait
    | to conveniently provide its functionality to your applications.
    |
    */

    use AuthenticatesUsers;

    /**
     * Where to redirect users after login.
     *
     * @var string
     */
    protected $redirectTo = RouteServiceProvider::HOME;

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('guest')->except('logout');
    }

    protected function authenticated(Request $request)
    {
        // フラッシュメッセージを表示
        session()->flash('flash_message', 'ログインしました');
        return redirect('/');
    }

    public function redirectToProvider(string $provider)
    {
        return Socialite::driver($provider)->redirect();
    }

    public function handleProviderCallback(Request $request, string $provider)  // Googlelogin
    {
        $providerUser = Socialite::driver($provider)->stateless()->user();

        $user = User::where('email', $providerUser->getEmail())->first();

        if ($user) {
            $this->guard()->login($user, true);
            return $this->sendLoginResponse($request);
        }

        return redirect()->route('register.{provider}', [
            'provider' => $provider,
            'email' => $providerUser->getEmail(),
            'token' => $providerUser->token,
        ]);
    }
}

ルーティング

<?php
Auth::routes();
// グーグルログイン
Route::prefix('login')->name('login.')->group(function () { 
    Route::get('/{provider}', 'Auth\LoginController@redirectToProvider')->name('{provider}');
    Route::get('/{provider}/callback', 'Auth\LoginController@handleProviderCallback')->name('{provider}.callback');
});

Route::get('/about','AboutContoroller@about')->name('articles.about');

Route::prefix('register')->name('register.')->group(function () {
    Route::get('/{provider}', 'Auth\RegisterController@showProviderUserRegistrationForm')->name('{provider}');
    Route::post('/{provider}', 'Auth\RegisterController@registerProviderUser')->name('{provider}');
});
Route::middleware('auth')->group(function () {
  Route::get('/edit', 'UserController@edit')->name('edit'); // ユーザーのプロフィール編集
  Route::put('/{name}/follow', 'UserController@follow')->name('follow');
  Route::delete('/{name}/follow', 'UserController@unfollow')->name('unfollow');
    });
});

Route::resource('/comments', 'CommentController')->only(['store']);

Answer 1

そもそも質問者さんは、合致するユーザーがいなかった場合の挙動はどうするおつもりでしょうか。
そのまま、Viewまでもっていくのでしょうか？

HTTP 404エラーにしたほうが後々楽です。

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Http\Request;

class UserController extends Controller
{

    public function show(string $name)
    {
        if (User::where('name', $name)->doesntExist())
            abort(404);

        $user = User::where('name', $name)->first()
            ->load(['articles.user', 'articles.likes', 'articles.tags']);

        $articles = $user->articles->sortByDesc('created_at');

        return view('users.show', [
            'user' => $user,
            'articles' => $articles,
        ]);
    }

    public function edit(string $name)
    {
        $user = User::where('name', $name)->first();

        // UserPolicyのupdateメソッドでアクセス制限
        $this->authorize('update', $user);

        return view('users.edit', ['user' => $user]);
    }
}

Comments

[Tikka123456]

回答ありがとうございます。そもそも、認証しているユーザーのプロフィール編集機能を作ろうとしているので合致するユーザーがいないというのは起こりにくいのかなと思います。

[kyoya0819]

現に前の質問文では「いない」という状況が起きているようですが。。

[Tikka123456]

すいません、確かにそうですね。kyoyaさんが提示したコードを試したら案の定404になりました。
ユーザー認証でユーザのデータが格納されていないという事なんですかね？

[kyoya0819]

ご提示のコードには「一切」認証に関わるコードがありません。

[kyoya0819]

なので、「格納されていないのか？」というご質問にはお答えできません。

[Tikka123456]

認証関係のコードを追記しました。SQLでSELECT ＊ ～を実行した際ちゃんとユーザーの情報は格納されていたので恐らく問題は無いのかなと思います。

Answer 2

: HasMany

を削除して、

public function articles()
{
    return $this->hasMany('App\Article');
}

で、動きませんか。
Laravel公式ドキュメントを参考に回答しました。

Comments

[Tikka123456]

回答ありがとうございます！結果は変わりませんでした。

Answer 3

「Trying to get property 'articles' of non-object」でググってみました。
下記が参考になるかもです。
Laravelで、Trying to get property of non-object　と出たときの2つの解決策

Comments

[Tikka123456]

回答ありがとうございます。すいません、せっかく調べていただいたのですがその記事のはもう試してしまいました。結果は変わりませんでした。