引数にboolの型指定を宣言した際の配列の出力方法

Question

前提・実現したいこと

現在、phpで西暦から十二支を求める問題を解いています。
漢字で返す時と、アルファベットで返す時の2パターンの文字列で返す関数を作成しました。
第二引数にはbool値の型宣言を指定し、trueの場合は漢字で返し、falseの場合は、アルファベットで返すのが条件です。
現在は、アルファベットの返り値でどんな西暦でもtoriと出力されてしまいます。（漢字は問題なく表示できています。）
なので、西暦にあったアルファベットを出力できる様にしたいです。
2000年なら、辰年なので「tatu」と出力したいです。

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

該当のソースコード

<?php
function eto(int $seireki, bool $hantei): string
{
    try {
        $kanji = array("申", "酉", "戌", "亥", "子", "丑", "寅", "卯", "辰", "巳", "午", "未");
        $alphabet = array("saru","tori","inu", "inosisi", "nezumi", "usi", "tora", "usagi", "tatu", "hebi", "uma", "hituji");


        $listKanji = $seireki - floor($seireki / 12) * 12;
        $listAlphabet = $hantei - floor($hantei / 12)*12; 

        $zyuniKanji = $kanji[$listKanji];
        $zyuniAlphabet = $alphabet[$listAlphabet];
        
        if(!preg_match("/^[1-9][0-9]{0,3}$/", $seireki)){
            echo "error";
            throw new Exception('');
        } elseif($hantei === false) { 
            return $zyuniKanji;
        } else {
            return $zyuniAlphabet;
        }
            
    } catch (Exception $e) {
        return '';
    }
}

print_r(eto(2000,true)."\n");
print_r(eto(2000,false)."\n");
print_r(eto(-2000,false)."\n");
?>

//出力
tori //tatuと出力したい
辰
error

試したこと

boolの型指定を宣言しているので、これだと、true,falseで判定するので最初の「saru」,「tori」しか出力されないのかと思いboolの型指定しないでやってみたのですがうまくいきませんでした。

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

ここにより詳細な情報を記載してください。

Answer 1

何をしたいのかよくわからない追加コードですが…

$listAlphabet = $hantei - floor($hantei / 12)*12;

ここで$hanteiがtrueなら、trueは1になりますから、この算術演算の結果は1となり$listAlphabetは1になります。

当然$zyuniAlphabet = $alphabet[$listAlphabet];はtoriになりますね。

Comments

[itagagaki]

あえて答え（修正コード）は書きません。

[Mason]

なるほど、$hanteiがfalseなら、0でsaruになるんですよね。
でしたら、bool値でtrue,falseだと西暦に合わせてアルファベットを出力（saru,tori以外）するには算術演算から見直した方がいいという認識でよろしいでしょうか？

[itagagaki]

漢字で返す場合であろうと、アルファベットで返す場合であろうと
・十二支の配列を0～11のインデックスで参照して答えを取り出せばよいという方針は同じ
・そのインデックス値も当然同じ＝インデックス値の求め方も同じ
では何を変えるべきかと言うと、参照する配列を切り替えれば良いということになりますよね？

[Mason]

ごめんなさい。理解できなかったです。
参照する配列を切り替えるとはどういったことでしょうか？
配列、算術演算はそのままに、配列の取り出し方を変えるという認識でしょうか？

[itagagaki]

漢字の答えは $kanji から引き出して成功していますね。
アルファベットの答えは同じようにして $alphabet から引き出せばいいわけですよね。

漢字の答えを $kanji から引き出すときのインデックスとして $listKanji を計算していますね。
寅は6という具合に。

アルファベットの答えを $alphabet から引き出すときのインデックスをわざわざ $listAlphabet におかしな計算をしていますが、この場合のインデックス値の計算は $listKanji と同じですよね。
Toraは6でしょう？

[itagagaki]

これはPHPの問題解決ではなくて、プログラミング基礎講座みたいな話になってしまっているので、回答はここまでにさせていただきます。

[Mason]

かしこまりました。
$listAlphabet削除して、$zyuniAlphabet = $alphabet[$listKanji];にしました。
ありがとうございました！