LaravelのFormファサードにおけるselect boxでforeachを使用したい

Question

##現状・お願い
LaravelのFormファサードにおけるselect boxでforeachを使用したいと考えています。

現状、foreachとFormファサードの兼ね合わせ方が分からず、
htmlのselectboxの書き方で代替しております。

しかし、textフォームなどにLaravelのFormファサードを使用しているためか、
select boxのforeach部分（具体的には「$val->id」「$val->client_name」）でエラーが出てしまいます。

foreachをLaravelのselectにて使用するためには、どのようにコードを記述すればいいでしょうか。

##エラーメッセージ
Trying to get property ‘id’ of non-object (View: /home/ec2-user/environment/invoice_and_todo/resources/views/todo/editItem.blade.php)

###editItem.blade

@extends('layouts.layout')
@section('title','案件管理画面')
 @section('body')
    <!--サイドメニューここから-->
    <div class="row">
      <div class="col s0 l2">
        <ul id="slide-out" class="sidenav sidenav-fixed ">
          <li><a href="{{url('/user')}}">クライアント一覧<i class="material-icons left">person</i></a></li>
          <li><a href="#">案件カレンダー<i class="material-icons left">date_range</i></a></li>
          <li><a href="{{url('/addClient')}}">クライアント追加<i class="material-icons left">add</i></a></li>
          <li><a href="{{url('addItem')}}">案件追加<i class="material-icons left">add</i></a></li>
        </ul>
      </div>
    <!--サイドメニューここまで-->
      <div class="col s12 offset-l1 l8">
      <nav class="header">
          <a href="#" data-target="slide-out" class="sidenav-trigger btn-floating"><i class="medium z-depth-1 material-icons">add</i></a>
      <h2 class="center-align">案件管理画面</h2>
      </nav>
      {{Form::open(['url' => route('editItem',['id'=>$list->id]),'method'=>'POST'])}}
        {{ csrf_field() }}
          <div class="input-field col s12">
            {{Form::text('item_name', $list->item_name,['class' => 'validate', 'id' => 'item_name'])}}
            {{Form::label('item_name','案件名')}}
          </div>
          <div class="input-field col s12">
            {{Form::text('delivery_date', $list->delivery_date,['class' => 'datepicker', 'id' => 'delivery_date'])}}
            {{Form::label('delivery_date','納期')}}
          </div>
          <div class="input-field col s12">
            {{Form::text('unit_price', $list->unit_price,['class' => 'validate', 'id' => 'unit_price'])}}
            {{Form::label('unit_price','納期')}}
          </div>
          <div class="input-field col s12">
            {{Form::select('states', ['未執筆','執筆済み','納品済み', '請求済み'] , null, ['class' => 'select', 'id' => 'states'])}}
            {{Form::label('states', 'ステータス')}}
          </div>
          <div class="input-field col s12">
            <select id="client_id" class="select" name="client_id">
              <option value="" disabled selected>クライアントを選んでください。</option>
              @foreach($list as $val)
              <option value="{{$val->id}}">{{$val->client_name}}</option>
              @endforeach
            </select>
            <label for="client_id">クライアント</label>
          </div>
          <div class="input-field col s12">
            {{Form::textarea('memo', $list->memo, ['class' => 'validate', 'id' => 'memo'])}}
            {{Form::label('memo','備考欄')}}
          </div>
          {{Form::submit('案件追加', ['class' => 'waves-effect waves-light btn blue accent-1'])}}
          {{Form::hidden('client_id', $list->client_id)}}
          {{Form::hidden('states', $list->states)}}
      {{Form::close()}}
      <form method="POST" action="/deleteItem/{{$list->id}}" method="post">
          {{ method_field('delete') }}
          {{csrf_field()}}
          <input type="submit" value="案件情報削除" class="waves-effect waves-light btn red lighten-1">
      </form>
      </div>
    </div>
     @endsection

###Controller(一部抜粋)

// 案件情報の追加・更新・削除
    public function addNewItem(Request $request){
        $item = new Item();
        $item->user_id = Auth::user()->id;
        $item->client_id = $request->client_id;
        $item->item_name = $request->item_name;
        $item->delivery_date = $request->delivery_date;
        $item->unit_price = $request->unit_price;
        $item->states = $request->states;
        $item->memo = $request->memo;
        $item->save();
                 return redirect('/addItem');
    }
    public function editItem($id){
        $list = Item::find($id);
        return view('todo.editItem', compact('list'));
    }

Comments

[m.ts10806]

次の質問と何かしら関連性があるのでは？というかforeachわざわざ使う必要がないと思うのですが。どういう結果を得たいのでしょうか

Answer 1

Contorollerの方で、1レコードしか渡しておらずforeachで回すことができないということに気づきました。

そこで、Controllerとビューを以下のように修正しました。

###ビュー

          <div class="input-field col s12">
            <select id="client_id" class="select" name="client_id">
	              <option value="" disabled selected>クライアントを選んでください。</option>
	              @foreach($itemList as $val)
	              <option value="{{$val->id}}">{{$val->client_name}}</option>
	              @endforeach
	            </select>
	            <label for="client_id">クライアント</label>
          </div>

###Controller

    public function editItem($id){
        $list = Item::find($id);
        $itemList = Item::all();
        return view('todo.editItem', compact('list', 'itemList'));
    }

上記の変更でエラー画面は表示されなくなったものの、クライアントのselect boxに実際のクライアントネーム（$val->client_name、あとおそらく$val->idも）が表示されず。

どなたか解決策を知っていらっしゃる方はお願いいたします。

Comments

[m.ts10806]

この回答は解決策ではなく自身で試している途中経過ではないでしょうか？
質問に追記されたほうが良いかと思います。（ベストアンサーは解除し、回答は削除依頼をだしてください）