PHP GETで飛ばす変数の中身

質問

PHP GETで飛ばす変数の中身

a.php→b.phpへ進むボタンを押したとき、
a.phpのフォーム入力でエラーがあった場合はa.phpへ戻るページを作っています。

各フォームの入力チェックは正規表現で判定し、
誤った入力がされた場合は$input2="2";
正しい入力の場合(else)は$input2="";
のような形で変数に値を代入しています。
※$input2には「何も入らない」か「2」が入るようにしているつもりです。

入力エラー時にa.phpへ戻る記述は
------------------------------------------------------------------------------
if($input1=="1"||$input2="2"||$input3="3"){
　　header("Location:a.php?SID&input1=$input1&input2=$input2&input3=$input3");
}
------------------------------------------------------------------------------
のような形にしています。（長ったらしい部分は省いています。）

ところが、エラーがないときにa.phpに戻ってb.phpに進まなくなってしまいました。
アドレスバーをみると【input2=1】となっており、なぜか$input2に「1」が入ってしまっているようです。

$input2には「何も入らない」か「2」が入るように記述しているつもりなのですが、
「1」が入ってしまう理由がよく分かりません。
エラーがない場合はb.phpへ進みたいので、「何も入らない」状態にしたいです。
なぜ「1」が入ってしまうのかお分かりの方がいらっしゃれば教えて頂けると嬉しいです。

※PHPは勉強し始めたばかりなので、不足している点などあればご指摘下さい。
よろしくお願いします。

回答No.1

yuu_x · ベストアンサー

始めたばかりということなので、

とりあえず、原因は単なるタイポ

if($input1=="1"||$input2="2"||$input3="3"){
↓
if ($input === '1' || $input2 === '2' || $input3 === '3') {

比較のつもりが代入になっています。

ただ、慣れていても誤ってしまうことがあるので、最悪の事態に備えるため、左辺と右辺を逆に書いたりもします。

if ('1' === $input || '2' === $input2 || '3' = $input3) { // 最後は誤り

このようにしておけば、もし誤って記述してしまっても、リテラルには代入できませんので、エラーメッセージで知らせてくれます。

ついでに、変数の型を意識して記述することをお勧めします。

例外もありますが、極力比較には == では無く === を、
数値を使うなら数値型で $input1 = 1; などとする。

ちなみに、提示コードの $input2 で表示された値は数値の 1 の意 ではなく、true の意です。
# true は言語構文であって値ではないので、表示されるときは 1（又は -1 など）

さらについでに、気になってしょうがないので
よくよく、変数の埋め込みに
"Location:a.php?SID&input1=$input1&input2=$input2&input3=$input3"
のような記述を見かけますが、可能な限り変数を {} で囲ったほうがいいでしょう。
"Location: a.php?SID&input1={$input1}&input2={$input2}&input3={$input3}"
又は
"Location: a.php?SID&input1=${input1}&input2=${input2}&input3=${input3}"

あ～それから、$input? が必ず数値ならいいですが、クエリ変数に渡すときは エスケープも忘れずに。SQL や、HTML など、なんでもそうですが、それらにはそれらのルールがあります。PHP のルールは通よしません。

お礼

yuu_xさん、とても丁寧な説明ありがとうございます！

PHP始めたばかりですが、こういうミスに気づかず数日間悩み続けてたりします(泣)
誤りの指摘だけでなく、色々とご説明頂けてとても助かりました。

それから、アドバイスまで頂いてありがとうございます。
・左辺と右辺を逆に書く
・比較には「===」を使う
・"Location:～内の変数は{}で囲う
はこれから実践させて頂きます！

> クエリ変数に渡すときは エスケープも忘れずに。
最後に初心者の質問で申し訳ないのですが、特殊文字のエスケープというものでしょうか？
「&」→「&」と書くとかですか？？