preg_matchで検索し、マイナス、小数点含んでたらエラーを出す方法。

Question

前提・実現したいこと

現在、PHPで問題を解いています。
干支を返す関数を作る問題なのですが、マイナスやピリオドが含まれているときはエラーを出したいです。
本来は、西暦としてありえないマイナスや小数点が含まれていた際にエラーを出したいのですが、
現状はマイナス、小数点だけでなく西暦として成り立つ数字で入力してもエラーになってしまいます。

ご教授していただければ幸いです。

<?php
function toEtoString(int $seireki): string
{
    try {
        $list12 = array("申", "酉", "戌", "亥", "子", "丑", "寅", "卯", "辰", "巳", "午", "未");

        $num12 = $seireki - floor($seireki / 12) * 12;

        $zyunishi = $list12[$num12];
        $search = '/-|./';
        if(preg_match($search, $seireki)){
            throw new Exception('');
        }
        return (string)$zyunishi;

    } catch (Exception $e) {
        return 'error';
    }
}
print_r(toEtoString(2000));
?>

試したこと

preg_matchの部分で-(マイナス)、.（ピリオド）をそのまま一つで指定すればerrorと出る様になり、西暦として成り立つ数字でも干支をちゃんと返します。

if(preg_match('/-/', $seireki))

or

if(preg_match('/./', $seireki))

補足情報（FW/ツールのバージョンなど）

Answer 1

正規表現における.は任意の1文字を指すため、.そのものを検索したい場合はエスケープする必要があります。

if(preg_match('/-|./', $seireki))

ただ、関数の引数がint $seirekiとなっているためにキャストされるので、ピリオドが$seirekiに渡ることはないでしょう。
渡された上でエラー判定したいのであれば引数の型をmixedなどにする必要があります。

Answer 2

以下の通りで、preg_matchを使わない方針はどうでしょうか。

// マイナスはエラー吐く
if ($seireki < 0) {
    throw new Exception('');
}

// 小数点あればエラー吐く
if (is_int($seireki)) {
    throw new Exception('');
}

Comments

[itagagaki]

確かに。
ただ、is_intではなく ! is_int ですね。
でも $seireki は int なのだから is_int は必ず true かな。

Answer 3

あり得ない文字にマッチするかどうかを調べるのではなくて、数値として成り立つ数字にマッチするかどうかを調べれば良いのでは？

/^[1-9][0-9]*$/

1～9999までなら

/^[1-9][0-9]{0,3}$/

Comments

[Mason]

そちらの方法でも試してみたのですが、マイナスを含んだ際にも干支が表示されるようになってしまいます。
なので、2000では干支が返ってきて、-2000や2.000ではerrorと表示される様にしたいです。

[itagagaki]

回答に書いた正規表現では数字の連続にしかマッチしないので、それは何かが間違っています。

[itagagaki]

^ と $ を付けていなかったとか

[Mason]

再度、確認して見ましたが小数点（ピリオド）の際はerror出るのですが、-2000（マイナス）の際は干支が返ってきてしまいます。
連続した数字でも最初にマイナスから始まるのはスルーされるのかなと感じました。

[itagagaki]

echo preg_match("/^[1-9][0-9]*$/", "2000"), "\n";
echo preg_match("/^[1-9][0-9]*$/", "-2000"), "\n";
を試してみてください。

それと $seireki は toEtoString(int $seireki) として宣言されているので、
たとえ toEtoString(2000.5) と呼び出しても $seireki は 2000 となり、小数点は含まないはずなのですが。