array_search()ができません。

Question

PHPで予約フォームを組んでいます。
同じ予約希望日でも3パターン（午前、午後１、午後２）があり、
それぞれに最大１０件の予約枠を設けています。

こちらのページでは、申込者がカレンダーの表示から
任意の日付をクリックすると、
その日の予約状況が表示される仕組みです。

日付がクリックされたら、GETで同ページに日付(年月日)を受け渡します。
PDO接続でデータベースからfetchAllした予約情報の中から、
まず同じ日付（2019-10-00）の予約がすでに入っているかどうかをしらべて、
ある場合→現在の予約状況の表示
ない場合→新規予約日の表示
というふうにif文で表示を分けようと考えました。

それで、array_search()で検索をかけたのですが、
予約が入っていない日も入っている日も、結果がbool(false)となってしまいます。
同じ日付の予約日がすでにある場合は、
配列の整数インデックスキーが返ってくると思ったんですが
使い方が間違っているでしょうか。

アドバイスいただけると助かります。

ところどころでvar_dumpしてみました。

(Apache2.4.6/PHP Version 7.2.17/MySQL5.7x)

【index.php】
// データベース接続
try {
  $pdo = new PDO(PDO_DSN_HELPFUL,DB_USER,DB_PASSWORD);
  $pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
  $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
  print '接続しました。<br />';
}catch(PDOException $e) {
  die('接続エラー：' . $e->getMessage());
}

// 予約日(reserved_date)・予約枠am,pm1,pm2(slot)・空席数(available_num)取得
// 最新の予約の受付日reception_dateから残りの空席を取得します
try {
  $sql = <<<VACANCY
  SELECT
  reserved_date,slot,available_num
  FROM
  reserv_info
  GROUP BY reserved_date,slot
  HAVING MAX(reception_date)
  ;
VACANCY;
  $stmh = $pdo->prepare($sql);
  $stmh->execute();
}catch(PDOException $e) {
  echo "エラー：" . $e->getMessage();
}

$rows = $stmh->fetchAll(PDO::FETCH_ASSOC);


foreach($rows as $row){
  echo $row["reserved_date"] . $row["slot"]. "残り空き予約枠：" . $row["available_num"] . "名<br />";
}

//カレンダーで選択された年月日
if(isset($_GET['year'])&& isset($_GET['month'])&& isset($_GET['day'])) {
  $page_flag = 1;

  $year = h($_GET['year']);
  $month = h($_GET['month']);
  $day = h($_GET['day']);
  $week = weekName($year,$month,$day); //曜日名(日〜土)を出す関数
  $preferred_date = $year . "-" .$month. "-" .$day;

  var_dump('予約希望日：'. $preferred_date);

  $keyIndex = array_column($rows, 'reserved_date'); //予約済み日付だけの配列に変換
  var_dump($keyIndex);
  var_dump(array_search('$preferred_date',$keyIndex));

}
...以下略

上の結果です。(データベースにはテスト用で３件の予約を入れてあります。)

2019-10-24午後の部① 13:00 - 14:30残り空き予約枠：6名

2019-11-11午前の部 9:00 - 10:30残り空き予約枠：3名
2019-11-13午前の部 9:00 - 10:30残り空き予約枠：8名
string(22) "予約希望日：2019-11-11" array(3) { [0]=> string(10) "2019-10-24" [1]=> string(10) "2019-11-11" [2]=> string(10) "2019-11-13" } bool(false)

Comments

[退会済みユーザー]

データベースにクエリー実行している箇所を示さないのはなぜでしょう？ データの取得時点から潜在的に問題があるような気がしてなりません。

[Masa-Y]

追加しました。お願いいたします。

Answer 1

SELECT文のクエリーで、WHERE句を使わないのはなぜでしょう？
常に予約データを全行引き出すのは無駄です。
今は数十件かもしれなくても、長期の利用で何千何万の中から加工するのは無駄な処理です。
データベース上のreserved_dateに適した形で「WHERE reserved_date = 日付文字列」みたいなのを加えるべきかと。

さらに、そのSELECT分でGROUP BY句を使っていますが、
reserved_dateとslotでユニークなデータを持っていそうな雰囲気で、
available_numが何をしているのかピンときません。
日付と枠で空き件数を足し算したいのであればSUM(available_num)した上でGROUP BY句を使うのならわかりますが。

SELECT文が妥当かどうかを第三者が判定できるよう、
CREATE TABLE文によってreserv_infoがどういうデータ構造を持っているのかをしてしてもらえれば、
より的確な回答につながるかと思います。

---

想定される予約状況管理として
空き状況をテーブルで持つ、というよりも、
予約を記録するテーブルを持つって考え方の方がしっくり来ます。
予約のデータがなければ、そこは空いていると考えられるからです。
日付、予約枠、予約顧客id
がセットであればそこそこ機能しそうな気がします。
予約を記録しつつ空き状況データを更新する、
まぁやれなくはないですが、
予約データのクエリーを工夫することで直接空き状況を取得する事もできます。
（PostgreSQL民なので具体例を示せず申し訳ないです。）

Comments

[Masa-Y]

詳しいアドバイスをありがとうございます。
WHERE句の活用がないのは確かにそうですね、
毎回すべてのデータを取得する形になっていました...。
必要最小限のデータ取得になるように
早速「WHERE reserved_date = 日付文字列」を組み込もうと思います。

今回の質問の趣旨の部分が解決できましたので、
いただいたアドバイスは今後参考にさせていただきます。
どうもありがとうございました。

Answer 2

$preferred_dateをクオテーションで括っていたことが原因でした。

Answer 3

とりあえず確認方法に問題があります。

// array_search('$preferred_date',$keyIndex)
//               ↓              ↓
   array_search( $preferred_date, $keyIndex, true )

Comments

[Masa-Y]

当初クォテーションなしでbool(false)でしたので、あとからつけてみましたが、
同じ結果でした。再度外しておきます(確認しましたがやはりbool(false)です)

[退会済みユーザー]

シングルクォーテーションで括ると、変数をそこに展開せず「$preferred_date」そのものの文字列になってしまいますよ、と言い換えれば伝わるでしょうか。

[Masa-Y]

こちらのご指摘がヒントになり、解決しました。
ありがとうございました。