【PHP】MySQLへのクエリが失敗する原因が不明

解決したい課題

PHPのコードを実行し、想定通りの結果が得られること。

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Document</title>
</head>
<body>
    <?php
    $DBHOST = "localhost";
    $DBUSER = "root";
    $DBPWD = "xxxxx";
    $DBNAME = "xxxxx";

    $conn = new mysqli($DBHOST, $DBUSER, $DBPWD, $DBNAME);

    if ($conn->connect_error){
        die("接続失敗:".$conn->connect_error);
    }
    
    $statement = "INSERT INTO name VALUES('$_POST[first_name]','$_POST[last_name]')";
    $conn->query($statement);
    $conn->close();


    ?>
</body>
</html>

想定結果

コードを実行するとブラウザで以下のフォームが表示される。

現状

MySQLでは以下の通りテーブルを作成しております。

PHPコードを実行するとブラウザでは以下のエラーが表示されます。

Notice: Undefined index: first_name in C:\xampp2\htdocs\index2.php on line 21

Notice: Undefined index: last_name in C:\xampp2\htdocs\index2.php on line 21

海外の教材を用いて学習を進めています。
特に不備は見当たらないため、原因が分からず困っております(´･･`)
そもそも、上記のコードでフォームが作成されるのか？と疑問に思いながら進めていますが、教材通りに進めていることは確認済みです。(「接続失敗」などのテキスト情報は一部日本語に変換してます)
どなたかアドバイスいただけますと幸いです！
よろしくお願いいたします(><)

javahack

2020/06/15 16:26

ツッコミどころがありすぎますが、まずは「クライアントから送られてきたデータをそのままセットする」SQLを記載している教材は捨てましょう。

narururu

2020/06/15 16:34 編集

ご回答ありがとうございます！javahackさん☺ 「クライアント～セットする」←セキュリティ的に大問題ということでしょうか('_')？折角購入した教材なので、も少し進めてみよかと思てるのですが(´･_･`) 因みに、問題は解決できました。

javahack

2020/06/15 16:49

kei344さんの回答のリンクにもありますが、「プリペアドステートメント」を使うようにしてください。本当はPDOに乗り換えた方が良いと思いますが、mysqliにも用意されているはずです。

narururu

2020/06/15 17:02

アドバイスありがとうございます。はい！そします。

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

回答2件

"INSERT INTO name VALUES('$_POST[first_name]','$_POST[last_name]')" が、もし、教材に書いてあるとおりなのだとしたら、別の教材を探しましょう。（PHPに良い教材があるかどうかは不明）

そもそも$_POSTの受け取りをまず勉強しましょう。

【$_GET, $_POSTなどを受け取る際の処理 - Qiita】
https://qiita.com/mpyw/items/2f9955db1c02eeef43ea

【PHPでデータベースに接続するときのまとめ - Qiita】
https://qiita.com/mpyw/items/b00b72c5c95aac573b71

投稿2020/06/15 16:33

kei344

総合スコア69407

narururu

2020/06/15 16:39 編集

いつもアドバイスありがとうございます！kei344さん☺ あらま、コードがシンプルで分かりやすいなあと思て購入した教材なのですが、何がいけないのでしょう。。。オークションサイトの仕組みを知りたいなと思て、その実装方法について説明されている教材でしたので、一旦進めようかと思てるのですが、セキュリティ的な問題であれば、実装後、FirestoreもしくはAWSにデータベースを移行しようと考えてます☺

narururu

2020/06/15 16:44

参考URLありがとうございます！やはりセキュリティ的な問題ですね。なんとなく、そんな気はしてました。しかし、オークションサイトを構築する教材でセキュリティ的な配慮がないとは、、確かにこの教材で学ぶのは辞めた方がいいのかもしれないですね。

javahack

2020/06/15 17:11

変な教材を使うよりも、リンク先の記事で学習して基礎を固めた方が良いと思います。

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

自己解決

index.htmlでフォームを記述しました。
そのフォームに名前を記入し送信すると、
xamppのデータベーステーブルに情報が格納されました。
恐らく、これを伝えたかったのかなと解釈しました。
教材でも、最終的な結果としてはテーブルに格納されていることでしたので目的は達成できました。
失礼しました。

投稿2020/06/15 16:24

narururu

総合スコア172