search

yyc1217 / twzipcode-data

開發用中華郵政郵遞區號 / Taiwan zipcodes for developers.
MIT License
15 stars 12 forks source link

自動轉換中華郵政XML檔案到JS #2

Closed ausir0726 closed 7 years ago

ausir0726 commented 7 years ago

it can automaticlly parse 中華郵政 xml file to js

之前在找相關郵遞區號的都是人家整理過的 不過好幾個都是整理以後改六都也沒更新... 本來想說要自己寫，但是看到這邊還有在維護，想說幫忙寫一個可以從中華郵政的 XML 直接轉的JS 這樣以後要維護的時候就不需要人工在那邊對哪一個縣市有改(雖然都很容易看) 如果有保留資料來源也可以清楚知道是中華郵政的甚麼時候的檔案，轉的時間又是甚麼時候 不然這種東西的更新頻率不高，沒有標記最後真的不知道哪一版了 QQ

如果有轉換，未來如果要增加 ZIP5 的功能也就沒有問題了

轉換後資料尚未正式覆蓋，目前轉換後會建立一支新的檔案在 src 尚未確認同意前暫時先不直接覆蓋 counties.js / zipcodes.js

另外有發現你有對一些 雙字 的行政區中間插入全形空白 XML 檔本身是沒有，所以我沒有另外做處理，如果有需要我覺得可以用 option util 去處理 我也想再補一些可以改 key 值的 util , 因為預設用 id 另外使用的時候有時候需要再套過...

thanks.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 45369346d7638ff2736e8ee92ef4ce54a1e7054f on ausir0726:master into 16e8fa1b62ec098370b7bcb2f92882dd58d8dd5e on yyc1217:master.

ausir0726 commented 7 years ago

發現有新竹市需要例外處理，等處理完再PR

yyc1217 commented 7 years ago

@ausir0726

  1. 關於資料來源的部份，這個專案的目標是提供3碼的郵遞區號，以及提供中、英文的版本。所以一開始我是一筆一筆手動輸入，因為覺得3碼變動的頻率太低了，就算真的有變動，範圍應該也不大

  2. xml轉成js的部份歡迎提供~但是否能repo本身不保留xml檔案，改成需另外下載（手動或自動）呢？因為xml檔案蠻大的，放在repo中似乎不太合適；而且如果每次更新都要手動去抓最新版，那不如就直接讓每個開發者都去中華郵政那邊抓最新版就好。 也許可以考慮將轉換使用的版本資訊用註解寫在js檔裡面，這樣除了從commit訊息得知以外，還可以直接瀏覽。

  3. 在行政區中補全形空白是覺得顯示時比較整齊一致（因為三個字的行政區最多），還是請預設為補全形空白，並提供option讓開發者自行選擇可以不要補

  4. 若你有增加util的話，請一併提供unit tests~

ausir0726 commented 7 years ago

  1. 一筆一筆整理我幹過一次... 而且我比較蠢，沒有寫到 npm 去 結果我當初拿的那份後來沒更新了 2014 （六都）前的資料 而且我把 JSON 都存在專案去了，現在要更新好痛苦～QQ 不知道哪天誰又生都了... 昏倒，不過我個人看 XML PARSER 的好處是之後可以再擴展 ZIP3+2 ANYWAY 想遠了～

  2. XML 6萬筆資料確實說大不大說小不小 《 大概就是六萬行 code ... 》 generate 是不是大，也只備存，也許照你說的在註解標示版本與連結即可...

  3. OK

  4. Rager that