PHP 文字変換について

Question

PHP mb_convert_kana()関数について質問します。
PHPでバリテーションによる型検査で以下のことを実現したいです。

（実現したいこと）
入力条件：半角・全角数字のみ
最終出力：半角数字
入力変換：全角数字を半角数字に自動変換
エラー：数字以外の文字が入力された場合、「数字入力してください」

（コード）
if ($d['xxx'] != '' and !preg_match('/^[0-9０-９]+$/', $d['xxx'])) {
$d['xxx'] = mb_convert_kana($d['xxx'], "n" );
} else {
$err[] = array('code' => '11', 'name' => 'xxx', 'msg' => '数字入力してください');
}

上記のコードだと出力エラーになります。
mb_convert_kana()関数がなく条件分岐しないコードだと出力はできます。
条件文で矛盾している部分など、ご教授いただけたら嬉しいです。よろしくお願い致します。

Comments

[maisumakun]

使用している文字コードは何でしょうか？

[yambejp]

$d['xxx']を配列にしている意味はありますか？
preg処理はutf8に限定した方が間違いないですが、環境ゆるしますか？

[mn_fliyng]

MySQL関数を使用しています。

[mn_fliyng]

必要があって$d['xxx']配列になっています。
環境OKなのでpreg処理でutf8に限定してみます。

Answer 1

preg関数とmb_convert_kana()関数を分けたらエラー出ずに出力されました。
以下のコードです。

if ($d['xxxe'] != '' and !preg_match('/^[0-9０-９]+$/', $d['xxx'])) {
   $err[] = array('code' => '11', 'name' => 'xxx', 'msg' => '数字を入力してください');
}
$d['xxx'] = mb_convert_kana($d['xxx'], "n" );

Answer 2

条件分岐せずにpreg関数とmb_convert_kana関数を分けた以下のコードで出力できました。

if ($d['xxx'] != '' and !preg_match('/^[0-9０-９]+$/', $d['xxx'])) {
		$err[] = array('code' => '11', 'name' => 'xxx', 'msg' => '数字入力してください');
	}

$d['xxx'] = mb_convert_kana($d['xxx'], "n" );

Comments

[yambejp]

これはだめ
数字以外のものが含まれているときにでも
数字の全角が半角にかわって、中途半端な変換で混乱のもと

[mn_fliyng]

ありがとうございます。preg関数とmb_convert_kana()関数を分けたらエラーでなくなりました。

[yambejp]

せめてelseに入れないと後悔するとおもいますよ
"1(半角)a２(全角)"が、バリデートに引っかかったのに
"1(半角)a2(半角)"に変換されます。
エラーのときは元データはいじらないのが原則

[mn_fliyng]

解決したのですが、自分のやり方だとすっきりしないのでyambejpさんの方法で試します。

[mn_fliyng]

ご指摘いただいた通り、このようにelseに入れてもとくにエラーは出ませんでした。ありがとうございます。

if ($d['xxx'] != '' and !preg_match('/^[0-9０-９]+$/', $d['xxx'])) {
$err[] = array('code' => '11', 'name' => 'xxx', 'msg' => '数字を入力してください');
}
else {$d['xxx'] = mb_convert_kana($d['xxx'], "n" );
}

[yambejp]

そう。elseに入れておけば「$d['xxx']がだめです」と表示したときに
もとのデータを表示することが可能です

Answer 3

!preg_match

否定がいらないのでは？

Comments

[mn_fliyng]

ありがとうございます。preg関数とmb_convert_kana()関数を分けたらエラーでなくなりました。