PHPにて、パラメータに不正な値が入力された場合に、パラメータの値を書き換えたい。

Question

$current = 1;
$p=filter_input(INPUT_GET,"p",FILTER_VALIDATE_INT,["options"=>["min_range"=>1]]);
if($p){
 $current = $p;
}

PHPにて、ページネーションを作成しました。（※ページネーションについての質問ではないので、ページネーション部分のコードは載せておりません）http://hoge/?p=2のようにすると、2ページ目に遷移します。

上記のコードは、パラメータpに不正な値が入力されたときの処理です。pは正の整数のみを受け付けます。不正な値が入力された場合は、$pの値を1にして、後続の処理で1ページ目へと遷移するようにしており、問題なく動作しています。

しかし、現在、http://hoge/?p=aaaなどと不正な値が入力された場合、p=aaaとなっているのに、1ページ目が表示されている状態です。パラメータの値と実際に表示されているページが異なっているところに違和感を感じています。

このように不正な値が入力された場合は、パラメータの値もp=1に書き換えた上で、1ページ目へ遷移するようにしたいです。pの値を書き換える方法は何かあるでしょうか？

それとも、パラメータの値と表示されているページ数が異なるというのは、許容されるものなのでしょうか？パラメータに不正な値が入力された場合に1ページ目へ飛ばすというエラー処理は問題なく動作しているので、特にpの値を書き換える処理はなくてもよいでしょうか？

お手数ですが、ご教授のほど、よろしくお願いします。

Answer 1

このように不正な値が入力された場合は、パラメータの値もp=1に書き換えた上で、1ページ目へ遷移するようにしたいです。pの値を書き換える方法は何かあるでしょうか？

/?p=1にリダイレクトさせましょう。

Comments

[minyouyuu]

リダイレクトですか...少し勉強したことがあるような気がしますが、調べてみます！ありがとうございました。

Answer 2

パラメータを書き換えるのはあまりよろしいことではないです。
チェックして初期値設定するだけで良いかと思います。

$current = filter_input(INPUT_GET,"p",FILTER_VALIDATE_INT,["options"=>["min_range"=>1]]) ?? 1;

NULL合体演算子

パラメータとはいってもあくまで内部処理に使うだけですし、
不正なパラメータをわざわざURLから書き換えるのも「そこまでやるべきか」というところになります。
不正なパラメータをそのまま使わないようにするだけで十分にも思います。

Comments

[minyouyuu]

なるほど、やはりパラメータを書き換えるところまでは実装しなくてよく、不正な値が使われなけばOKとするのですね。ありがとうございました！NULL合体演算子も知らなかったです。便利そうですね！使ってみます。

[m.ts10806]

不正な値は勝手に弾かれるわけではないので、そういう実装にする必要があります。
ホワイトリスト、ブラックリストを覚えると良いです。(どちらが望ましいかは要件や状況次第)