phpの確認画面でのUndefined indexのエラーに関して

Question

php にて、入力確認画面で下記のように記述すると

$uname = htmlspecialchars($_POST["uname"], ENT_QUOTES, "UTF-8");
if ( empty($uname) ) {
	print "お名前を入力してください。";
	exit;
}

下記のエラーが表示されます

Notice: Undefined index: uname in ...

解決するために下記のような記述をしているのですが、もっとスマートに記述できないものでしょうか。

if ( isset($_POST["uname"]) ){
	$uname = htmlspecialchars($_POST["uname"], ENT_QUOTES, "UTF-8");
} else {
	$_POST["uname"] = NULL;
}
if ( empty($uname) ) {
	print "お名前を入力してください。";
}

また、現在勉強中なので、実務でこのエラーが出たことを想定して都度解決しつつやっているのですが、error_reporting(0); にしてやった方が現実的なのでしょうか。

そのあたりも教えていただけますと幸いです。よろしくお願いいたします。

Answer 1

$_GET や $_POST から値を取り出すのは（フレームワークではない）素の PHP でやるなら filter_input がオススメらしいです。

$uname = filter_input(INPUT_POST, "uname");

もしくは三項演算子でもよいと思います。

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

---

三項演算子を使う方法は、リクエストの内容によっては $_POST["uname"] に配列が入ることがあります。もしそれが問題になるのであれば、文字列にキャストしておけば良いと思います。

$uname = isset($_POST["uname"]) ? (string)$_POST["uname"] : null:

この方法だと $_POST["uname"] が配列のときに Array などという文字列に変換されてしまいますが、どうせ invalid なリクエストなのでそうなったところで困る人はいないだろうと思われます。

---

PHP7 なら次のようにより短くかけます。

$uname = $_POST["uname"] ?? (string)$_POST["uname"];

---

htmlspecialchars は値を取り出す時ではなく、値を出力するときにやるべきです。

なので、そのタイミングでは行わず、HTML の出力時に次のようにすると良いです。

<span class="uname"><?= htmlspecialchars($uname) ?></span>

htmlspecialchars のその他に引数 ENT_QUOTES UTF-8 は、PHP のバージョンによってあったほうが良かったりなくてもよかったりした気がするのですが、都度これを書くのが面倒であれば、つぎのような関数を作ると良いです。

function h()
{
    return htmlspecialchars($_POST["uname"], ENT_QUOTES, "UTF-8");
}

---

error_reporting(0); にしてやった方が現実的なのでしょうか。

非現実的です。error_reporting(0); にして開発するとまったく上達することもなく、また、多くの時間を無駄にすると思われます。

必ず error_reporting(E_ALL); とか error_reporting(-1); とかにしておくことをおすすめします。

---

もっと根本的なところとして、確認画面で次のようなエラーが出ている時点でなにかおかしいですけれども？

Notice: Undefined index: uname in ...

入力画面をスキップしたとかならともかく、入力画面→確認画面、と遷移してきてこのエラーが表示されるなら、入力画面側で何かがおかしいと思われます。

Comments

[singlestroke]

$uname = isset($_POST["uname"]) ? $_POST["uname"] : null:
は
$uname = isset($_POST["uname"]) ? $_POST["uname"] : null;
ですよね？
詳しいご回答ありがとうございます。

Answer 2

isset若しくはarray_key_existsが良く使われてます。

http://php.net/manual/ja/function.array-key-exists.php

Answer 3

こんな感じはどうでしょう

$uname = (string)filter_input(INPUT_GET, 'uname');
if ( empty($uname) ) {
print "お名前を入力してください。";
exit;
}

Answer 4

未定義変数の扱いについては、実装設計における課題ですね。

まず直接的な対応としては、こちらが参考になりそうです。公式ではこちら。

未定義変数の扱いですが、最近では下記の傾向が強くなってきたイメージがあります。
１．未定義であることで、同じ名前の変数を、別々の意図をもって使いまわし利用する。
２．未定義であることに気が付かず、ついつかってしまう。

いずれも未定義による、その処理が正しく動くかどうかも判断ができなく、予期できない動作になってしまう。

そのため、未定義に対する実装については、入念な注意と十分な設計が求められます。
業務としては、レビューで対応する場合や、コーディング規約、コーディング規約をもとにしたツールの利用、エディタの自動判定機能を駆使し、コードに対して予期しない動作が入り込まない工夫をしています。

そのため、未定義処理を入れることが問題ではなく、その影響がわからなくなることが
問題なので、その点を認識し、要件を加味した上で利用範囲を限定する程度にとどめる使い方であればあり得る実装と考えます。

Comments

[singlestroke]

参考になりました。ご回答いただいた内容は自分としてはまだまだ先のことのような気がしますが、考え方が少し分かりました。視点が全然違っていましたね。。。
ありがとうございます。