PHPのコードが正しく書いてあるように見えるのに動作しない

Question

前提・実現したいこと

・書籍「いきなりはじめるPHP」に載っているプログラムを書いているのですが、
正しく書いているように見えるのにエラーメッセージが出ます。

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

コードをブラウザで表示しようとすると
Notice: Undefined index: code in C:\xampp\htdocs\phpkiso\kensaku.php on line 10
とメッセージが出ます。

該当のソースコード

<?php
$code=$_POST['code']; //←ここが10行目

$dsn='mysql:dbname=phpkiso;host=localhost';
$user='root';
$password='';
$dbh=new PDO($dsn, $user, $password);
$dbh->query('SET NAMES utf8');

$sql='SELECT*FROM anketo WHERE code='.$code;
$stmt=$dbh->prepare($sql);
$stmt->execute();


while(1){
    $rec=$stmt->fetch(PDO::FETCH_ASSOC);
    if($rec==false){
        break;
    }
    echo $rec['code'];
    echo $rec['nickname'];
    echo $rec['email'];
    echo $rec['goiken'];
    echo '<br>';
}

$dbh=null;
?>

試したこと

10行目にあたる部分を書き直してみたのですが動作しません。

補足情報（FW/ツールのバージョンなど）

かなり初歩なことをお聞きして申し訳ありません。そしてわかりにくい箇所があったらすみません…。ご回答よろしくお願いいたします。

Answer 1

正しく書いているように見えるのにエラーメッセージが出ます。

本書のコードはそもそも間違いが多いです。

<?php
$code= filter_input(INPUT_POST, 'code'); // 変更
// $code=$_POST['code']; // コメントアウト

$dsn = 'mysql:dbname=phpkiso;host=localhost;charset=utf8'; // 変更
$user = 'root';
$password = '';

$options = [PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION]; // 追加

$dbh = new PDO($dsn, $user, $password, $options);
// $dbh->query('SET NAMES utf8'); // コメントアウト不要（セキュリティ上の間違い）

$sql = 'SELECT*FROM anketo WHERE code=?';
// $sql='SELECT*FROM anketo WHERE code='.$code; // これは絶対にダメ（プレースホルダを使うこと）

$stmt = $dbh->prepare($sql);

$stmt->execute([$code]); // 変更
// $stmt->execute(); 

Comments

[退会済みユーザー]

間違いが多いのですね…。評判が良かったので買ったのですが古い本ということもあって今勉強するには少し不向きな教材になってしまっているのかもしれませんね。次は新しく評判も良い教材を使って勉強したいと思います。書いていただいたコードも参考にしてみます。ありがとうございます！

[退会済みユーザー]

まあ、その当時だったとしても、サンプルコードは素人が書いたのかと思われる酷さです。
なにせ、セキュリティ上の欠陥が多く、実用に耐えない。

[退会済みユーザー]

そうなんですね…。この本はさくっと終わらせて違う教材をやったほうがいいですね。コードの良し悪しが全くわからない初心者のため大変助かりました。ありがとうございます。

[退会済みユーザー]

そうですね。さらっとでいいと思いますよ。
くれぐれも、「気づけば…」の方もコードはひどいので買わないように笑

[退会済みユーザー]

わかりました！気を付けます笑

Answer 2

コードをブラウザで表示しようとすると

$_POSTは、その名の通り「POSTされたデータ」を取得するためのものです。

ふつうにアクセスした場合はGETで呼び出されるので、$_POSTには何も入らず、E_NOTICEが出ます。

Comments

[退会済みユーザー]

それでメッセージが出たんですね！納得できました。ご親切にありがとうございます。

Answer 3

postでcodeというパラメータを渡していない場合、$_POST["code"]を参照したときにNoticeがでます
Noticeは厳密にはエラーではないので設定をゆるくして無視するという考え方もありますが
やはりきっちり処理を入れたほうがいいでしょう

一番簡単なのはfilter_input

$code=filter_input(INPUT_POST,'code'); 

以前はissetでチェックを先にするのが主流でした

$code=isset($_POST["code"])?$_POST["code"]:null;

あまりこだわりのない人はエラーを無視する処理をしていた場合もあります

$code=@$_POST["code"];

ただしこれは汚い処理なのであまりおすすめしません

Comments

[退会済みユーザー]

filter_inputを使った書き方をすれば一番簡単ということですね。こちらのコードに差し替えてみたところNoticeがでなくなりすっきり（？）しました。ありがとうございます！

Answer 4

自己解決しました。このソースコードを書いたPHPファイルを直接ブラウザで見た時は上記のエラーメッセージが出たのですが、こちらのコードは出力用でこちらを検索するためのhtmlファイルから開いてみたところ正常に動作しました。お恥ずかしい限りです。閲覧くださった方々ありがとうございました。