PHPで無効な数字や文字列の場合に例外処理を出したいです。

Question

前提・実現したいこと

現在PHPで問題を解いています。
十二支を文字列で返す関数を作るところまでは出来ました。
これを無効な数字や文字列の場合には例外処理を出したいです。
ご教授お願い致します。

該当のソースコード

//例外処理出したい
<?php

function eto($seireki){
    $list12 = array("申","酉","戌","亥","子","丑","寅","卯","辰","巳","午","未");
    
     $num12 = $seireki - floor(($seireki /12)) * 12;
    
     $zyunishi = $list12[$num12];

  return $zyunishi; 
}

print_r(eto(1993));
?>

//実行結果 "酉"

試したこと

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

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

        $zyunishi = $list12[$num12];
        if(is_string($zyunishi)){

            throw new Exception('無効な数字');
        }
        return $zyunishi;

    print_r(eto(1993));

    } catch (Exception $e) {

        echo "例外処理が発生しました";
        echo "\n";
        echo $e->getMessage();

    }
}
?>

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

Comments

[m.ts10806]

現状の問題は何ですか？
if(is_int($zyunishi)){
だと数値の時にthrowしてるように見えますが、
これは要件通りの実装ですか？

[Mason]

現状の問題は、何も表示されないことです。
is_stringにし整数の時にthrowしたりしてみましたがうまくじっこうできないです。
要件は、これを無効な数字や文字列の場合には例外処理を出したいです。

[m.ts10806]

＞現状の問題は、何も表示されないことです。

「試したこと」のコードだとeti()は誰からも呼ばれてませんし、
returnのあとに書いてあるprint_rは実行されることがないですし、関数名etoになってます。

[Mason]

すいません。もう少し詳しくお願い致します。

[m.ts10806]

回答にうつしました。

Answer 1

<?php
function eti($seireki) //呼び出されてない
{
    try {
        $list12 = array("申", "酉", "戌", "亥", "子", "丑", "寅", "卯", "辰", "巳", "午", "未");

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

        $zyunishi = $list12[$num12];
        if(is_string($zyunishi)){ //$zyunishiが干支返すなら必ず文字列になるのでは？

            throw new Exception('無効な数字');
        }
        return $zyunishi;

    print_r(eto(1993)); // returnで関数が終了するのでここは通らない行。それにeto()になってる

    } catch (Exception $e) {

        echo "例外処理が発生しました";
        echo "\n";
        echo $e->getMessage();

    }
}

//↓に書けばとりあえず実行はされるが
print_r(eto(1993));
?>

「何が無効なのか」をきちんと定義されたほうが良いと思います。
「文字列かどうか」をチェックしたいなら$zyunishiではなく$seirekiをチェックすべきです。
$seirekiが「年数としてありえない」なら、checkdate関数する必要があるでしょうしね。
引数の型宣言も有効かもしれません。（TypeError）

つまり、バリデーションするならもっと前。
関数の引数をチェックすること。