-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
愛知県豊田市西丹波町三五十の解析に失敗する #215
Comments
@mimidesunya こんにちは。 Issue の報告をいただきありがとうございます。そして返信まで長い時間がかかってしまい申し訳ありませんでした。 もしお時間やご興味がありましたら、この try catch の追加をプルリクエストにしていただくことできますでしょうか?(私の方で取り込む形でも問題ありません!) 検証用のテストがすでに作られていて、 なお、 #144 でも同様のケースが報告されていて、@mimidesunya さんのおかげでこちらの issue にも同時に対応できそうです。ありがとうございます! |
issue-215-fix-japanesee-numeral にpushしようとしたら、権限がないと言われてしまいましたニャ |
@mimidesunya あ、分かりづらくて申し訳ありません、直接コミットではなく、いったんフォークしていただけると助かります 🙇
このような形でいけるかと思うのですが、いかがでしょうか・・? |
ありがとうニャ |
* Add test case with 愛知県豊田市西丹波町三五十 (#215) and 広島県府中市栗柄町名字八五十2459 (#214) * issue #215 fix --------- Co-authored-by: miyabe <[email protected]>
@mimidesunya ありがとうございます、うまくいきました!#225 -> #221 -> master という流れで取り込みました 🙇 |
@mimidesunya リリースを行い、新しいバージョン 2.9.2 では https://codepen.io/geolonia/pen/oNBrqzL この度はプルリクエストを送っていただきありがとうございました! |
'The attribute of kanji2number() must be a Japanese numeral as integer.'エラーが発生します。
原因は小字である三五十を漢数字として解析しようとしているためで、三五〇でも三百五十でもないので不正な漢数字となります。
kan2num.tsを以下のようにして、解析できない漢数字は無視するようにすれば、とりあえず処理できます。
`
import { kanji2number, findKanjiNumbers } from '@geolonia/japanese-numeral'
export const kan2num = (string: string) => {
const kanjiNumbers = findKanjiNumbers(string)
for (let i = 0; i < kanjiNumbers.length; i++) {
try {
// @ts-ignore
string = string.replace(kanjiNumbers[i], kanji2number(kanjiNumbers[i]));
} catch (error) {
// ignore
}
}
return string
}
`
The text was updated successfully, but these errors were encountered: