編集履歴

質問編集履歴

バージョン情報の詳細を記述

2020/11/12 09:24

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -100,7 +100,9 @@
 ## 環境
-Laravel6
+Laravel6.0
+(laravel6.6にアップグレードしても改善なし）

背景を修正

2020/11/12 09:24

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -284,6 +284,8 @@
+### getだとエラーなし
 ```
     $query = DB::table('posts')
@@ -310,14 +312,50 @@
+    $posts = $query->orderBy('posts.created_at', 'desc')->get();
+```
+`get()`で取得する分にはエラーなしで実行できます。
+### paginateだとエラー
+```
+    $query = DB::table('posts')
+            ->join('likes', 'posts.id', '=', 'likes.post_id')
+            ->select('*', 'count(likes.id) AS likes_count')
+            ->groupBy('posts.id');
+    if ($lgtm_min !== null) {
+      $query->having('likes_count', '>=', $lgtm_min);
+    }
+    if ($lgtm_max !== null) {
+      $query->having('likes_count', '<=', $lgtm_max);
+    }
     $posts = $query->orderBy('posts.created_at', 'desc')->paginate(20);
 ```
-しかし、
 エラー
@@ -330,6 +368,10 @@
 調べてみると、
 `withCount()`は、`having()`と`->paginate()`と併用して利用できないことが判明しました。
@@ -476,100 +518,6 @@
-    // period search
-    if ($period !== null) {
-      switch ($period) {
-        case "day":
-          $query->where([
-            ['posts.created_at', '>=', date("Y-m-d 00:00:00")],
-            ['posts.created_at', '<=', date("Y-m-d 23:59:59")]
-          ]);
-        case "week":
-          $query->where([
-            ['posts.created_at', '>=', date("Y-m-d 00:00:00", strtotime("-1 week"))],
-            ['posts.created_at', '<=', date("Y-m-d 23:59:59")]
-          ]);
-        case "month":
-          $query->where([
-            ['posts.created_at', '>=', date("Y-m-d 00:00:00", strtotime("-1 month"))],
-            ['posts.created_at', '<=', date("Y-m-d 23:59:59")]
-          ]);
-        case "period":
-          $query->where([
-            ['posts.created_at', '>=', date("{$period_start} 00:00:00")],
-            ['posts.created_at', '<=', date("{$period_end} 23:59:59")]
-          ]);
-      }
-    }
-    if ($keyword !== null) {
-      // tags search
-      if (count($tags) !== 0) {
-        $query
-          ->join('post_tags', 'posts.id', '=', 'post_tags.post_id')
-          ->join('tags', 'post_tags.tag_id', '=', 'tags.id')
-          ->whereIn('tags.name', $tags)
-          ->groupBy('posts.id')
-          ->havingRaw('count(distinct tags.id) = ?', [count($tags)]);
-      }
-      // keywords search
-      foreach ($no_tag_keywords as $no_tag_keyword) {
-        $query
-          ->where(function ($query) use ($no_tag_keyword) {
-            $query
-              ->where('posts.title', 'like', '%' . $no_tag_keyword . '%')
-              ->orWhere('posts.body', 'LIKE', "%{$no_tag_keyword}%");
-          });
-      }
-    }

モデルを追加

2020/11/12 09:22

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -106,10 +106,130 @@
 ## モデル
+### postモデル
 ```post
+<?php
+namespace App\Models;
+use Illuminate\Database\Eloquent\Model;
+class post extends Model
+{
+    protected $fillable = [
+        'title', 'body', 'user_id'
+    ];
+    public function user()
+    {
+        return $this->belongsTo('App\User');
+    }
+    public function tags()
+    {
+        return $this->belongsToMany('App\Models\tag', 'post_tags');
+    }
+    public function likes()
+    {
+        return $this->hasMany('App\Models\like');
+    }
+}
+```
+### likeモデル
+```like
+<?php
+namespace App\Models;
+use Illuminate\Database\Eloquent\Model;
+class like extends Model
+{
+    protected $fillable = [
+        'user_id', 'post_id'
+    ];
+    public function post()
+    {
+        return $this->belongsTo('App\Models\post');
+    }
+    public function user()
+    {
+        return $this->belongsTo('App\User');
+    }
+}
+```
+## マイグレーション
+```post
          Schema::create('posts', function (Blueprint $table) {
             $table->bigIncrements('id');

コメントアウトを追加

2020/11/12 08:59

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -458,10 +458,14 @@
     if ($order == 'new') {
+      //paginate()だとエラーあり。こちらもget()にすればエラーなし
       $posts = $query->orderBy('posts.created_at', 'desc')->paginate(20);
     } else {
+      //get()だとエラーなし。こちらもpaginate()にすればエラーあり
       $posts = $query->orderBy('likes_count', 'desc')->get();
     }

抜粋なしのコードを追加

2020/11/12 08:42

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -274,4 +274,206 @@
+## 抜粋なしのコード
+要望があったので、記述します。
+```index
+        $query = Post::withCount('likes');
+        $posts = DetailedSearch::DetailedSearch($query, $keyword, $request);
+        return view('posts.index', compact('posts', 'all_posts_count', 'keyword'));
+```
+```DetailedSearch
+<?php
+namespace App\Services;
+class DetailedSearch
+{
+  public static function DetailedSearch($query, $keyword, $request)
+  {
+    // values
+    $order = $request->input('order');
+    $lgtm_min = $request->input('lgtm-min');
+    $lgtm_max = $request->input('lgtm-max');
+    $period = $request->input('period');
+    $period_start = $request->input('period-start');
+    $period_end = $request->input('period-end');
+    //keyword
+    $keyword_space_half = mb_convert_kana($keyword, 's');
+    $keywords = preg_split('/[\s]+/', $keyword_space_half);
+    preg_match_all('/#([a-zA-z0-9０-９ぁ-んァ-ヶ亜-熙]+)/u', $keyword, $match);
+    $no_tag_keywords = array_diff($keywords, $match[0]);
+    $tags = $match[1];
+    //LGTM sum search
+    if ($lgtm_min !== null) {
+      $query->having('likes_count', '>=', $lgtm_min);
+    }
+    if ($lgtm_max !== null) {
+      $query->having('likes_count', '<=', $lgtm_max);
+    }
+    // period search
+    if ($period !== null) {
+      switch ($period) {
+        case "day":
+          $query->where([
+            ['posts.created_at', '>=', date("Y-m-d 00:00:00")],
+            ['posts.created_at', '<=', date("Y-m-d 23:59:59")]
+          ]);
+        case "week":
+          $query->where([
+            ['posts.created_at', '>=', date("Y-m-d 00:00:00", strtotime("-1 week"))],
+            ['posts.created_at', '<=', date("Y-m-d 23:59:59")]
+          ]);
+        case "month":
+          $query->where([
+            ['posts.created_at', '>=', date("Y-m-d 00:00:00", strtotime("-1 month"))],
+            ['posts.created_at', '<=', date("Y-m-d 23:59:59")]
+          ]);
+        case "period":
+          $query->where([
+            ['posts.created_at', '>=', date("{$period_start} 00:00:00")],
+            ['posts.created_at', '<=', date("{$period_end} 23:59:59")]
+          ]);
+      }
+    }
+    if ($keyword !== null) {
+      // tags search
+      if (count($tags) !== 0) {
+        $query
+          ->join('post_tags', 'posts.id', '=', 'post_tags.post_id')
+          ->join('tags', 'post_tags.tag_id', '=', 'tags.id')
+          ->whereIn('tags.name', $tags)
+          ->groupBy('posts.id')
+          ->havingRaw('count(distinct tags.id) = ?', [count($tags)]);
+      }
+      // keywords search
+      foreach ($no_tag_keywords as $no_tag_keyword) {
+        $query
+          ->where(function ($query) use ($no_tag_keyword) {
+            $query
+              ->where('posts.title', 'like', '%' . $no_tag_keyword . '%')
+              ->orWhere('posts.body', 'LIKE', "%{$no_tag_keyword}%");
+          });
+      }
+    }
+    // search order
+    if ($order == 'new') {
+      $posts = $query->orderBy('posts.created_at', 'desc')->paginate(20);
+    } else {
+      $posts = $query->orderBy('likes_count', 'desc')->get();
+    }
+    return $posts;
+  }
+}
+```
 ご教授のほどよろしくお願い致します。

エラー文を追加

2020/11/12 08:41

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -200,6 +200,18 @@
+エラー
+```
+SQLSTATE[42S22]: Column not found: 1054 Unknown column 'likes_count' in 'having clause' (SQL: select count(*) as aggregate from `posts` where (`posts`.`title` like %100% or `posts`.`body` LIKE %100%) having `likes_count` >= 1)
+```
+調べてみると、
 `withCount()`は、`having()`と`->paginate()`と併用して利用できないことが判明しました。

`having()`を追加

2020/11/12 08:38

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -200,7 +200,7 @@
-`withCount()`は、`->paginate()`と併用して利用できないことが判明しました。
+`withCount()`は、`having()`と`->paginate()`と併用して利用できないことが判明しました。

詳細を記述

2020/11/12 08:24

投稿

gyu

スコア3

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -2,7 +2,7 @@
-下記のSQL文をクエリビルダに変換をし、投稿（posts)のいいね（likes）の総数を取得したい。
+下記のSQL文をクエリビルダに変換し、投稿（posts)のいいね（likes）の総数を取得したい。
@@ -166,6 +166,74 @@
 ```
+    $query = DB::table('posts')
+            ->join('likes', 'posts.id', '=', 'likes.post_id')
+            ->select('*', 'count(likes.id) AS likes_count')
+            ->groupBy('posts.id');
+    if ($lgtm_min !== null) {
+      $query->having('likes_count', '>=', $lgtm_min);
+    }
+    if ($lgtm_max !== null) {
+      $query->having('likes_count', '<=', $lgtm_max);
+    }
+    $posts = $query->orderBy('posts.created_at', 'desc')->paginate(20);
+```
+しかし、
+`withCount()`は、`->paginate()`と併用して利用できないことが判明しました。
+[【Laravelのissue】withCount(), having() and paginate() together not working](https://github.com/laravel/framework/issues/28476)
+そのため、`withCount()`を利用せずに`likes_count`を定義する必要が出てきました。
+そして、下記のSQL文をクエリビルダに変換しようと考えています
+```SQL
+SELECT *,count(likes.id) AS likes_count
+FROM posts
+JOIN likes
+on posts.id = likes.post_id
+GROUP BY posts.id
+```
+```
     $query = Post::withCount('likes');
@@ -190,42 +258,6 @@
-しかし、
-`withCount()`は、`->paginate()`と併用して利用できないことが判明しました。
-[【Laravelのissue】withCount(), having() and paginate() together not working](https://github.com/laravel/framework/issues/28476)
-そのため、`withCount()`を利用せずに`likes_count`を定義する必要が出てきました。
-そして、下記のSQL文をクエリビルダに変換する必要があります。
-```SQL
-SELECT *,count(likes.id) AS likes_count
-FROM posts
-JOIN likes
-on posts.id = likes.post_id
-GROUP BY posts.id
-```
 今はSQLで解決しようとしてますが、これが難しければ集計テーブルを作成し、それを`join`で結合するかも検討しています。