Q&A
オートインクリメントしてるのならnullはおろか任意のidもいれる必要はないのですけど、どのような設計思想でそのように作ってるのでしょうか
現在、PHPとMySQLで、読んできたホントその内容について参照してきたウェブサイトをまとめたCRUDアプリを創作しています。
今のところ、データベース接続とSELECTでのデータベースの全データ表示まではうまく行っているところです。以下が該当のコードです。
【config.php】 <?php /* データベース接続設定 */ $connect = mysqli_connect($dsn,$username,$password,$dbname); /* これが、list,index,edit,deleteに出てきた第一引数「$connect」の正体である */ $dsn = 'localhost'; //ここにホスト名を入力 $username = 'root'; //ここにユーザーネームを入力 $password = ''; //ここにパスワードを入力して（ない場合は入力不要） $dbname = 'crud'; //ここにデータベースサーバー名を入力 $table_1 = 'googles'; //ここにdデータベースのテーブル名を入力 ?> <?php /* Viewファイルで使う変数設定 */ $catchPhrase = ; $list_title = 'ググり本'; ?>
【list.php】 <html> <?php include("config.php"); ?> <head> <title><?= "『".$list_title."』データベース"; ?></title> <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css"> <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script> <script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/js/bootstrap.min.js"></script> <link rel="stylesheet" type="text/css" href="https://cdn.datatables.net/1.10.19/css/jquery.dataTables.min.css"> </head> <!-- このファイルがデータ一覧を表示する大元のファイルといえる --> <body> <div class="container" style=""> <p align="center"><?= $catchPhrase; ?></p> <h3 align="center"><?= "読んできた".$list_title."とリンクサイト一覧";?></h3> <a href="index.php?=add-record" class="btn btn-info"><?= $list_title."データ登録画面へ" ?></a><br><br> <div class="row"> <table id="example" class="display" style="width:100%"> <thead> <tr> <th></th> <th>タイプ</th> <th>書籍orウェブサイト</th> <th>参考ウェブサイト</th> <th></th> <th></th> </tr> </thead> <tbody> <?php include("config.php"); $query ="SELECT * FROM $table_1 ORDER BY id DESC"; $sql = mysqli_query($connect,$query); while($row = mysqli_fetch_array($sql)) { ?> <tr> <td><input type="hidden" value="<?php echo $row["id"];?>"></td> <td><?php echo $row["type"];?></td> <td><?php echo $row["text"];?></td> <td> <a href="<?php echo $row["sightUrl"];?>" target="_blank" onClick="return confirm('このページにアクセスしますか?')"> <?php echo $row["sight"];?> </a> </td> <td> <a href="edit.php?id=<?php echo $row['id']; ?>" class="btn btn-info">編集</a> </td> <td> <a href="delete.php?id=<?php echo $row['id'];?>" class="btn btn-danger" onClick="return confirm('ホントに削除しますか?')">削除</a> </td> </tr> <?php } ?> </tbody> </table> </div> </div> </body> </html> <script type="text/javascript" src="https://code.jquery.com/jquery-3.3.1.js"></script> <script type="text/javascript" src="https://cdn.datatables.net/1.10.19/js/jquery.dataTables.min.js"></script> <script type="text/javascript"> $('#example').DataTable({ "language" : {"url":"any.json"}, "initComplete" : function(settings, json) { this.xxxApi().doAnythng();//ここで処理する } }); /* $(document).ready(function() { $('#example').DataTable(); } ); */ </script>
phpMyAdminのテーブル構造と、list.phpでの現在の表示は、以下画像のとおりです。
しかし、このページの「ググり本データ登録画面」ボタンから遷移するindex.phpで新規にデータを挿入しようとしても、list.phpのページにそれを反映させることができないまま、現在に至っています。
index.phpのコードは、以下のとおりです。
【index.php】 <?php include("config.php"); if(isset($_POST['submit'])){ $id = $_POST['id']; $type = $_POST['type']; $text = $_POST['text']; $textUrl = $_POST['textUrl']; $sight = $_POST['sight']; $sightUrl = $_POST['sightUrl']; $sql = "INSERT INTO googles (id,type,text,textUrl,sight,sightUrl) VALUES (null,:type,:text,:textUrl,:sight,:sightUrl)"; $query = $dbConn->prepare($sql); // $query->bindparam(':id', $id); $query->bindparam(':type', $type); $query->bindparam(':text', $text); $query->bindparam(':textUrl', $textUrl); $query->bindparam(':sight', $sight); $query->bindparam(':sightUrl', $sightUrl); $query->execute(); $dbConn = null; header("Location:list.php"); } ?> <html> <head> <title>Add Data</title> <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css"> <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script> <script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/js/bootstrap.min.js"></script> </head> <body> <div class="container" style=""> <div class="row"> <h3>Add Book Data</h3> <div class="col-sm-6"> <a href="list.php?=data-list" class="btn btn-info"><?= "読んできた".$list_title."とリンクサイト一覧へ";?></a><br> <form method='POST' role='form' action=''> <!-- <div class='form-group'> <label for='Id'>Id</label> <input name='Id' id='Id' class='form-control' type='hidden'> </div> --> <div class='form-group'> <label for='type'>Type</label> <div id='type' class='radio'> <label><input type='radio' name='type' value='book' checked>本</label> </div> <div id='type' class='radio'> <label><input type='radio' name='type' value='net'>ウェブサイト</label> </div> </div> <div class='form-group'> <label for='text'>Text</label> <input name='text' id='text' class='form-control' type='text' placeholder='本・ウェブサイトのタイトルを入力して下さい。'> </div> <div class='form-group'> <label for='textUrl'>TextUrl</label> <input name='textUrl' id='textUrl' class='form-control' type='url' placeholder='ウェブサイトの場合はURLを入力して下さい。'> </div> <div class='form-group'> <label for='sight'>Sight</label> <input name='sight' id='sight' class='form-control' type='text' placeholder='参考サイト名を入力して下さい。'> </div> <div class='form-group'> <label for='sightUrl'>SightUrl</label> <input name='sightUrl' id='sightUrl' class='form-control' type='url' placeholder='参考サイトのURLを入力して下さい。' required > </div> <div class="form-group"> <input type="submit" name="submit" value="Add" class="btn btn-primary btn-block"> </div> </form> </div> </div> </div> </body> </html>
index.phpのどの部位が間違っていて、その記述をどのように修正すれば、list.phpに正常に遷移して、しっかりと反映されたデータがlist.phpでも表示されるようになるでしょうか。
（仮説１： if(isset($_POST['submit'])){で括ることが間違っているのか。）
（仮説２： "INSERT INTO ~"の構文が間違っていて、特にテーブル名やカラム名を``で囲っていない為に問題が起きているのか。）
尚、正常にDBにデータが反映されたことが確認できた後に、$＿POST['text']などの、テキストボックスから送信される値に「htmlspechalchars」の設定を施す予定でいます。
また初歩的な事項の確認となるかと思われますが、ご回答お願い致します。
DBのテーブル上でidのカラムがnot null制約があるのに、index.php中のINSERTでidに値をセットしていないせいではないのですか？それとも // でコメントにしているのは試行の跡でしょうか。
エラーハンドリングがないのでエラーが起きていても拾えません
「エラーハンドリング」という記述を受けて、config.phpの記述を次のように訂正しました。
<?php
/* データベース接続設定 */
$dsn = 'localhost'; //ここにホスト名を入力
$username = 'root'; //ここにユーザーネームを入力
$password = ''; //ここにパスワードを入力して（ない場合は入力不要）
$dbname = 'crud'; //ここにデータベースサーバー名を入力
try {
$dbConn = new PDO("mysql:host={$dsn};dbname={$dbname}", $username, $password);
$dbConn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
} catch(PDOException $e) {
echo $e->getMessage();
}
$table_1 = 'googles'; //ここにdデータベースのテーブル名を入力
?>
<?php
/* Viewファイルで使う変数設定 */
$list_title = 'ググり本';
?>
が、エラーハンドリングというのはこの「try {} catch(PDOException $e) {echo $e->getMessage();}」のこと（sejukuなどでの外部サイトでは「例外処理」と記述されていた）で、書き方としてはこの記述で合っていますか？
或いは、index.php（INSERTが書いてあるもの）の方にもこれを記述するべきですか？
合っていますが、接続のところは一度確立されればネットワーク障害でもない限りほとんどそこで例外は起きませんので、むしろ実行時に必須です。
回答1件
0
ベストアンサー
プライマリーキーでAuto_incrementの項目が設定されている場合のinsertは以下の2パターンしかありません。
・Auto_incrementを利用する時
insert先の項目にその項目を指定しない。
・Auto_incrementを利用せず、値を指定する時
insert先の項目にその項目を指定する。指定する値はNull以外の重複しない値
Auto_inclementは値が指定されない（NUllも含む)場合に機能します。
質問ではIDにNullを指定しているのでAuto_inclementされず、Nullが採用されますがNot Null制約でエラーです。
SQL
1INSERT INTO googles (id,type,text,textUrl,sight,sightUrl) 2VALUES (null,:type,:text,:textUrl,:sight,:sightUrl)
Auto_inclement項目を指定しないように変更して下さい。
SQL
1INSERT INTO googles (type,text,textUrl,sight,sightUrl) 2VALUES (:type,:text,:textUrl,:sight,:sightUrl)
投稿2020/10/18 03:51
総合スコア25206
返信遅くなり、失礼致しました...
googlesの後の()の中とVALUESのそれに、idとnullを指定していたのがいけなかったのですね。
idをオートインクリメントにしている時には、そのidを指定してはいけない。エラーになる。
この基本を押さえられていなかったが故に起きてしまった問題だったのですね。
LAMPの基礎事項を、もっと手頃な入門書で確実に押さえておく必要がありそうです。
あの後、お陰様でCRUDシステム自体は正常に機能させられるほどにまで実装でき、テキストボックスから入力した値のサニタイズ処理も正常に記述することができました。
これも、今回お答え頂いたおかげです。ありがとうございました。
あなたの回答
tips
プレビュー
