isset()が無視される？

Question

再入力用のメールアドレスに誤りがないかをチェックするクラスがあったとします。

<?php
class Email{
	function equal($mail1,$mail2){
		if(isset($mail1,$mail2)){
			if($mail1 !== $mail2 ){
				$error = '<div class="r_em">' . "メールアドレスが一致しません" . '</div>' ;
				return $error;
			}
		}
	}
}
?>

<tr>
	<td>E-mail</td>
	<td colspan="2"><input type="email" name="email1" maxlength="50" size="46" placeholder="[e-mail]" ></td>
</tr>
<tr>
	<td>E-mail (確認用)</td>
	<td colspan="2"><input type="email" name="email2" maxlength="50" size="46" placeholder="[e-mail]" ></td>
	<td><?php echo Email::equal($_POST['email1'],$_POST['email2']); ?></td>
</tr>

色々整理してメソッドを引数で指定するような作りにしたのですが、
isset()で初回アクセス時は$_POSTを参照しないようにしてるのですが下記のように例外がでてしまいます。

Notice: Undefined index: email1 in C:\xampp\htdocs\xxx\register_ja.php on line 26

Notice: Undefined index: email2 in C:\xampp\htdocs\xxx\register_ja.php on line 26

何が原因でどうすれば解消されますか？

ちなみに引数で指定するような作りにする前の状態

function equal(){
	if(isset($_POST['email1'],$_POST['email2'])){
		if($_POST['email1'] !== $_POST['email2'] ){
			$error = '<div class="r_em">' . "メールアドレスが一致しません" . '</div>' ;
			return $error;
		}
	}
}

にしてみたら例外は表示されなくなりました。

しかし、これだと「そこ」でしか使えず別のプロジェクトへの使い回しができないので
引数で指定するような作りで例外が表示されないように解決したいのです。

別のプロジェクトでもname属性がemail1とemail2になるように注意すればいいだけかもしれませんが、
ちょっと気になったので質問してみました。

Answer 1

引数で指定するようにする前の状態のほうが自然に見えます。
$_POSTはスーパーグローバル変数なので引数で渡す必要はないかと思います。

他のプロジェクトで使い回すにしても、 Email::equal メソッドがメールに依存しているので、メールアドレスのチェック以外に利用できないです。

エラー原因についてはtest_hfさんの回答のとおりです。
issetが無視されているのではなく、isset判定してないことが原因です。

Comments

[earnest_gay]

ありがとうございます！
結局最初の状態っで使うことにしました！

Answer 2

原因はおそらくここですね↓

<td><?php echo Email::equal($_POST['email1'],$_POST['email2']); ?></td> 記載されているソースを見る限りでは定義していない$_POST['email1'],$_POST['email2']を呼んでるようにみえます。

Comments

[earnest_gay]

ありがとうございます！
結局最初の状態っで使うことにしました！

Answer 3

<td><?php echo Email::equal($_POST['email1'],$_POST['email2']); ?></td>

この段階で $_POST['email1'] が評価されているからですね。
'email1' が未定義ということでNoticeエラーが出ています。（'email2' も同様）

変更前の

if (isset($_POST['email1'], $_POST['email2'])) {
   ...
}

のように、直接 $_POST を調べるのが確実ですが、面倒くさかったら次のような関数を作って対応するのもありかな？

function params($obj, $key, $default = null) {
    if ( ! isset($obj[$key])) return $default;
    return $obj[$key];
}

<td><?php echo Email::equal(params($_POST, 'email1'), params($_POST, 'email2')); ?></td>

Comments

[earnest_gay]

ありがとうございます！