Laravelで画像投稿機能実装後、画像を投稿していないにも関わらず画像を取得しようとしてしまう

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(Laravel)で簡易的なSNSを作っています。
画像投稿機能を実装したのですが、画像を投稿しなかった場合にも以下のように表示されてしまいます

ご教授いただければ幸いです

        <p>
          {{ $post->body }}
        </p>
        
        <hr>
        <a class= "sample"  href="/users/{{ $post->user->id }}">
          <img src="/storage/post_images/{{ $post->id }}.jpg" class="card-img-bottom">
        </a>

ポストコントローラー

public function index()
    {
   $posts = Post::latest()->get();
        
            
        return view('post/index', ['posts' => $posts]);
    }
  public function new()
    {
        return view('post/new');
        
    }
    
    public function store(Request $request)
    {
        //バリデーション
        $validator = Validator::make($request->all() , ['caption' => 'required|max:255']);

        //エラー
        if ($validator->fails())
        {
            return redirect()->back()->withErrors($validator->errors())->withInput();
        }

        $post = new Post;
        $post->caption = $request->caption;
        $post->user_id = Auth::user()->id;

        $post->save();
        
        if($request->photo == null){
             return redirect('/');
        }
        
         else{
        $request->photo->storeAs('public/post_images', $post->id . '.jpg');
        
        return redirect('/');
         }
    }
}

試したこと

[PHPでファイルの存在を確認する関数の使い方を現役エンジニアが解説【初心者向け】]
(https://techacademy.jp/magazine/22088)
を参考に下記のコードで画像ファイルがある場合にaタグの中身を表示するようなif文を書いてみたのですが見た目は変わりませんでした

        <?php
        $filename = "/storage/post_images/{{ $post->id }}.jpg";
        if (file_exists($filename))
        ?>
        <a class= "sample"  href="#">
          <img src="/storage/post_images/{{ $post->id }}.jpg" class="card-img-bottom">
        </a>

補足情報（FW/ツールのバージョンなど）

Laravel Framework 5.5.48,
PHP 7.2.28
AWS Cloud9
mysql Ver 14.14 Distrib 5.7.26

Answer 1

画像ファイルがある場合にaタグの中身を表示するようなif文を書いてみたのですが

Storage ファサードを使って以下のようにしてみてはいかがでしょうか。

@if (Storage::exists("post_images/{$post->id}.jpg"))
    <a class="sample" href="#">
        <img src="{{ Storage::url("post_images/{$post->id}.jpg") }}" class="card-img-bottom">
    </a>
@endif

参考: Laravel 5.5 ファイルストレージ

Comments

[takamori]

ありがとうございます！
解決しました

Answer 2

<a class= "sample"  href="/users/{{ $post->user->id }}">
  <img src="/storage/post_images/{{ $post->id }}.jpg" class="card-img-bottom">
</a>

ここで条件なしに表示する表にしているのでそのような挙動は正常です。

これの回避策はいくつか考えられますが、
以下のようにセッションに結果を保存して、Bladeで表示するかどうかの判定をするのはいかがでしょうか？

public function index()
    {
   $posts = Post::latest()->get();


        return view('post/index', ['posts' => $posts]);
    }
  public function new()
    {
        return view('post/new');

    }

    public function store(Request $request)
    {
        //バリデーション
        $validator = Validator::make($request->all() , ['caption' => 'required|max:255']);

        //エラー
        if ($validator->fails())
        {
            return redirect()->back()->withErrors($validator->errors())->withInput();
        }

        $post = new Post;
        $post->caption = $request->caption;
        $post->user_id = Auth::user()->id;

        $post->save();

        if($request->photo == null){
             return redirect('/')->with('image', false); //->with..を追加
        }

         else{
        $request->photo->storeAs('public/post_images', $post->id . '.jpg');

        return redirect('/')->with('image', true); //->with..を追加
         }
    }
}

@if(session('image'))
<a class= "sample"  href="/users/{{ $post->user->id }}">
  <img src="/storage/post_images/{{ $post->id }}.jpg" class="card-img-bottom">
</a>
@else
画像はありません
@endif

特にテストしていないので、エラーが出たり、動作しなかった場合は言ってください。

Comments

[takamori]

回答ありがとうございます。
いただいたコードを以前のものと入れ替えてみたのですが、
画像を投稿した場合、以前のものと同じ挙動でした。
画像を投稿しなかった場合やページを更新した時に、全ての投稿に画像はありませんと出力されてしまい、以前投稿した画像も表示されなくなってしまいました。

ご教授いただければ幸いです