laravel5.1 updateでsaveできない

Question

DBをupdateする処理を書いているのですが、最後のsaveのところでなぜか失敗します。

$test = Test::where('login_id','=',$id)->get();
$test->memo = $memo;
$test->save();

この処理を行うと

Method save does not exist.

と出てきてしまいます。ルートもpatchで登録していますし、データを送信するときにも

<input type="hidden" name="_method" value="PATCH">

と送っているので大丈夫だと思っていたのですが・・・。一体何が原因なんでしょうか？

Answer 1

get()はデフォルトでは結果が1件の場合でも「配列」を返します。get()をつかう場合はこのようになります。

$tests = Test::where('login_id', $id)->get();
foreach($tests as $test) {
    $test->memo = $memo;
    $test->save();
}

結果が1件だと分かりきっている場合はfirst()を使いましょう。

$test = Test::where('login_id', $id)->first();
$test->memo = $memo;
$test->save();

もしくはこのようにupdate()メソッドを使っても良いかもしれません。

Test::where('login_id', $id)->update(['memo' => $memo]);

Comments

[退会済みユーザー]

本当助かりました。配列に対してだとsaveメゾットはうまく起動しないんですかね？２つ目の方法も試したのですが、なぜか最終的なsqlが
`test` set `memo` = aiueo where `id` is null
となってしまい、エラーになってしまいました。modelの方で主キーはやはり登録するべきなのかもしれませんね。一番最後の方法はうまくいきました。

[natady]

>配列に対してだとsaveメゾットはうまく起動しないんですかね？　→　やっていることは$array = []; $array->save(); みたいなものなので、正常に動作しません。 >エラーになってしまいました。　→ エラーはどんなエラーが吐かれていましたか？

[退会済みユーザー]

2つ目の方法でupdateしようとすると、最終的なsql文が
`test` set `memo` = aiueo where `id` is null
となって主キーが空じゃないやつをupdateするという条件を勝手につけるシステムみたいです。なのでmodelの方で主キーを登録したらエラーになりませんでした。わざわざありがとうございます！

Answer 2

Method save does not exist.

エラーメッセージに書いてあるとおり、としか言いようがありませんが…
「saveメソッドが存在しない」と言われているので、$testの参照しているオブジェクトがsave()メソッドを持っているか確認してください。

例)：Test::Where()で想定と違うクラスのインスタンスが返っている、save()メソッドの定義で綴りを間違えている、継承しているクラスがsave()を持っていると思ったら持っていなかった、等