フォームの入力判定について

Question

勉強しながらフォームをコピペで改造しております

フォームで入力判定したいのですが、
[1]以下の文法で入力判定に誤りは無いでしょうか？
[2]$errors['password_length']はエラー数のカウントのためだけにあるのでしょうか？
なにか特別な意味があって、password_lengthについてなにか別に記述をするものなのでしょうか？この場所に一回だけ記述してるので、間違っていないのならいいんですが
[3]名前1-4はひとまとめにできるのでしょうか？全体をもっとシンプルに書く方法があればお願いします

	//パスワード
	if ($password == ''):
		$errors['password'] = "パスワードが入力されていません。";
	elseif(!preg_match('/^[0-9a-zA-Z]{5,30}$/', $_POST["password"])):
		$errors['password_length'] = "パスワードは半角英数字の5文字以上30文字以下で入力して下さい。";
	else:
		$password_hide = str_repeat('*', strlen($password));
	endif;
	
	//名前１
	if ($name00 == ''):
		$errors['name00'] = "名前１が入力されていません。";
	elseif(mb_strlen($name00)>12):
		$errors['name00_length'] = "名前１は12文字以内で入力して下さい。";
	endif;
	
	//名前2
	if ($name01 == ''):
		$errors['name01'] = "名前2が入力されていません。";
	elseif(mb_strlen($name01)>8):
		$errors['name01_length'] = "名前2は8文字以内で入力して下さい。";
	endif;
	
	//名前3
	if ($name02 == ''):
		$errors['name02'] = "名前3が入力されていません。";
	elseif(mb_strlen($name02)>8):
		$errors['name02_length'] = "名前3は8文字以内で入力して下さい。";
	endif;
	
	//名前4
	if ($kana01 == ''):
		$errors['kana01'] = "名前4が入力されていません。";
	elseif(mb_strlen($kana01)>12):
		$errors['kana01_length'] = "名前4は12文字以内で入力して下さい。";
	endif;
	
	//名前5
	if ($kana02 == ''):
		$errors['kana02'] = "名前5が入力されていません。";
	elseif(mb_strlen($kana02)>12):
		$errors['kana02_length'] = "名前5は12文字以内で入力して下さい。";
	endif;

どれかでもお答え頂ければ・・・よろしくお願いします

Comments

[kei344]

$errors['account_length'] が書かれていません。

[creative_09]

$errors['password_length']でした。すみません

[kei344]

ここは「質問への追記・修正の依頼」です。質問本文は編集することが可能なので、修正されてはいかがでしょうか。また、配布されているコードであれば出展元URLを提示されてはいかがでしょう。

[creative_09]

修正がかのうなんですね。やってみます。配布元では時間がかかるので、ここで聞いてみました

[kei344]

URLを提示することに時間がかかるとはどういうことでしょうか。

[ikedas]

ご提示のソースコードの出所を明記して下さい (ウェブページならURL、書籍なら題名とページ)。

[creative_09]

出どころを検索していますが、わからなく、分かり次第、追記します

Answer 1

どれかでもお答え頂ければ・・・よろしくお願いします

聞きたいことが別ならまた別の質問を投げるという風に一個の質問では、質問は絞った方がいいと思います。

[1]以下の文法で入力判定に誤りは無いでしょうか？

どの項目にどのようなチェックをかけたいのか分からないので答えようがないと思います。

[2]$errors['password_length']はエラー数のカウントのためだけにあるのでしょうか？

①項目名、②何のエラーか という命名っぽいので、単にパスワードの形式チェックのエラー格納用のキーだと思います。

[3]名前1-4はひとまとめにできるのでしょうか？全体をもっとシンプルに書く方法があればお願いします

パスワード以外は空文字チェックと文字数制限のチェックが同じなので、判定を使い回せます。
こんな感じでいかがでしょうか

<?php
// Formのinputのリスト
$inputList = [
	'password' => [
	    'displayName' => 'パスワード'
	],
	'name00' => [
		'displayName' => '名前１', 
		'maxLength' => 12
	],
	'name01' => [
		'displayName' => '名前2', 
		'maxLength' => 8
	],
	'name02' => [
		'displayName' => '名前3', 
		'maxLength' => 8
	],
	'kana01' => [
		'displayName' => '名前4', 
		'maxLength' => 12
	],
	'kana02' => [
		'displayName' => '名前5',
		'maxLength' => 12
	]
];

// エラー格納用のリスト
$errorList = [];
foreach($inputList as $key => $info) {
	// POST受け取り
	$$key = isset($_POST[$key]) ? $_POST[$key] : '';

	// 空文字チェック
	if (trim($$key) === '') {
		$errorList[$key.'_empty'] = $info['displayName'].'が入力されていません。';
		continue;
	}

	if ($key !== 'password') {
		// 文字数チェック
		if (mb_strlen($$key) > $info['maxLength']) {
			$errorList[$key.'_format'] = $info['displayName'].'は'.$info['maxLength'].'以内で入力してください。';
		}
	} else {
		// password形式チェック
		if (!preg_match('/^[0-9a-zA-Z]{5,30}$/', $$key)) {
			$errorList['password_format'] = "パスワードは半角英数字の5文字以上30文字以下で入力して下さい。";
		}
	}
}

// パスワード難読化
$passwordHide = preg_replace('/./', '*', $password);

Comments

[creative_09]

ありがとうございます。
まだまだ把握しきれない部分が多いのですが、こちらを参考に勉強します