Undefined array key の対処法

Question

実現したいこと

エラーを解消したい。

前提

会社からの指示で気づけばプロ並みPHP改訂版という書籍を参考にしています。
業務で本書籍を読了する必要があるため、たびたび質問をさせていただくかと思います。
本書記載通りにコードを入力してもエラーばかりが出ます。
お手数お掛けして申し訳ございませんが、ご教授いただけますと幸いです。

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

Warning: Undefined array key "gazou" in C:\xampp\htdocs\product\pro_edit_check.php on line 15
商品名:スイカ
価格:898円

Warning: Trying to access array offset on value of type null in C:\xampp\htdocs\product\pro_edit_check.php on line 58
上記のように変更します。

該当のソースコード

<?php

$pro_code=$_POST['code'];
$pro_name=$_POST['name'];
$pro_price=$_POST['price'];
$pro_gazou_name_old=$_POST['gazou_name_old'];
$pro_gazou=$_FILES['gazou'];

$pro_code=htmlspecialchars($pro_code,ENT_QUOTES,'UTF-8');
$pro_name=htmlspecialchars($pro_name,ENT_QUOTES,'UTF-8');
$pro_price=htmlspecialchars($pro_price,ENT_QUOTES,'UTF-8');

if($pro_name=='')
{
	print'商品名が入力されていません。<br/>';
}
else
{
		print'商品名:';
		print$pro_name;
		print'<br/>';
}

if(preg_match('/\A[0-9]+\z/',$pro_price)==0)
{
	print'価格をきちんと入力してください。<br/>';
}
else
{
		print'価格:';
		print$pro_price;
		print'円<br/>';
}

if (isset($pro_gazou['size']) && $pro_gazou['size'] > 0)

{
	if($pro_gazou['size']>1000000)
	{
		print '画像が大き過ぎます';
	}
	else
	{
		move_uploaded_file($pro_gazou['tmp_name'],'./gazou/'.$pro_gazou['name']);
		print '<img src="./gazou/'.$pro_gazou['name'].'">';
		print '<br />';
	}
}

if($pro_name=='' || preg_match('/\A[0-9]+\z/',$pro_price)==0 || $pro_gazou['size']>1000000)
{
		print'<form>';
		print'<input type="button" onclick="history.back()" value="戻る">';
		print'</form>';
}
else
{
		print'上記のように変更します。<br/>';
		print'<form method="post" action="pro_edit_done.php">';
                print'<input type="hidden" name="code" value="'.$pro_code.'">';
		print'<input type="hidden" name="name" value="'.$pro_name.'">';
		print'<input type="hidden" name="price" value="'.$pro_price.'">';
		print'<br/>';
		print'<input type="button" onclick="history.back()" value="戻る">';
		print'<input type="submit" value="OK">';
		print'</form>';
}

?>

試したこと

$pro_gazou が設定されているか確認するためにissetを使用いたしました。

Comments

[m.ts10806]

入力フォームのコードも提示されないことにはなんとも言えません。

あと、当該書籍は名前の割に出来の悪さで有名だったりもするので、セキュリティ対応も皆無に近くあまり業務向けのスキルは身につかないと思います。

[m630]

コメントありがとうございます。
至らぬ質問で申し訳ございませんでした。

自分でも色々と調べていたので当該書籍の評判に関しても承知しております。
ただ会社指示のため、本書籍を使わないという選択ができない状態です…。

不躾なお願いではありますが、おすすめの書籍などありましたらお教えいただけますと幸いです。
この度はコメントありがとうございました。

[m.ts10806]

あまり書籍メインでやるのはよくないのと(自身も［パーフェクトPHP]くらいしかまともに使ったことがない) 、結局はどんな書籍利用してもPHPマニュアル確認して機能理解しつつ進めることになるので、まずは辞書代わりにPHPマニュアル活用できるようになるところからではないかと。
書籍丸写しだけやってても身につきませんからね。

[otn]

htmlspecialchars関数の使い方を根本的に間違っています。
書籍やサイトなど何かをこの使い方の参考にしたのなら、それは今後参考にせず、それから学んだことを全部忘れるように努めましょう。

Answer 1

データの受け渡しにはfilter_inputを利用してください
あわせてpriceのように数値型の値をバリデートなどもできます。

$code=filter_input(INPUT_POST,'code');
$name=filter_input(INPUT_POST,'name');
$price=filter_input(INPUT_POST,'price',FILTER_VALIDATE_INT);

その他、画像のチェックは

$gazou=$_FILES['gazou']??null;
if ($gazou and $gazou['error']==0){
・・・

のように行います。
また、htmlspecialcharsした値を評価してはいけません。

Answer 2

この回答はGPT-4による自動生成回答です

$pro_gazou が設定されているか確認するために isset() を使用するのは良いアプローチですが、$pro_gazou['size'] に対して isset() を使用する必要があります。以下のように修正してください。

// 修正前
if (isset($pro_gazou['size']) && $pro_gazou['size'] > 0)

// 修正後
if (isset($pro_gazou['size']) && $pro_gazou['size'] > 0)

また、エラーメッセージにあるように、$pro_gazou['size'] が未定義の場合にアクセスしようとしている箇所があるため、その部分も修正が必要です。

// 修正前
if($pro_name=='' || preg_match('/\A[0-9]+\z/',$pro_price)==0 || $pro_gazou['size']>1000000)

// 修正後
if($pro_name=='' || preg_match('/\A[0-9]+\z/',$pro_price)==0 || (isset($pro_gazou['size']) && $pro_gazou['size']>1000000))

これらの修正により、エラーが解消されるはずです。