PHPとSQLをコンパクトにまとめたい

Question

前提・実現したいこと

部分一致検索して商品が見つかった場合はその商品だけを表示して、もし検索しても見つからなかったときはエラーメッセージと商品一覧を表示しています。動作上は特に問題ありませんが、コンパクトにまとめることは可能でしょうか？

エラーメッセージ

if (isset($_POST['keyword']) && mb_strlen ($_POST['keyword']) > 0) { 
        $keyword = trim($_POST['keyword']);
        try {   
            partial_match_search();
            if (count($result) > 0 ) {
                $msg[] = ($keyword . 'を含む商品が見つかりました!');
            } else {
                product_list ();
            }
        } catch (PDOException $e) {
            $err_msg[] = '商品を検索できませんでした。';
        } 
} else {
   product_list ();
}
    // 関数を定義
    function partial_match_search () {
        $sql = 'SELECT product.id, name, price, img, status, stock
                FROM product 
                JOIN item_stock
                    ON product.id = item_stock.stock_id
                WHERE name like ?';
        $stmt = $dbh->prepare($sql);
        $stmt->bindValue(1, $keyword, PDO::PARAM_STR);  
        $stmt->execute(['%' . $keyword . '%']);
        $result = $stmt->fetchALL();
        
    }

    function product_list () {
        try {
            // 商品を一覧で表示する
            $sql = 'SELECT product.id, name, price, img, status, stock
                    FROM product 
                    JOIN item_stock
                        ON product.id = item_stock.stock_id
                    WHERE status = 1';
            $stmt = $dbh->prepare($sql);
            $stmt->execute();
            $result = $stmt->fetchALL();
            $err_msg[] = ($keyword . 'を含む商品は見つかりませんでした。');
        } catch (PDOException $e) {
            $err_msg[] = '商品を取得できませんでした。';
        }
    }    

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

ソースコード

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[退会済みユーザー]

phpMyAdminはこの場では全く関係ないです、むしろデータベースの本体mysqlを質問タグにつけるべきでしょう。そして、コード部分の指定方法が間違っているので読みづらいです。 https://teratail.com/help#about-markdown

[K_3578]

質問と関係ないですけど
質問編集された際に「基本テンプレート」を押されてるので無駄に
前提・実現したいこと~補足情報のテンプレートが増えてるんで消した方が質問文の無駄が減るし
見やすくなると思います

Answer 1

構造化しましょう？
縦方向にダラダラ長いだけで可読性が低くなり、問題点の特定が難しくなります。

<?php

function search()
{
  // ～
}

function list()
{
  // ～
}

//  ユーザーが検索した場合の処理    
if (isset($_POST['keyword']) === TRUE && mb_strlen ($_POST['keyword']) > 0) { 
  // 
  search();  // 命名はあとで意味がわかるように工夫すること
}
else {
  list();
}

で、search()に該当する処理ももし長く感じるなら区切ってもいいかもしれませんし。

関数として切り離すと、変数のスコープの影響も出てくるため、
必要に応じて関数の引数として与えることも忘れずに。

あと、この箇所は私ならこう直す。

        $sql = 'SELECT product.id, name, price, img, status, stock
                FROM product 
                JOIN item_stock
                    ON product.id = item_stock.stock_id
                WHERE name like ?';

↓

        $sql = <<<EOT
SELECT product.id, name, price, img, status, stock 
FROM product 
JOIN item_stock ON product.id = item_stock.stock_id
WHERE name like ?
EOT;

ヒアドキュメント構文と言います。
変更前の$sqlには、インデントしたスペースやタブコードも混ざってしまっちゃうので。
別に混ざっていても動くことは動くんですが、全角空白の紛れ込みの予防にもなるので。

Comments

[turtle410]

if (isset($_POST['keyword']) && mb_strlen ($_POST['keyword']) > 0) {
$keyword = trim($_POST['keyword']);
try {
partial_match_search();
function partial_match_search() {
$sql = 'SELECT product.id, name, price, img, status, stock
FROM product
JOIN item_stock
ON product.id = item_stock.stock_id
WHERE name like ?';
$stmt = $dbh->prepare($sql);
$stmt->bindValue(1, $keyword, PDO::PARAM_STR); //文字列なのでSTRにかえる
// SQLを実行
$stmt->execute(['%' . $keyword . '%']);
$result = $stmt->fetchALL();
}
if (count($result) > 0 ) {
$msg[] = ($keyword . 'を含む商品が見つかりました!');
} else {
try {
product_list();
function product_list($sql, $stmt) {
// ステータスが1(公開)の販売商品のみを一覧で表示する。
$sql = 'SELECT product.id, name, price, img, status, stock
FROM product
JOIN item_stock
ON product.id = item_stock.stock_id
WHERE status = 1';
$stmt = $dbh->prepare($sql);
$stmt->execute();
//全て取得するのでfetchALLでレコードを取得する
$result = $stmt->fetchALL();
}
のような感じでしょうか？

[退会済みユーザー]

回答のコメント欄に書かれても見づらいだけなので、質問文に改善後として追記したらいいと思います。

[退会済みユーザー]

function の宣言を外出し（例えばphpコード冒頭など）すればいいんじゃないですかね。
縦方向に何画面もスクロールしなくてもいいように、いいボリュームで関数化するなどして区切りましょう、って意味です。

[turtle410]

質問文を更新しました。functionでまとめましたが、Fatal error: Call to undefined functionと表示されます。

[退会済みユーザー]

ユーザー関数の定義が、どこかのコードブロックの中（例えば if () {～} の中）にあると、そのスコープを出ちゃうと存在しないことになってしまうため、
関数の定義の箇所をもう一度確認してください。
あと、質問の元のコードは消さないほうが良かったのになぁ。一応履歴で読めるけど、めんどう。