DBの更新方法を教えてください。

Question

##現状・お願い
現在、laravelを使ってDBへのデータ登録・更新機能を作成しています。
データ登録はできるのですが、その後の更新ができません。

更新をするためにはどのようなコードが必要でしょうか。お教えくださいませ。

##エラーメッセージ
SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'client_id' cannot be null (SQL: update items set client_id = ?, states = ?, items.updated_at = 2019-06-19 00:47:33 where id = 9)

##試したこと
データが受け渡しされていないことが問題かと思われたため、input type="hidden"を使用して、エラーメッセージ内で?表示となっているclient_idとstatesのvalueをformの中に埋め込み、送信しました。

###editItem.blade.php

@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 action="/updateItem/{{$list->id}}" method="POST">
        {{ csrf_field() }}
          <div class="input-field col s12">
            <input id="item_name" type="text" class="validate" name="item_name" value="{{$list->item_name}}">
            <label for="item_name">案件名</label>
          </div>
          <div class="input-field col s12">
            <input id="delivery_date" type="text" class="datepicker" name="delivery_date" value="{{$list->delivery_date}}">
            <label for="delivery_date">納期</label>
          </div>
          <div class="input-field col s12">
            <input id="unit_price" type="text" class="validate" name="unit_price" value="{{$list->unit_price}}">
            <label for="unit_price">単価</label>
          </div>
          <div class="input-field col s12">
            <input id="memo" type="text" class="validate" name="memo" value="{{$list->memo}}">
            <label for="memo">備考欄</label>
          </div>
          <input type="submit" value="案件編集" class="waves-effect waves-light btn blue accent-1">
      </form>
      <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">
          <input type="hidden" name="client_id" value="{{$list->client_id}}" >
          <input type="hidden" name="states" value="{{$list->states}}" >
      </form>
      </div>
    </div>
    
@endsection

###TodoController（一部抜粋）

    // 案件情報の追加・更新・削除
    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'));
    }
    public function updateItem(Request $request){
        $item = Item::where('id', '=', $request->id)->first();
        $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('/user');
    }
    public function deleteItem(Request $request)
    {
        $client = Item::where('id', '=', $request->id)->first();
        $client->delete();

        return redirect()->back();
    }
}

###web.php

Route::get('/', 'PageController@index');
Route::get('/user', 'UserController@user');
Route::get('/addClient', 'TodoController@addClient');
Route::get('/addItem', 'TodoController@addItem');
Route::get('/editItem', 'TodoController@editItem');
Route::get('/items/{id}', 'TodoController@items');
Route::get('/invoice', 'InvoiceController@invoice');
Route::get('/editUser', 'InvoiceController@editUser');
Auth::routes();
Route::get('/home', 'HomeController@index')->name('home');

// クライアント情報の追加・更新・削除
Route::post('/addNewClient','TodoController@addNewClient');
Route::get('/editClient/{id}', 'TodoController@editClient');
Route::post('/updateClient/{id}','TodoController@updateClient');
Route::delete('/deleteClient/{id}','TodoController@deleteClient');

// 案件情報の追加・更新・削除
Route::post('addNewItem','TodoController@addNewItem');
Route::get('/editItem/{id}', 'TodoController@editItem');
Route::post('/updateItem/{id}','TodoController@updateItem');
Route::delete('/deleteItem/{id}','TodoController@deleteItem');

Answer 1

action="/updateItem/{{$list->id}}"のform内にclient_idがないからだと思います。
action="/deleteItem/{{$list->id}}"のformと同じくhiddenで置いておけば良いのではないでしょうか。

※いずれにしてもブラウザ「ソースを表示」から想定の場所に想定の値が入っているかは必ず確認してください。

Comments

[HirakuMorishima]

ありがとうございます。
その通りです。

[m.ts10806]

解決されたようでなによりです。
エラーから推測できることもあるので、必要な情報が確実に渡されているかどうかはデバッグしてくださいね

Answer 2

client_idにnullを指定できないというエラーに見えるので
データバインドがうまくできていないのでは？

Comments

[HirakuMorishima]

ご返答ありがとうございます。
確認してみます。