phpのバージョンの違いのエラー

質問

すみません。
ページの前後を簡単に行き来するためにぺ－ジ名をpage40みたいにして
＜のリンクが押されれば
40-1=page39
>のリンクが押されれば
40+ 1=page41
のようにしたのですが、バージョンを8.0にしたら、以下のようなエラーが出ました。エラーの内容自体は、型が異なるので計算できない？ですよね？
どう修正すればいいのかわからないので教えてください。

Fatal error: Uncaught TypeError: Unsupported operand types: string - int in

コード
<div class="pre-cntrl">
<?php
$url = rtrim($_SERVER["REQUEST_URI"], '/');
$uri = substr($url,-2);
if(ctype_digit($uri))
{
$pre =$uri-1;
echo '<a href="page' . $pre . '"><i class="fas fa-chevron-left"></i></a><br />';
}
else {
$uri = substr($url,-1);
$pre=$uri-1;
echo '<a href="page' . $pre . '"><i class="fas fa-chevron-left"></i></a><br />';
}
?>
</div>

回答No.2

AsarKingChang · ベストアンサー

basename()
https://www.php.net/manual/ja/function.basename.php

$page=intval(substr(basename($_SERVER["REQUEST_URI"]),4));
if ($page===0) // error
else
{
echo '<a href="page' . ($page-1) . '">... 前のページ
echo '<a href="page' . ($page+1) . '">... 次のページ
}

私ならこんな感じ。
最初にbasenameで、アドレスの「最後の/以降を取り出す」
先頭から4文字削除したものを「数字が入った文字列として取り出す」
それを、文字列から数値に変換。
0が帰ってきたら、不正なリクエストだったとして、エラー処理
それ以外なら、ページ制御に回す。

お礼

サンプルまでありがとうございました。

回答No.1

AsarKingChang

まず、きになったことが。。
substrで-2で取り込んでますが、これ逆に
先頭から4文字目の方がよくないですか？
先頭が必ず"page"だとわかっているわけですから、
strlen("page")=4.... しかし、実質固定なので、
substr($uri,4);　4文字目以降全部を取り出す。
ちなみ、変数名がurl になってますが、REQUEST_URIは
URLではなくURIが帰ります。
取り出したら、それはまだ「文字」なので、
https://www.php.net/manual/ja/function.intval.php
こいつでINT（数値）に変換します。
それに対して加減算をすれば、OKかと思います。

もし、パラメタが、"pageABCD"のように
数字に変換できない物が使用されている場合、
intvalは0を返すので、
0が取り込まれた時点で異常と判断
（0ページは普通は、元々ないので、それを基準にする）

if(ctype_digit($uri))　これは不要。
intvalが数字に変換できないことを0で返すので、
変換とチェックが同時に行えるので、綺麗になります。

また、1桁2桁わけもいらないので、ソース量も半分になり、
バグの発生率も減るので、どうでしょうか？

PHP8から型のチェックが厳重になってるので、その辺で
引っかかったものと思われます。

ってな感じでは、答えになりますでしょうか？

お礼

理解できるまでに時間がかかると思いますが、
ありがとうございました。