月別検索をしたとき、一致するデータを引っ張ってくることができません。

Question

前提・実現したいこと

プルダウンメニューから選択した「月」と合致するデータをDBから引っ張りたいです。
そのため、以下の関数を用意しました。
（プルダウンメニューから選択した「月」は、$_POST['month-selected'] になります。）

該当のソースコード

if(!empty($_POST['month-selected'])){
    $_SESSION['month-selected'] = $_POST['month-selected'];
    function getDateFromDB(){
        try{
            $dbh = dbConnect();
            $sql = 'SELECT date, menu FROM menu
                    WHERE (DATE_FORMAT(date, "%m") = ":month_selected")
                   ';
            $data = array(':month_selected' => $_SESSION['month-selected']);
            $stmt = queryPost($dbh, $sql, $data);
            $result = $stmt->fetchAll();
            debug('SQL文の結果表示：'.print_r($result, true));
            if (empty($result)){
                debug('一致する結果はありません');
            }else{
                debug('一致する結果はあります');
                return $result;
            }
        }catch (Exception $e){
            error_log('エラー表示：'.$e->getMessage());
            debug('データベースに接続できませんでした');
        }
    }
}

上記関数内で使用しているqueryPost()は以下の通りです。

function queryPost($dbh, $sql, $data){
    $stmt = $dbh->prepare($sql);
        if(!$stmt->execute($data)){
          debug('クエリに失敗しました');
          debug('失敗したSQL：'.print_r($stmt, true));
          $err_msg['common'] = MSG07;
          return 0;
        }else{
          debug('DBには接続できました');
          return $stmt;
        }
      }

<select name="month-selected">
    <option>選択してください</option>
    <?php $month = array(1,2,3,4,5,6,7,8,9,10,11,12);
        foreach ($month as $key){ ;?>
            <option value="<?php echo printf("%02d", $key); ?>"><?php echo $key; ?></option>
    <?php
        };
    ?>
</select>

以上の関数を実行すると、デバッグに「一致する結果はありません」と出てしまいます。

試したこと

具体的な数値を入れたとき、きちんと該当のデータを引っ張ってこれることを確認しました。

$sql = 'SELECT date, menu FROM menu
    WHERE (DATE_FORMAT(date, "%m") = 7)
    ';

どこを修正すれば良いのか、アドバイスいただけたら大変有難いです。

Comments

[saoyagi2]

queryPost() という関数も見せていただけますでしょうか。

[saoyagi2]

<option value="<?php echo $key; ?>" の後に '>' が抜けています。これ直したらどうなるでしょう

[shippei]

ご指摘ありがとうございます。
以下queryPost()です。
上記質問文も修正いたします。

function queryPost($dbh, $sql, $data){
$stmt = $dbh->prepare($sql);
if(!$stmt->execute($data)){
debug('クエリに失敗しました');
debug('失敗したSQL：'.print_r($stmt, true));
$err_msg['common'] = MSG07;
return 0;
}else{
debug('DBに接続できました');
return $stmt;
}
}

[shippei]

<option>の '>'抜けについてもご指摘ありがとうございます。
修正いたしましたが、検索結果は変わりません。

Answer 1

以下のように修正してみてはいかがでしょうか。

--- main.php.org	2020-07-30 10:55:54.000000000 +0900
+++ main.php	2020-07-30 10:56:13.000000000 +0900
@@ -4,9 +4,9 @@ if(!empty($_POST['month-selected'])){
         try{
             $dbh = dbConnect();
             $sql = 'SELECT date, menu FROM menu
-                    WHERE (DATE_FORMAT(date, "%m") = ":month-selected")
+                    WHERE (DATE_FORMAT(date, "%m") = ":month_selected")
                    ';
-            $data = array(':month-selected' => $_SESSION['month-selected']);
+            $data = array(':month_selected' => $_SESSION['month-selected']);
             $stmt = queryPost($dbh, $sql, $data);
             $result = $stmt->fetchAll();
             debug('SQL文の結果表示：'.print_r($result, true));

プレースホルダの名称に '-'(ハイフン)はおそらく使えません。'_'(アンダースコア）などのシンボルに使用できる記号に置き換えてみてください。

--- view.php.org	2020-07-30 10:56:41.000000000 +0900
+++ view.php	2020-07-30 10:56:57.000000000 +0900
@@ -2,7 +2,7 @@
     <option>選択してください</option>
     <?php $month = array(1,2,3,4,5,6,7,8,9,10,11,12);
         foreach ($month as $key){ ;?>
-            <option value="<?php echo $key; ?>"><?php echo $key; ?></option>
+            <option value="<?php printf("%02d", $key); ?>"><?php echo $key; ?></option>
     <?php
         };
     ?>

フォームから渡った month-selected は文字列で引き渡されます。７を選択した場合

$sql = 'SELECT date, menu FROM menu
    WHERE (DATE_FORMAT(date, "%m") = "7")
    ';

というクエリが実行されることになります。

データベース（おそらく MySQL）は、型が違う場合は自動的に型変換をして比較を行ってくれるのですが、型が同じ場合はそのまま比較します。DATE_FORMAT() の %m は月を２桁の数字で返しますので "07" ですが、例示いただいたクエリの場合は数値で渡していますので "07" = 7 という条件を 7 = 7 と変換して検索が行われます。一方、プログラムで実行した際には "07" = "7" という条件のまま検索され、一致するレコードが0件になってしまったのでしょう。
上記の修正では option value の値をゼロ詰して２桁の文字列にしてみました。おそらくこの修正をする前でも10月11月12月は正常にレコードが取得できたのではないでしょうか。

Comments

[shippei]

saoyagi2さん
ご丁寧に教えていただき、本当にありがとうございました！
　　＞＞プレースホルダの名称に '-'(ハイフン)はおそらく使えません。
→勉強不足でした。教えていただいて初めて知りました。

型に対する認識の甘さも痛感しました。大変参考になりました。
ただ、ゼロ詰めしてもやはり該当するデータを引っ張ってこれません・・・。
（debugで$_SESSION['month-selected']の数値がゼロ詰めされていることを確認しました。）

[saoyagi2]

PDOStatement::debugDumpParams でクエリの内容を確認してみるとどうでしょう。

[shippei]

お返事ありがとうございます！
不勉強でこのようにパラメータの情報を確認することを知りませんでした。
実行した結果以下の通りになります。

SQL: [136] SELECT surve_date1, menu1 FROM menu1 WHERE (DATE_FORMAT(surve_date1, "%m") = ":month_selected") Params: 1 Key: Name: [15] :month_selected paramno=-1 name=[15] ":month_selected" is_param=1 param_type=2

クエリはきちんと通っていると言ってよろしいでしょうか？
恥ずかしながらマニュアルを確認しましたが、
例えばparam_typeがパラメータのデータ型を指していることは分かりましたが、
2がどのデータ型のことかまでは分からないなど、
情報の読み取り方を分かっておらず、
もしよろしければこちらも教えていただければ大変有難いです。

[saoyagi2]

クエリはちゃんと通っているようですね。
となると、原因がわからなくなってしまいました。お役に立てず申し訳ありません。

[shippei]

とんでもないです！
回答以外にいろいろと勉強させていただけました。
答えを考えていただき、本当にありがとうございました。