-
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
番地の前までの住所が漢数字で終わっているケースに対応 #145
Conversation
6c14464
to
ce5a0ad
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なるほど。
エッジケースはもうちょっとあるような気がしますが(例えば 一二三123三二一
とか?笑)これで大丈夫ですかね。。もうちょっとテストがほしいかも
そうですね。もうちょっと足してみます 👍 |
ちょっと試してみたのですが、カバー範囲を広げようとすると番地号の正規表現の箇所全部の修正が必要でかなり複雑になりそうです。 その場合の話ですが、正規化レベル3だけど番地の正規化が失敗している状態で、レベルに該当の情報を含めることができない状態です。 |
なるほど。 |
各レベルの処理は切り分けてリファクタリングしたいですね。
を追加するのがいいのかもですが、全てのレベル3が4にずれてしまって変更が大きすぎるのでできないですね。 |
イシューだけ残して #144 ペンディングにします。 |
No description provided.