laravel bladeでforeachの回数を指定したい

Question

laravelでモデルのリレーションを組んでいますが、
blade上のみで、foreachで回数を指定し、最新の3件のみを表示させたいです。

以下のようにして実現できているのですが、
@phpを使っており、もっと簡単、かつ@phpを使わない方法でforeachに回数を指定できないか悩んでおります
なにか方法はあるのでしょうか？
よろしくおねがいします

@php
$i = 0
@endphp

                    @forelse($user->productComments as $productComment)
                        @if($i >= 3)
                            @break
                        @endif

                        <div class="">
                                    @if($productComment->body)
                        </div>

@php
$i++
@endphp

                    @empty
                        <p class="text-center">まだありません。</p>
                    @endforelse

Answer 1

単にデータ取得時に３件だけ取得してくるように組めばblade上でゴチャゴチャせずに済むのでは。
結果、使わないデータを保持する必要もなくなりパフォーマンスにも良い影響が出ます。

View上にロジックを組むのは最小限におさえるべきです。まず組まなくて良い方法から検討してください。

Comments

[creative_09]

コントローラー側で3件のみを取得してくるということでしょうか？
もともとリレーションで紐付いており、userを取得すればcommentは取得できておりますので
コントローラー側ではuserしか取得する処理は書いておりません。

コントローラー側でcomment3件を取得し、viewに渡し、bladeで表示させた場合、
新たにコントローラー側で処理を追加するような動きになるかと思い、
リレーションで紐付いてるならそこから取得すればいいや！というふうに考えてしまいました。
foreachで回すだけなので。そこで最新3件の場合・・・・となったわけですが
パフォーマンス的にも余分なデータは持たないほうがいいですね。
やはりコントローラー側で処理を作ったほうが良いのでしょうか・・・

[m.ts10806]

通常は必要なデータだけ取得します。
表示用に加工が必要ならそれも加工したものをviewに渡します。

[creative_09]

必要な情報としてすべてのコメントを取得しているのでやはり、それを3件のみを必要情報としてコントローラーで処理しviewへ渡すという流れですね。
コントローラー側で絞り込みの処理を追記
viewへ渡す
この流れですね
ありがとうございます。

[m.ts10806]

$loop変数というのは存在しますが、今回のケースのために使われるものではないかと思います。
Viewは簡潔に、単に表示だけさせたほうが役割に合っていますし可読性も良く、問題切り分けがしやすくなります。現時点のコードも既にグチャグチャになってますしね。
viewでやろうとするのが間違いです。

[creative_09]

ありがとうございます。
おっしゃるとおり、コントローラー側で3件分の処理を追記する形で書き換え用と思います。
最後まで付き合って頂きありがとうございます。