メールアドレスが同じか比較し、異なった場合には違うという表示をさせたい

Question

メールアドレスが同じか比較し、異なった場合には違うという表示をさせたいのですが、なかなかうまくいきません。

nameがemail1とemail2の入力フィールドがあり、
処理は下記コードでしております。

if($_POST['email2'] !== $_POST['email1'] ){
	$error = "メールアドレスが一致しません。" ;
} else {
	$error = "メールアドレスは一致しています。" ;
}

上記のコードでテストすると、初回アクセス時（最初）から
メールアドレスが一致しません
と表示されてしまっています。

ユーザーはまだ何も入力していないので、異常表示がされてはいけません。

一応、同じアドレスと異なるアドレスを入れてテストしてみると、条件通りに表示はされます。

このメールのチェック処理は最終的にはクラス化して部品として使いまわしていきたいです。

class Email{
	function collation(){
		if($_POST['email2'] !== $_POST['email1'] ){
			$error = "メールアドレスが一致しません。" ;
			return $error;
		} else {
			$error = "" ;
			return $error;
		}
	}
}

①同じアドレスだった場合、異常表示することは何もないので下記のように書いてました。

if($_POST['email2'] !== $_POST['email1'] ){
    $error = "メールアドレスが一致しません。" ;
} 

すると、例外エラーが発生するのでなんでだろうと思いつつ、
falseの場合の処理（冒頭のコード）も加えると例外がなくなりました。

でも、falseの時は異常表示する必要はないので
たった数行ですがコード的にも必要ない書き方があれば教えて頂きたいです。

②問題点その２として、初めてそのページにアクセスした場合だけ、
例外のUndefined indexが出てきています。

適当に送信するボタンをクリックするとこの例外は消えてます。
その後、同じアドレスと異なるアドレスを入れてテストしてみると、条件通りに表示はされます。

例外の詳細は

email1 in C:\xampp\htdocs\xxx\register.php on line 6
email2 in C:\xampp\htdocs\xxx\register.php on line 6

で、該当箇所は

if($_POST['email2'] !== $_POST['email1'] )

です。

初回アクセス時は$_POSTを参照させなければいいと思い。

if(isset($_POST['email2']) !== isset($_POST['email1']) )

に改良しました。

これで初回アクセス時には例外エラーは表示されなくなりました。

しかし今度は、同じアドレスと異なるアドレスを入れてテストしてみると、何も表示されなくなりました。

何が原因でどうしたら解決されるでしょうか...

【追記】
こうでしょうか！？

class mail{
	function collation(){
		if(isset($_POST['email2'])){
			if($_POST['email2'] !== $_POST['email1'] ){
				$error = "メールアドレスが一致しません。" ;
				return $error;
			} else {
				$error = "" ;
				return $error;
			}
		}
	}
}

Answer 1

個人的には表示用とform用で別でメソッドを用意したほうがいいと思いますが、
このままのつくりで、送信時は必ずemail1とemail2が送られる想定の場合なら
こうかな。
class mail{
function collation(){
if(filter_input(INPUT_POST, 'email2') !== filter_input(INPUT_POST, 'email1')){
$error = "メールアドレスが一致しません。" ;
}
}
}

Comments

[earnest_gay]

回答ありがとうございます！
つい$_POSTを使いがちですがfilter_inputも使っていきます。

Answer 2

はっきりとは書かれていませんが、書いている内容から推察すると、フォーム入力画面とPOST先が同じファイルになっているということですよね。

もしそうであれば、考えないといけないのは、

• 初回アクセスなのか、入力後のPOSTなのか

の識別かと思います。
これはいくつかやり方があると思いますが、例えばGETアクセスなのかPOSTアクセスなのかでも見分けがつくかと思います。

ただ、仮にそういった見分け方を実装したとしても、エラーチェックとしてはご質問の問題にはぶつかりますね。
ということで本題です。

---

初回アクセス時は$_POSTを参照させなければいいと思い。

if(isset($_POST['email2']) !== isset($_POST['email1']) )

に改良しました。

発想は近くまで行っていますがやっていることが間違ったようです。

まず isset() 関数 は「変数がセットされていること、そして NULL でないことを検査する」ということで、isset($_POST['email2']) も isset($_POST['email1']) も true か false を意味しています。

つまり $_POST['email2'] と $_POST['email2'] を比較したことにはなりません。

• $_POST['email2']、$_POST['email2'] がセットされているか
• セットされている場合、一致をチェック

となります。

つまり

if( isset( $_POST['email1'], $_POST['email2']) )
{
    if( $_POST['email2'] !== $_POST['email1'] )
    {
        $error = "メールアドレスが一致しません。" ;
    }
}

また、こういった部品的なチェックツールを汎用化するのであれば、これ自体は評価結果のみを返すようにして、その結果に応じてこの部品を使うソース側でメッセージの表示をコントロールする方がいいような気もします。

// ※動作テストしていないのでどこか間違ってるかもしれませんが、流れとして。

class inputCheck
{
    static function email_collation($a, $b)
    {
        if( isset( $a, $b )
        {
            return ( $a == $b );
        } else {
            return false;
        }
    }
}

// ---- 以下が利用側ソースコード

if( isset( $_POST['email1'], $_POST['email2'] ) )
{
    if( ! inputCheck::email_collation($_POST['email1'], $_POST['email2']) )
    {
        $error = "メールアドレスが一致しません。" ;
    }
}

こんな感じではいかがでしょう。