【Laravel】groupByを使ったcountの方法

Question

前提・実現したいこと

LaravelとPHPを勉強し始めて１０日くらいの初心者です。
DBにアクセスしてその月のチーム内(チームは1つだけ)のユーザー毎のプレイ数をカウントして
viewのテーブルに送りたいと考えていますが、思っている結果にできないので相談させてください。
理想は下記の"補足"のようにしたいのですが、数値の引き出し方法や計算法がまだ理解不足でして
どうかご教示をお願いいたします。

該当のソースコード

private function getMonthPlay(string $user_id, Team $team)
{
      $date = Carbon::now();
      $plays = Play::select('accountId', DB::raw('count(created_at) as total'))
      ->where('team_id', $team->id)
      ->whereMonth('created_at', $date->month)
      ->GROUPBY('accountId')
      ->get();

    return $plays;
}
  public function playerMember(Request $request)
  {
      
      $user = Auth::user();
      $teams = $user->teams;
      $team = $teams[0];
      $users = $team->users;

      $team_id = $team->id;
      $user_id = $user->id;

      $month_plays = $this->getMonthPlay($user_id,$team);
      return view('playerMember',compact('users','user','month_plays'));
  }

<table>
<tr>
<th>Id</th><th>play数</th>
  @foreach ($users as $user)
   <tr>
     <td>{{$user->id}}</td>
     <td>{{$month_plays}}</td>

  </tr>
  @endforeach
  </table>

Id	play数	
3	[{"accountId":1,"total":10}]	
1	[{"accountId":1,"total":10}]	
2	[{"accountId":1,"total":10}]

補足情報

Id play数
3 1
1 4
2 5

Comments

[m.ts10806]

タイトルには要件を記載してください。
https://teratail.com/help/question-tips#questionTips3-1
>ありがちな失敗としては、「助けてください」「初心者です」という表題をつけてしまうことです。

また「初心者アイコン」があるのでそちらを利用してもらえれば良いですし、
「Laravel」もタグがあるのでタイトルに含む必要はありません。

質問は編集できますので適宜ご対応ください。
あと・・・「Laravelを勉強し始めて１０日」とのことですが、PHP自体はどうでしょうか。

[m.ts10806]

あと、ちょっと細かいですが、SELECT()ではなくselect()ですね。関数は大文字小文字大別しない言語ですが、確実に書いておいたほうが良いです。

[mank]

すみません、ご指摘ありがとうございます。PHP自体も同様に10日ほどです。タイトル修正いたします。

[m.ts10806]

なるほど。では、Laravelはかなり厳しそうですね・・・。
んーちょっと悩みますね。オブジェクト指向をきちんとおさえてないといけないところはあるので。

[m.ts10806]

これも細かいのですがgrouoByではなくgroupByですね。さすがにこれを間違えると機能しません（未定義関数のエラーが出ます）

[退会済みユーザー]

コードを書く環境を整えたほうが良いのでは？ 文法エラーを指摘してくれるようなの。

[mank]

ご指摘重ねてありがとうございます、オブジェクト指向もなんとなくでしたら理解してきてはいるのですが

[m.ts10806]

10日でオブジェクト指向を何となくでも理解できるならすごいですね。私はいまだにPHPマニュアル確認しながら書いていますし。（もちろん「コピペして間違いなく機能を使うため」という要件が大きいですが）
ただ、SQLをきちんと書けないと今回の対応は一筋縄ではいけないので「こういう定義のテーブルにこういうデータが入ってきていてこういうSQLを動かしてこういう結果を得たい」という風に具体的に設計から落とし込む必要があると思います。

[mank]

ありがとうございます。理解というのは言葉が過ぎていたと思います。もう少し設計から考え直してみます。

[mank]

コードを書く環境も見直してみます。ありがとうございます。

[m.ts10806]

それも見直しは必要ですが、質問に追記してもらえれば的確なアドバイスに可能になりますので（特にテーブル定義、データ、想定しているSQL）

[hentaiman]

SQLはcreate tableもselectも打てないけどフレームワークを使ったDB使うコードは書ける、みたいな人が一時増えてましたよね。特にrails流行った時期。
当然分かる側の人に取ってはエーッ!?て思う事態だけど、分からない人はデータが多くなくたまたま負荷がかからず複雑でない処理なら対応出来てしまっているから、それが良いことなのか悪いことなのか分からない。

Answer 1

問題解決の糸口として、
Laravelのクエリビルダを使って集計処理をするのは、初学者には複雑なので、一度、素のSQL文で目的の集計処理ができるようにしてみてください。
その後、Laravelでは以下のように、SQL文とプレースホルダーを使ってSQL文を安全かつ簡単に実行できます。

$users = DB::select('select * from users where active = ?', [1]);

Laravel 5.8 データベース：利用開始
SQLクエリの実行

Comments

[mank]

返信が遅くなって申し訳ないです。アドバイスありがとうございます。今頂いたアドバイスをもとに順番に集計処理を試していっている状況です。少しずつですが雰囲気がつかめてきました！ありがとうございます！