予約システムを作ってるんですが、例えば2021年1月31日の10:00で予約があった場合、次見る人はその部分を"disabled"で押せないようにしたいで
す。
そのために、DBに2021年1月31日の10:00のデータがあった場合"disabled"にするという関数を作ったのですが、全ての日付の10:00が"disabled"になってしまいます。　
DBに2021年1月31日の10:00のデータがあった場合その日付の10:00のみを"disabled"にする方法を教えてください。

PHP
1<?php
2
3function str($value)
4{
5  return htmlspecialchars($value, ENT_QUOTES);
6}
7
8
9$ymd = str($_GET['year']) . '-' . str($_GET['month']) . '-' . str($_GET['day']);
10
11
12session_start();
13
14
15  $_SESSION['year'] = str($_GET['year']);
16 $_SESSION['month'] = str($_GET['month']);
17   $_SESSION['day'] = str($_GET['day']);
18  $_SESSION['week'] = str($_GET['week']);
19
20
21
22$varDate = "{$_SESSION['year']}年{$_SESSION['month']}月{$_SESSION['day']}日({$_SESSION['week']})";
23 
24$_SESSION['fullname'] = htmlspecialchars($_POST['fullname'], ENT_QUOTES);
25
26   $_SESSION['email'] = htmlspecialchars($_POST['email'], ENT_QUOTES);
27
28 $_SESSION['ymdTime'] = htmlspecialchars($_POST['ymdTime'], ENT_QUOTES);
29
30 $_SESSION['contact'] = htmlspecialchars($_POST['contact'], ENT_QUOTES);
31
32 
33 $fullname = $_SESSION['fullname'];
34    $email = $_SESSION['email'];
35  $ymdTime = $_SESSION['ymdTime'];
36  $contact = $_SESSION['contact'];
37
38
39
40
41function Option($varDate, $ymdTime)
42{
43
44    $sql = "SELECT * FROM info WHERE varDate=? AND ymdTime=?";
45
46  try {
47
48   $stmt = dbc()->prepare($sql);
49
50   $stmt->bindValue(1, $varDate);
51   $stmt->bindValue(2, $ymdTime);
52
53$option = $stmt->execute();
54
55    return $option;
56  } catch (Exception $e) {
57    
58    echo $e->getMessage();
59
60    return $option;
61  }
62}
63
64$issetOptions = Option($varDate, $ymdTime);
65
66?>
67
68
69<html>
70
71
72
73<form action="" method="post>
74
75<select name="ymdTime">
76         
77        <option value="選択">選択してください</option>
78
79          <option value="<?php echo "{$ymd}-10" ?>" <?php if (!empty($issetOptions)) echo "{$_SESSION['ymdTime']} disabled" ?> >10:00~</option>
80             
81  </select>  
82 </form>         
83
84</html>

m.ts10806

2021/01/21 10:01

>disabledで非表示にする方法細かいんですが「非活性」です。

行動規範の内容に同意します

回答2件

全ての日付の10:00が"disabled"になってしまいます。　

$issetOptionsって$stmt->execute();の結果を返してるだけですよね。

PDOStatement::executeってboolしか返しませんから、SELECTしたデータは何一つ持ってません。
empty()の判定もおかしくなりませんか？

falseが返ってきていれば 変数が空であるとみなされるのは、変数が存在しない場合や変数の値が false に等しい場合ですという仕様に則りdisabledは出ないでしょうけど、SELECT実行に問題なければ、データが取れなくてもtrueが返るので、必ずdisabledが出ます。

fetchなり何なりで値を取り出して、その結果によって判定されたほうが良いです。
Exceptionの時にreturnさせてる$optionもスコープ的には未定義です。

他：
str₍₎って何のために準備したのでしょう。
直にhtmlspecialchars()書いてるところのほうが多い気がします。

ただ、「データ取得時」にしてはいけません。
「データ出力時」にしてください。

サニタイズ/入力値検証/エスケープの考え方#エスケープ

エスケープを行う場所は『PHPからPHPの外に出ていくその場所』です。
場所が離れている場合、その変数がエスケープされたか否か、HTMLエスケープしたか否か、わからなくなりがちです。
エスケープは必ず出力するそのときにその場所で行います。

あと最も気になるのは既に指摘が入ってますが、テーブル構成ですね。
全部文字列で持ってませんか？
かなり悪手だと思います。
日付はdate型、時間はtime型、日時はdatetime型、数値はinteger（桁数によるけど）型
など、適切な型にされたほうがパフォーマンスにも良い影響があります。

１度、各テーブルの関係を示す「ER図」を書いてみると良いですよ。
管理しづらさに気づけるはずです。

投稿2021/01/21 09:56

編集2021/01/21 09:59

m.ts10806

総合スコア80875

ベストアンサー

二つのアプローチがあります。

ビュー側のバックエンド面で解決する

予約が有効かどうかの比較を、日付も比較するようにして、
日付と時間、両方を併せて比較し、両方がマッチした箇所のoptionだけ、
disabledをかけるという方法です。

DBの構造から改善する

現状の、単一の予約レコードだと、ビュー側の処理が非常にめんどうになるため、
ビュー側でも扱いやすい、また拡張性を持たせたいようなテーブル構造にすべきです。

visitPlanDate（予約プランの日付レベルのテーブル）
visitPlanTime（予約プランの時間レベルのテーブル予約が有効かどうかはこのテーブルで管理）
visitReservation（予約の本体を入れる）

とテーブルを用意し、
各々をprimaryキー（要は一意のID）を使い紐付ける手法にすれば、
各々の日付、時間別の処理が格段にしやすくなります。

つまり、visitPlanDateレコードに紐づいている複数の時間レコード（visitPlanTime）があり、ユーザーが時間を選ぶと、
visitReservationでレコードされるが、visitPlanTimeと紐づけておく、
とします。
そうすれば、visitPlanDate、visitPlanTime、visitReservationが全てつながり、
各々、日付、その予約時間、予約自体別に連動した処理が可能になります。
（ビュー側の処理も含め）