Undefined index のエラーが出たときの対処法

前提・実現したいこと

掲示板に投稿削除機能を追加したい

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

Undefined index というエラーになる。
emptyの条件分岐をどこに入れたら良いのか、分岐後の処理をどうすれば良いのかが分からない。

該当のソースコード

``PHP


### 試したこと

<html>
 <head>
  <title>3-3</title>
  <meta charset="utf-8">
 </head>	
     <body>
	<form action="mission_3-3.php" method="post">

	名前<br>
	<input type="text" name="name" value=""><br>

	<br>コメント<br>
	<textarea name="comment" cols="30" rows="8"></textarea><br>
	<input type="submit" name="submit" value="送信"><br>

	<br><br>削除対象番号:<br>
	<input type = "text" value = "" name = "delete"><br>
	<br><input type = "submit" value = "削除"><br>

	</form>

<?php
	$filename="mission_3-3.txt";

	if(!isset($_POST["name"]))
		{
		echo "名前を入力してください！";
		};
	if(!isset($_POST["comment"]))
		{
		echo "コメントを入力してください！";
		};

	
	//■ファイルがあるかどうかの確認
	$toukou_number=1;
	$name=$_POST["name"];
	$date=date("Y年m月d日 H:m:s");
	$comment=$_POST["comment"];
	$toukou=$toukou_number."<>".$name."<>".$comment."<>".$date;


	if(file_exists($filename))
		{
		$text_file = file($filename);//ファイルがあったら、ファイルを配列化する
		foreach($text_file as $value)
			{
			$last_toukou=end($value);//最後の投稿の要素を取り出す
			$divide_end_number=expode("<>",$last_toukou);//最後の投稿の要素を、さらに＜＞で分割する
			$toukou_number=$divide_end_number[0]+1;//投稿番号の部分を取り出し、投稿番号が「最後の投稿の番号＋1」の数になるように設定
			}
		}
		else
		{
		$toukou_number=1;//ファイルがなかったら「投稿番号＝1」	
		};

	

	//■投稿機能
	if(!empty($_POST["name"]) && !empty($_POST["comment"]))
			{
			foreach($text_file as $value)
				{
				$fp=fopen($filename,"a");
				fwrite($fp,$toukou."\n");
				fclose($fp);
				$a=explode("<>",$value);
				echo $a;
				};
			};
	//■削除機能
	if(isset($_POST["delete"]))
	{
		$fp_w=fopen($filename,"w");
		fclose($fp_w);
		$fp_a=fopen($filename,"a");
		$divide_toukou=explode("<>",$text_file);
		foreach($divide_toukou as $value_2)
			{
				if($divide_toukou[0]!==$_POST["delete"])
				{
				fwrite($fp_a,$toukou.\n);
				}echo $value_2;
			}
	};
		if(!empty($_POST["edit"]))
		{
		$edit = $_POST["edit"];
		$texts_edit = file($filename);
			$edit_name = "";
			$edit_comment = "";
			foreach($texts_edit as $value_edit)
				{
				$texts_edit_element = explode("<>" , $value_edit);
				if($texts_edit_element[0] == $edit)
					{
					$edit_name = $texts_edit_element[1];
					$edit_comment = $texts_edit_element[2];
					};
				};
		};
?>

</body>
</html>


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

ここにより詳細な情報を記載してください。

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

回答1件

ベストアンサー

Notice: Undefined index
が出るのは配列を参照するときにその指定したキーのデータが存在しないときなので、
例えば $_POST['name'] や $_POST['comment'] や $_POST['delete'] や $_POST['edit'] を
直接参照しようとしてそれらにデータが存在しない場合に表示されます。

$_POST以外でも、
$texts_edit_element = explode("<>" , $value_edit);の場合では
$texts_edit_element[0]があるかないか、$texts_edit_element[1]があるかないか、
なければそこで Notice: Undefined index が示されます。
$divide_toukou=explode("<>",$text_file);なども。

表示させないためには、参照する前にisset()で存在確認するのが一般的です。

$name = '';
if (isset($_POST['name'])) {
  $name = $_POST['name'];
}

explode()で文字列をデリミタで分割して配列にする場合は、
count(配列名)にて要素数が想定通りかどうかを確認するのも良いでしょう。
もっとも、phpの処理で投稿データをデリミタで連結して1行の文字列データにしてファイルに書き出しているのだから、
それを読み込んで、デリミタで分割して配列に戻したときに
数が足りないというのは処理がどこかおかしいわけで、
そのときの保存ファイルをチェックすれば原因を察することが出来ます。

投稿2019/10/12 06:44

編集2019/10/12 06:50