EloquentでDBから取得したデータに要素を追加してBladeに渡したい

Itemsテーブルから取得したデータを、フロントへ出力する際に要素を追加してBladeファイルに渡したいです。

php
1public function up()
2    {
3        Schema::create('items', function (Blueprint $table) {
4            $table->id();
5            $table->string('name');
6            $table->string('price');
7            $table->timestamps();
8        });
9    }

現在は以下のようにtapヘルパを利用して配列を整形していますが、このやり方は正しいでしょうか？
他にシンプルに記述できる方法があれば教えていただきたいです。

php
1class ItemComposer
2{
3    public function compose(View $view)
4    {
5        $items = Items::paginate(12);
6
7        foreach ($items as $item) {
8            tap($item, function ($item) {
9                $dt = new Carbon($item->created_at);
10
11                $item->date = $dt->format('Y/m/d');
12                $item->title = "{$item->name} {$item->price}";
13                $item->url = route('item.show', $item->slug);
14            }
15        }
16
17        $view->with(compact('items'));
18    }
19}

m.ts10806

2020/05/27 19:15

コードだけでは要件伝わりません。何をどのようにしたいのか具体的に記載してください。どういう情報があるのか、サンプルデータや定義も例示してください。あと、できてるならそれでいいのではと思うのですが、「きれいに」という感覚によるものだとアドバイスは得られにくいかと思います。もう少しゴールを具体的にしてください。

行動規範の内容に同意します

回答1件

ベストアンサー

php
1
2        foreach ($items as $item) {
3            tap($item, function ($item) {
4                $dt = new Carbon($item->created_at);// なぜこんなことを？created_at はこんなことしなくてもはじめから　Carbon クラスですよ
5
6                $item->date = $dt->format('Y/m/d');
7                $item->title = "{$item->name} {$item->price}";
8                $item->url = route('item.show', $item->slug); // blade で route('item.show', compact('item)) でよくない？
9            }
10        }

このコードを見る限り、Model でアクセサを使えば済むことじゃないか？と思います。

php
1
2class Item extends Model
3{
4    /**
5     * @example $item->title
6     */
7    public function getTitleAttribute(){
8        return $this->name . ' ' . $this->price;
9    }
10}