Laravel foreachの中でDB操作ができない

Question

foreachの中でEloquentモデルを操作すると下記のエラーになります。

Creating default object from empty value

Userは複数のTeamに所属しており、ユーザーが削除された時に、所属チームのnumberを1減らす処理を書きたいです。

// $my_teams ユーザーが所属しているチームidのコレクション
foreach ($my_teams as $id) {
     $team = Team::find($id);
     $team->number--;  //この行がエラー
     $team->save();
     $team = null;
}

エラーはオブジェクトを初期化していない時に発生するようですが、
$team = Team::find($id); の時点で想定のオブジェクトは取得できています。

foreachの中で操作しているのが原因なのでしょうか？

Answer 1

Userは複数のTeamに所属しており、ユーザーが削除された時に、所属チームのnumberを1減らす処理を書きたいです。

teamモデルに number というプロパティが存在していることにまず違和感があります。

通常RDBを使っていれば、team モデル、Userモデルをリレーションでつなぎ、
{{ $team->users->count() }} でメンバー数は取得できるはず。

設計がまずいのでは？と推測します。

Comments

[kriht]

ありがとうございます。

リレーションメソッドを使ってメンバー数を取得できるのは承知しておりますが、ソート等の基準としてカラムが必要なので、仕方なくこのような設計にしました。

ですので、関係する全てのnumberカラムの値を変更する方法が知りたいです。

[退会済みユーザー]

> ソート等の基準としてカラムが必要
ごめんなさいよく意味がわからないです。
普通に設計してもメンバー数をキーにソートは可能なので、
わざわざカラムが必要な理由が不明です。

[退会済みユーザー]

またループの中でSQLを投げる今の形だと、チーム数＊2回のSQLを無駄に投げるのでボトルネックになりやすく悪手です。

[kriht]

ご返信ありがとうございます。
ソートに関してはorderByメソッドを想定していたので、カラムが必要だと考えていました。
$teams = Team::orderBy('number', 'asc')->paginate(10);

設計を見直してみます。ありがとうございました。