Laravel8 1つの検索フォームでAND検索・NOT検索を実装したが、コードを洗練させたい

Question

練習でLaravelを使い掲示板サイトを作成しており、検索フォームを以下の仕様で作りたいと考え、更に下記のコードにて実装しました。

• 1つの検索欄でAND検索、NOT検索の実装（要はGoogleのような検索機能の実装）
• 空白を認識し語句を分割する
• 2つ以上の語句があればAND検索
• -（半角ハイフン）を含む語句があればその語句をNOT検索

//掲載にあたり一部 実際のコードとの差異あり

public function search(Request $request)
    {
        // フォーム入力値の空白を認識し分割
        $words = preg_split('/[\p{Z}\p{Cc}]++/u', $request->search, 2, PREG_SPLIT_NO_EMPTY);

        // -の付く語句を検索
        $index = preg_grep('/-/', $words);

        if (empty($index)) { // -語句無ければ通常の、またはAND検索
            if (count($words) == 2) { // 語句が2つあればAND検索
                $query = Article::where('title', 'like', "%$words[0]%")->where('title', 'like', "%$words[1]%");
            } else if (count($words) == 1) {
                $query = Article::where('title', 'like', "%$words[0]%");
            } else {
                return back();
            }
        } else {
            $exclude = implode(preg_replace('/-/', '', $index));　// 除外語句の整形、格納
            $word = implode(preg_grep('/-/', $words, PREG_GREP_INVERT));　// 通常語句の格納

            $query = Article::where('title', 'like', "%$word%")->where('title', 'not like', "%$exclude%");
        }

        $article = $query->paginate(7);
        return view('layouts.result', ['article' => $article]);
    }

参考記事

• 【PHP】 空白文字で文字列を分割して検索用キーワードの配列を作る
• Laravel8で除外キーワードで検索する方法について

認識している問題点

• 語句が増える度where句をつなげる必要がある。
• 現在分割を2語句までとしているが、今後増やす場合いちいち条件分岐つくらなければならない、よってコードが長ったらしくなる。
• 仮にOR検索を含める場合、更に複雑化する。

手探りの為、メソッドの継ぎ接ぎにより検索を実現させている状態です。至らぬ点あればご指摘お願いいたします。

Answer 1

まず Article::where() は、Article::newQuery()->where() と書き換えることができます。最初に Article::newQuery() で何も条件を設定していない Builder を取得することで、検索条件が 1 個か複数かで分岐する必要がなくなります。

例)

$query = Article::newQuery();
$query->where();
$query->where(); // AND
$query->where(); // AND

その上で、個々の検索条件文字列に対して検索条件をループで設定していくと良いと思います。

$query = Article::newQuery();
foreach ($words as $word) {
    if (- がない場合) {
        $query->where();
    } else {
        // - がある場合 NOT 検索
        $query->where();
    }
}
$article = $query->paginate(7);

Comments

[Y0shito]

yh1244様の提示いただいたコードで実装出来ました。
この度はありがとうございました。

Answer 2

yh1244様の提示いただいたコードを元に、拙作ではありますが以下のコードで実装し直しました。
これにより大幅な省コード化、検索語句が増えても対応できるようになりました。

    public function search(Request $request)
    {
        $query = Article::newQuery();
        $words = preg_split('/[\p{Z}\p{Cc}]++/u', $request->search, 3, PREG_SPLIT_NO_EMPTY);

        foreach ($words as $word) {
            if (preg_match('/-/', $word) == 0) {
                $query->where('title', 'like', "%$word%");
            } else if (preg_match('/-/', $word) == 1) {
                $query->where('title', 'not like', '%' . preg_replace('/-/', '', $word) . '%');
            }
        }

        $article = $query->paginate(7);
        return view('layouts.result', ['items' => $article,]);
    }