質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • PHP

    24093questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • データベース

    838questions

    データベースとは、データの集合体を指します。また、そのデータの集合体の共用を可能にするシステムの意味を含めます

予約システムにおいて、データベースへの更新がされません。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 519

Tsukumo_Aoi

score 13

前提・実現したいこと

体育館の予約システムを組んでいます。プログラム自体に問題はなく、エラーも発生しないのですが、なぜかデータベースに予約内容が追加されません。

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

エラーは発生していません。
データベースへの予約内容の追加がされません。

該当のソースコード

<html>
<head>
  <title>reserve_scr</title>

</head>
  <body>
  <?php

  if (empty ($_GET['reserveID'])) {
    print"予約内容を入力してください<br>\n";
    disp_form();
  }
  else {
    tsuika($_GET['reserveID'],$_GET['date'],$_GET['jikan'],$_GET['place']);
  }

  function disp_form() {
    $pdo = new PDO('mysql:host=localhost;
    dbname=taiikukandb;charset=utf8','dbuser','abc123');
    $pdo -> setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
    $sql = "SELECT max(reserveID) as new_reserve_id FROM `yoyaku_tbl`";
    $stmt = $pdo -> query($sql);
     $row = $stmt -> fetch(PDO::FETCH_ASSOC);
       $reserveID = htmlspecialchars($row["new_reserve_id"] +1);



    print <<< FRM1
    <form action="./reserve_scr.php" method="get">
    <table border=0">
    <tr>
       <td>予約番号</td>
       <td>
      {$reserveID}

    </td></tr>

    <tr>
       <td>日付</td>
       <td><input type="text" name="date">(半角8桁 例:20100509)</td></tr>

    <tr>
       <td>時間帯</td>
       <td><select name="jikan">
       <option value="gozen">午前</option>
       <option value="gogo">午後</option>
       <option value="yakan">夜間</option></select></td></tr>

    <tr>
       <td>使用場所</td>
       <td><select name="place">
       <option value="main">メインアリーナ</option>
       <option value="sub">サブアリーナ</option>
       <option value="kakugi">格技場</option>
       <option value="Aroom">控え室A</option>
       <option value="Broom">控え室B</option>
       <option value="Croom">控え室C</option>
       <option value="Droom">控え室D</option></select></td></tr>
</table>
<br>
<input type="submit" value="予約">
</form>
FRM1;

}


//追加処理
function tsuika($reserveID,$date,$jikan,$place) {

  try{
    $pdo = new PDO('mysql:host=localhost;
    dbname=taiikukandb;charset=utf8','dbuser','abc123');
    $pdo -> setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
    $pdo -> setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
    $pdo -> beginTransaction();

    $query="insert into yoyaku_tbl(reserveID,date,jikan,place)
           values(:reserveID,:date,:jikan,:place)";

    $stmt = $pdo -> prepare($query);

    $stmt -> bindValue(':reserveID',$reserveID,PDO::PARAM_INT);
    $stmt -> bindValue(':date',$date,PDO::PARAM_STR);
    $stmt -> bindValue(':jikan',$jikan,PDO::PARAM_STR);
    $stmt -> bindValue(':place',$place,PDO::PARAM_STR);

    $stmt -> execute();
    $pdo -> commit();

 } catch(PDOException $Exeption) {
   $pdo-> rollback();
   die('DBエラー:'.$Exeption -> getMessage());
 }
print <<< FRM2
<br>
<form action="./reserve_scr.php" method="get">
<input type="submit" value="もう一回!">
</form>
FRM2;

}
  ?>
</body>
</html>

試したこと

データベースへのアクセス関連を見直しましたが、僕の知識ではどこが変なのか見当がつきません。

補足情報(言語/FW/ツール等のバージョンなど)

PHPを使っています。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+2

追加したときにtuika関数は呼ばれていますか?
コードの先頭で、$_GET['reserveID']をしていますが、disp_form関数で出力されるHTMLに、name="reserveID"が見当たらないので呼ばれていないのではないかと思います。
データベースから取得した値を格納している$reserveIDがHTMLのタグごと入っているのであれば問題はないですが、まさかそんな作りではないでしょうし。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/06/30 12:02

    今記載して試してみましたが、それでも更新されてません...

    キャンセル

  • 2017/06/30 12:03

    まずは、更新されているか否かではなく、tuika関数が呼ばれているかどうかを確認しましょう。
    確認しました?

    キャンセル

  • 2017/06/30 12:17 編集

    関数自体は呼ばれてるようなんです。
    今少し手をくわえたのですが、
    ```
    <tr>
    <td>予約番号</td>
    <td><input type="hidden" name="reserveID" value=".'$reserveID'.">
    {$reserveID}</td>
    </tr>
    ```
    としたところ、
    DBエラー:SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '0' for key 'PRIMARY'

    と出てしまいました。

    キャンセル

  • 2017/06/30 13:10

    書き型が悪いですね。
    この書き方だと、例えば$reserveIDに"123"が入っていたとすると、value部分は次のように展開されます。
    value=".'123'."
    なので、こう書きましょう。
    value="{$reserveID}">{$reserveID}</td>

    DBエラーは恐らくキー重複だと思われるので、$reserveIDが正しくtuika関数まで伝わるようになれば解決すると思います。
    現状INT型を想定しているのに、.'123’. という値をパラメーターとして使った結果、正しく数値化できずに0に変換され、既に登録済みのレコードとキーが重複したためエラーになったものと思われます。

    キャンセル

+2

フォームで "reserveID" を送っていないので、if (empty ($_GET['reserveID'])) の判定が常に true になり、常に表示処理しか起きないようです。

コードを書いたのに実行されない&エラーも出ないという場合は、print デバッグでよいので期待しているメソッドや判定に進んでいるかまずチェックしてみましょう。

例えばこんな感じ↓

//追加処理
function tsuika ($reserveID, $date, $jikan, $place)
{
    error_log('tuika() 呼ばれたよ');

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/06/30 12:03

    なるほど、判定チェックですね!

    ありがとうございます!

    キャンセル

+1

直接の回答とは関係ないですが、予約ID+1をhtmlspecialchars()内で計算するくらいならSQLでしてしまった方がいいと思います。new_って名称もついてますし。

SELECT max(reserveID)+1 as new_reserve_id FROM `yoyaku_tbl`


(たぶん以前の質問でそんなサンプル書いた気がします)

また登録処理などDB更新の処理はGETよりPOSTにした方がより安全です。

<form action="./reserve_scr.php" method="get"><form action="./reserve_scr.php" method="post">
$_GET ⇒ $_POST

また予約ID以外を入力チェックしてないようなので、空(カラ)の値で登録していいかどうか
項目毎に決めて必要に応じて入力チェックしておいたほうが良いでしょう。
予約システムとしては提示の項目は一通り必須でないと予約情報として成り立たなそうですし、日付も自由入力なので入力形式制限しとかないと予約情報表示のときに苦労しますし(DBの型とかも関係します)・・

もっと言えば、hiddenであってもブラウザの開発ツールで改ざん可能なので、予約IDは表示のみにとどめておきhiddenでは保存せずtuika()で再度同じ値を同じSQLで払い出した方がより安全な情報が登録できます。

更に追記ですが、予約IDが「必ず最大値+1」というルールがあるのならmysqlだと「オートインクリメント」PostgreSQLなら「シーケンス」を利用することでわざわざテーブルに対してmax+1のような計算をしなくても楽にID払出が出来るようになります(オートインクリメントだとそもそも払いださなくて良くなります)
ご検討ください。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • PHP

    24093questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • データベース

    838questions

    データベースとは、データの集合体を指します。また、そのデータの集合体の共用を可能にするシステムの意味を含めます