The output of `kansuji2arabic_num` is unified to a string type.

[indenkun]

自分で提案したコードの不備で大変申し訳ないのですが次の問題があり、その解決策について提案します。

kansuji2arabic_num()の結果について1234という結果が得られる、千二百三十四は数値型でリターンしていますが、一二三四については文字列型になってしまいました。

一つの関数で入力されるデータの形に依存してはいますが、特に入力者側で引数を指定するなどの意図をしていないにも関わらず結果の型が変わってくるのはあまり良くないと思いました。

この件について、数値型の文字列型のいずれがいいのかということについて考えると、

• kansuji2arabic以外の関数が文字列型での結果のみをサポートしている。
• 〇一二など0から始まる数をこの関数でも文字列型で"012"と返すことを想定している。これを数値型とすると12となってしまうので、例えば郵便番号などの処理において不適切になってしまうのではないか。

という2点を考慮し、文字列型での結果の統一が妥当なのではないかと考えました。

そこで、kansuji2arabic_num()について結果が文字列型で統一されるように修正しました。それに合わせてテストも修正しました。

度々のPull Requestになってしまいすみません。 お手すきのときに検討していただければと思います。

[uribo]

すみません、確認が遅くなってしまいました。ありがとうございます👍

[indenkun]

確認ありがとうございます。