haruboring / address-complete-divide-api Goto Github PK

Is your feature request related to a problem? Please describe.
郵便番号から住所情報を保管するAPIを毎回叩いているが、速度的に少し問題があるため、自前で郵便番号→住所情報のデータベースを作成し、そこからデータを抜き取ることによって補完を行う形式に変更し、確認を行う。

Describe the solution you'd like
MySQLとかではなく、非構造化データベースのNoSQLとか使う方が速度的には優位（なはず）

Describe alternatives you've considered
現状維持

Is your feature request related to a problem? Please describe.
テストが存在していない

Describe the solution you'd like
単体テストを追加する

Additional context
コードレビューを受けていないため、実装に誤りがあることも大いに想定される。
よってテストを網羅的に行い、バグを発見・修正できるメリットもある。

Is your feature request related to a problem? Please describe.
郵便番号が空の時や、形式が郵便番号に適していない時も外部APIを叩くと無駄な処理時間が発生し、効率が悪い。

Describe the solution you'd like
郵便番号が妥当かどうかを判断する（bool型を返す）関数を追加する。

Describe the bug
Coverageが更新されていない

To Reproduce
READMEを見ればわかる

Expected behavior
READMEに96%のように表記されるはず

Screenshots
撮り忘れた。改善済みが以下

Additional context
pytest-covをインストールし忘れているため、requirements.txtを更新する

Is your feature request related to a problem? Please describe.

外部APIを叩く際に有効な郵便番号として、111-1111や、1111111という形式があるが、１１１１１１１や１１１ー１１１１などはおそらく無効な郵便番号となる。（確認すること）

Describe the solution you'd like
郵便番号を正規化する関数をAPIを叩く前に挟む

Describe the bug
空文字の住所を検索するとエラーが起きるので、
formatted_zipcodeが空文字の時にget-itemしないように変更する