全角→半角処理について
YuukiToriyama opened this issue · comments
Rust移植をつくってみようなどと思いコードを拝読させて頂いております。
src/main.tsを読んでいたとき、次の部分についてふと疑問に思いました。
normalize-japanese-addresses/src/main.ts
Lines 60 to 63 in f332bf8
zen2han
が
normalize-japanese-addresses/src/lib/zen2han.ts
Lines 1 to 5 in f332bf8
と定義されているので、二重に正規表現の検索をかけるようになってやや冗長ではないかと思いました。
まあ、変えたところで速度が格段に上がる箇所ではありませんし、見た目も今のコードのほうが良いのでこだわる必要もありませんが、57行目から63行目は次のようにしても動くようです。
let addr = zen2han(address)
.replace(/ /g, ' ')
.replace(/ +/g, ' ')
@YuukiToriyama ありがとうございます!
@champierre こちら #110 と一緒に対処して次のリリースに含めていただけますと。
cc: @keichan34