Laravel:Controllerからviewに配列が渡せない

Question

こんにちは。

Laravelを使って、ログイン中のユーザーがフォームのチェックボックスに回答して、その回答内容を、/menu/{user_id}のユーザーのプロフィールページ(ログイン不要)からみられるようにしたいです。

複数チェック可のチェックボックスの回答の配列をimplodeメソッドで文字列に変換してDBに格納しています。次にDBからその文字列を引っ張て来て、explodeメソッドを使ってviewに表示しようとしています。回答はproductsテーブル内のallergyというカラムに文字列を格納しているので、Controllerとviewのコードを以下のようにしています。

Controller

    public function menu($user)
    {
        $user = User::findOrFail($user);

        $allergy = $user->products->allergy->get();
        $allergy_array = explode(',', $allergy);

        return view('new_menu')->with(['user' => $user, 'allergy_array'=>$allergy_array]);
    }

view

@foreach($user->products as $pro)
@if($pro->category_id == 1)
<div>                                        
    @if (in_array(1, $allergy_array))
    ...
    @endif
</div>
@endif
@endforeach

しかし、これだと

Property [allergy] does not exist on this collection instance.

というエラーが生じてしまいます。
get()をfirst()にしてみたり、エラーを検索にかけてみたりしたのですが、答えにたどり着けませんでした。間違っている箇所わかる方いらっしゃいましたらご教示いただきたいです。よろしくお願いします。

Comments

[phper.k]

エラーメッセージの行番号とその行がどこなのかわかるように質問を書いてください。

Answer 1

$user->productsで取得できるものは、Productモデルではなく、Productモデルが格納されたCollectionなんじゃないかと思います。

そのため、「collectionインスタンスにはallergyというプロパティは存在しません」と言われてしまってるのではないでしょうか？？><

Property [allergy] does not exist on this collection instance.

リレーション部分の記載がないですが、おそらくuserとproductは一対多の関係ですかね？
そうであれば、allergyには例えば以下のようにアクセスできるかと思います^^

....
foreach($user->products as $product)
{
    $allergy = $product->allergy;
    ....
}

追記

あと記載していただいた内容でしたら、allergyカラム内を配列に整形する処理は、modelに書くのはいかがでしょうか。

product.php

class Product extends Model
{
    public function getAllergyArrayAttribute(): array
    {
        return explode(',', $this->allergy);
    }
}

bladeファイル

@foreach($user->products as $pro)
    @foreach($pro->allergy_array as $allergy)
        {{ $allergy }}
    @endforeach
@endforeach

上記はLaravelのEloquentのアクセサという機能を使ってのサンプルです。実際の環境や用途に合わせて試してください^^ (markdown使えず汚くすみません)

Comments

[toyop]

kawa_kawawakaさん
いつもアドバイスありがとうございます。
ご指摘通り、userとproductは一対多の関係になっておりまして（情報不足で申し訳ございません）、
@foreach($user->products as $pro)
{{ $pro->allergy }}
@endforeach
とするとallergyのカラムに格納されている文字列が表示されます。ただこの文字列を配列に変換したいので、Controllerの方でexplodeメソッドを使っています。そのため、文字列を配列に変換した（と自分では思っている）変数、$allergy_arrayをviewに送りたいのですが、そこで上記のエラーが表示されてしまい、つまずいてしまっています。
長文になってしまいすみません。もしよかったらアドバイスいただけると嬉しいです。よろしくお願いします。

[kawa_kawawaka]

collectionインスタンスに存在しないallergyというプロパティにアクセスしようとしているのが、今回のエラーの原因だと思いますので、そうしなければエラーは出ないと思いますー。

(ほぼ同じ回答になりますが、何をアドバイスすればい以下わからず＞＜)

[toyop]

そうなんですよね...。
最初にいただいたアドバイスをもとにもうちょっと頑張ってみます！

[kawa_kawawaka]

どう実装したいのかがわからないので、アドバイスのしようがないという状態です＞＜

ちなみにですが
`$allergy = $user->products->allergy->get();`
はどういうデータを取得できる想定でしたか？

[toyop]

お時間割いていただき本当に恐縮です。
実装としては各ユーザーがマイページから料理と、その料理に含まれるアレルギー物質をチェックボックスでDBに登録でき、その内容を、http://localhost:8000/new/{user}に表示するというものになります。
`$allergy = $user->products->allergy->get();`ではproductsテーブル内のallergyカラム内の文字列を取得しているつもりです。
viewの方で$user->products->allergyでproductsテーブル内のallergyカラム内の文字列を取得できているので同じようにすればよいのかなと考えた次第です。

[kawa_kawawaka]

「viewの方で$user->products->allergyでproductsテーブル内のallergyカラム内の文字列を取得できている」
本当にallergyカラム内の文字列を取得できますか？取得できているとして、それは1つのproductに紐づくallergyカラムでしょうか、複数のproductに紐づくallergyカラムのかたまりでしょうか？？

[kawa_kawawaka]

あと記載していただいた内容でしたら、allergyカラム内を配列に整形する処理は、modelに書くのはいかがでしょうか。

product.php
```
class Product extends Model
{
public function getAllergyArrayAttribute(): array
{
return explode(',', $this->allergy);
}
}
```

bladeファイル
```
@foreach($user->products as $pro)
@foreach($pro->allergy_array as $allergy)
{{ $allergy }}
@endforeach
@endforeach
```

上記はLaravelのEloquentのアクセサという機能を使ってのサンプルです。実際の環境や用途に合わせて試してください^^ (markdown使えず汚くすみません)

[toyop]

kawa_kawawakaさん
アドバイスいただいた通り、コントローラーに記述していた内容をモデルに記述しましたら、実現できました！ありがとうございます！
collection instaceについてはもっと勉強したいと思います。
拙い質問にお付き合いいただいて重ね重ね感謝申し上げます。

[kawa_kawawaka]

よかったです!!!^^

取得できるデータがどういうデータなのか確認しながら進めてみると解決に近付いたかもしれないですね^^
Laravelだとdd(),dump()といったデバックに使える関数があるので、変数にどんな値が入ってるのか試してみるといいかもです！！！！