search

uribo / zipangu

Japanese utility functions and data
https://uribo.github.io/zipangu
Other
56 stars 7 forks source link

Proposal to support `arabic2kansuji_all()` argument in `label_kansuji()`. #52

Closed indenkun closed 2 years ago

indenkun commented 2 years ago

label_kansuji()でnumber = "kansuji"としたとき、ラベルの数字を漢数字にするときに使用している{arabic2kansuji}のarabic2kansuji_all()の違和感のある変換挙動を一部変更し変換規則を引数で選択できるようにしたので、label_kansuji()でもその引数を（...で）受付け、内部のarabic2kansuji_all()に渡してもらえると、label_kansuji()でも漢数字の変換規則を変選択できるようになると思います。

具体的にはarabic2kansuji_all()の挙動の変更としては110000000が入力されたときに、従来は「一億千万」と変換されていましたが、これを「一億一千万」に変換するようになりました。 ただし、この挙動の場合、15000000で「一千五百万」になりますが、「千五百万」も特に違和感がなくこれを選択できるようにadd.one_thousand = FALSEと引数で選択できるように準備しました。 また、1500は「千五百」になりますが、場合により「一千五百」もあり得るかと考え、これに変換できるadd.one_thousand.allという引数を用意しました。 これらの引数をlabel_kansuji()で受け付けてもらえれば嬉しいです。

この変更をした{arabic2kansuji}はCRANに0.1.3として登録済みです。

機能提案のない軽微な変更ですが、ご検討いただければ幸いです。

uribo commented 2 years ago

@indenkun ご提案と実装をありがとうございます。 ... を使った仕組みは便利ですね。 マージさせていただきます。