laravelで、プロフィール画像を表示したい

Question

laravelで、掲示板アプリを作成中です。
非ログイン状態の投稿一覧画面で、投稿１つ１つに、その投稿をしたユーザーのプロフィール画像が表示されるようにしたいです。
@authの中であれば、プロフィール画像の表示ができるのですが、@authの外だと、投稿一覧を表示しようとすると、Trying to get property of non-objectとエラーが発生してしまいます。
ログインしている状態でないと、$user->profile_imageが取れないので、そもそもテーブル定義の時点で破綻しているのでしょうか??

とった手法としては、usersテーブルに、profile_imageというカラムを追加し、S3に画像をアップロードしつつ、パスはデータベース上に登録するというものです。
テーブル定義はこちらになります。

users | CREATE TABLE `users` (
  `id` bigint unsigned NOT NULL AUTO_INCREMENT,
  `name` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `email` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `email_verified_at` timestamp NULL DEFAULT NULL,
  `password` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `remember_token` varchar(100) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
  `created_at` timestamp NULL DEFAULT NULL,
  `updated_at` timestamp NULL DEFAULT NULL,
  `profile_image` varchar(255) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
  PRIMARY KEY (`id`),
  UNIQUE KEY `users_email_unique` (`email`)
)

下記が該当箇所です。

          @auth
          <!-- 画像を表示 -->
          <img src="{{ $user->profile_image }}" style="width:50px; height:50px; border-radius:50%; position:relative; 
          top: 79px; left: 35px;">
        
          <p class="login-user-name">{{$user->name}}</p>
          <div id="p2">
            <a href="/logout">サインアウト</a>
            <p><a href="/profile">ユーザー</a></p>
          </div>

          @endauth

<?php

namespace App\Http\Controllers;
use App\Post;
use App\User;
use Storage;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;

class PostsController extends Controller
{
    public function index()
    {
      //ログインしているユーザー情報を取得
      $user = Auth::user();
      $posts = Post::all();
      return view('posts.index',['posts' => $posts,'user'=>$user]);
    }

    public function store(Request $request)
    {
      $request->validate([
        'text' => 'required',
      ]);
    
      // モデルからインスタンスを生成
      $post = new Post;
      // postsテーブルのtextに、フォームから送られたtextを保存する
      $post->text = $request->text;
      // userテーブルのidを、postsテーブルのuser_idとして保存する
      $post->user_id = $request->user()->id;
      // フォームリクエストの中に、画像ファイルが含まれているかどうかを条件分岐
      if($request->hasFile('image')){
      //s3アップロード開始
      $image = $request->file('image');
      // バケットの`test`フォルダへアップロード
      $path = Storage::disk('s3')->putFile('test', $image, 'public');
      // アップロードした画像のフルパスを取得
      $post->image_path = Storage::disk('s3')->url($path);
      }
      // 保存
      $post->save();
      // 保存後 一覧ページへリダイレクト
      return redirect('/');
      
    }

  
}

Answer 1

{{$post->user->profile_image}}とすることによって、解決できました。
あってますでしょうか...??

Comments

[hayato7]

あっていますよ。

[fork_]

この方法でしか値が取れない、と固くなっていました...。
投稿からユーザーを引っ張ってきて、ユーザーが持っているurlを取得する方法があるのですね....
とても勉強になりました。ご教示いただき、ありがとうございました！！

[hayato7]

付け加えていうなら、N+1問題、eager loadingなどについても知っておくとなお良いと思います。

https://qiita.com/tmf16/items/b9dd262c7738472c86d9

[fork_]

かしこまりました。いただいた記事、読んでみます。

Answer 2

投稿一覧画面に表示したいのは、他人の投稿を含む全ての投稿であるということだと思います。

その場合、$userから画像のurlを取得しようとしても、ログインユーザーの画像がとれてしまうことになります。（質問にある通り、ログインユーザーがいなければ、エラーが表示される）

それではいけないので、各投稿(Post)からそれに紐付くUserを引っ張ってきて、プロフィール画像のurlを取得します。
詳しい方法は書きませんが、『laravel リレーション』などと調べれば色々情報が得られると思います。
（ちなみに今回の場合、UserとPostの関係は、1対多です)

Comments

[fork_]

ありがとうございます。苦戦中です...。
プロフィール画像を保存する時点で、userテーブルとpostsテーブルを紐付けないといけないのでしょうか??
取得するときだけ紐付ければ良いでしょうか??　ちなみに、モデル側ではリレーションの設定をしている状態で、コントローラー側で値を取得する方法がわからない状態です。

[hayato7]

> プロフィール画像を保存する時点で、userテーブルとpostsテーブルを紐付けないといけないのでしょうか??
その際に紐付ける必要はありません。
いま投稿を保存するときに、postsテーブルにuser_idを保存していますよね。（このpostsテーブルがもつuser_idを外部キーといい、他のテーブルの情報と紐付ける役割をしています）
なので、投稿データを取得する際に、そのデータに紐づいているユーザー情報も取得してあげればいいです。

いまスマホから打っているのですみませんが、詳しいコードについては、夜になってもいいなら、提示できると思います。