フォームの半角カナの変換

Question

phpでお問合せフォームを作成しています。
チュートリアルのサイトを参考に、contact.phpの中で確認画面→完了画面、送信完了までの流れはできました。

半角カナ→全角カナ変換が抜けていたので、
validation関数を使って、未入力チェックを行っている部分に
条件を追記してみましたが、確認画面で半角カナのママ表示されてしまいます。

初心者のため、そもそも追記場所があっているのか判断つきかねています。
どうぞよろしくお願いします。

###該当のソースコード

<?php
//変数の初期化
$page_flag = 0;
$clean = array();
$error = array();

// サニタイズ
if( !empty($_POST) ) {

	foreach( $_POST as $key => $value ) {
		$clean[$key] = htmlspecialchars( $value, ENT_QUOTES);
	} 
}

if( !empty($_POST['btn_confirm']) ) {

	$error = validation($clean);

	if( empty($error) ) {
		$page_flag = 1;
	}

} elseif( !empty($_POST['btn_submit']) ) {
	
	$page_flag = 2;
	
	//変数とタイムゾーンを初期化
	//以下、メール送信の記述省略

function validation($data) {
	
	$error = array();

	// バリデーション
	if( empty($data['toiawase']) ) {
		$error[] = "「お問合せ」は必ず入力してください。";
	}elseif( 400 < mb_strlen($data['toiawase']) ) {
		$error[] = "「お問合せ」は400文字以内で入力してください。";
	}
	return $error;
}
?>

###試したこと

	// バリデーション
	if( empty($data['toiawase']) ) {
		$error[] = "「お問合せ」は必ず入力してください。";
	}elseif( 400 < mb_strlen($data['toiawase']) ) {
		$error[] = "「お問合せ」は400文字以内で入力してください。";
	}else{
		$data['toiawase'] =  mb_convert_kana($data['toiawase'],"SKV");
}

###補足情報(言語/FW/ツール等のバージ
```ョンなど)
より詳細な情報

Answer 1

関数内の$dataは$error = validation($clean);で渡されている$cleanのコピーなので、
変更しても元の$cleanは変更されません。

$error = validation($clean);の前で、$cleanに対して、半角全角のコンバートをされてはどうですか？

Comments

[shi_ue]

ちなみに、validation （妥当性確認という意味）の中に、元データの変更を入れてしまうというのは、少々悪手ですね。

[OndaSeiko]

解説、ありがとうございます。
質問投稿の前に、shi-ue様の記述順も試していたのですが、$cleanに対しての記述をしていなかったので、コンバートが正しくできていなかったようです。理解が一歩進みました。

[OndaSeiko]

なるほど。元のチュートリアルにはコンバート部分がそもそもなかったので、後付けで加えたのですが、ロジックの見直しも必要ということですね。

[shi_ue]

「半角カナから全角カナへのコンバートはどんな状況でもなされる」という前提条件を付けるのでれば、データを受け取った段階でやるべきだと思います。

それから、validation (妥当性の確認）に入らないと、もしvalidationの中に、文字数ではなく半角の数（全角を二文字で考える）で判断するものがあったりすると狂いますよね？

今はご自分だけで作っておられるので何でもいいのですが、チームプレーとなるとそうもいかなくなります。
その辺りも意識したほうがいいかもしれません。

[OndaSeiko]

度々ありがとうございます。
納品前提のフォームを初めて一人で作成しており、本当にこれであっているのか、いろいろ調べていて、今回初めてteratailを利用しました。html/CSSコーダーからのphp初心者のため、ロジックが全然できていない点を踏まえて、勉強していきます。

Answer 2

shi_ue様のご指摘を元に、記述を変更し、下記の様にしました。
確認画面に遷移する時に、サニタイズした値にmb_convert_kana関数で変換している、ということだと認識しています。

if( !empty($_POST['btn_confirm']) ) {
	//半角・全角の変換
	$clean['toiawase'] =  mb_convert_kana($clean['toiawase'],"SKV");

	$error = validation($clean);

	if( empty($error) ) {
		$page_flag = 1;
	}

Answer 3

関数の引数は通常値渡しになり、関数内で変更しても呼び出し元の変数には影響がありません。
関数内で変更したい場合は、参照渡し（リファレンス渡し）にすることです。
validation関数の定義を次のようにしてください。

function validation(&$data) {

尚、全角カナ変換は問題なく出来ているようです。

関数の引数
リファレンス渡し

Comments

[OndaSeiko]

早速ありがとうございます。解消しました！
参照渡しなど、自分にとって、少し応用が入ると理解が足りていない点も明確になりました。