jp_address 1.0.1 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ef948c1cac5b64467a6007a4679ce93ef6be1e68e367dbf52cabd69f7d3099f5
-  data.tar.gz: e08a3c24c255492740a037f02f27b09b3522034186753711c1f0fe1f524afa8a
+  metadata.gz: 67701127b1fbec6ae9292647b6373acdeca9dcb51b78addeefaeed7544852531
+  data.tar.gz: d3fb7f7ee44c1976825cb22c0394414a9dec14aa220536e17bf57daa382a7575
 SHA512:
-  metadata.gz: 489a71906ce28abc642138bc4129d8f3608098d1d6a666d43bef11ee083e6a31ccc4364f51a55a37ddae0eaad5e98bac1abe57041b5fbafd9aaf1a83d8cb2afb
-  data.tar.gz: 44d2f3ea727acff307d755a186817f341b6fb7e0b539822e65f39a294b78ec066903a5fa062835b367e3353bd05414ead9a84eadfbb4bb9d003ac7fa1a3d6d95
+  metadata.gz: 33814d00bd9491bce91816e44066be23025af90b8c68ff62a3924fe6ed1316d5f08339864642a3848f55a7cff23838712816ec5747edabe633e8ace33c91a8fa
+  data.tar.gz: becaad6991cc0cc30d8dc05f479488500b02707c3028c6b8841af2ed3e35a2042e01c5332a15ce20caa9962deb4231b629f9f83503f4e09a0bd0b9b7e1a36c74

data/CHANGELOG.md

@@ -0,0 +1,57 @@
+# CHANGELOG
+
+## 2.0.0
+
+最新の Ruby / Rails への対応と、郵便番号データの取り込みまわりのバグ修正。
+
+### 対応バージョン
+
+* Ruby 3.1 〜 3.4、Rails 7.1 / 8.0 / 8.1 に対応。
+  （Ruby 3.1 + Rails 7.1 〜 Ruby 3.4 + Rails 8.1 で実際にテスト済み）
+* Rails 6.x 以前のサポートを終了。
+* Ruby 3.4 で `csv` が bundled gem になったため、gemspec の依存に追加。
+  これがないと `require 'csv'` に失敗する。
+* `railties` / `activerecord` を runtime 依存として明示。
+* rubyzip 3.x に対応（`>= 2.3, < 4.0`）。
+
+### バグ修正
+
+* **町域名が複数行に分かれている郵便番号で町名が消えていた**。
+  ken_all.csv は町域名が長いと括弧の途中で改行して次の行に続くが、その
+  継続行を別の町名として扱っていたため、共通部分が見つからず町名が空に
+  なっていた。括弧が閉じるまでの行を読み飛ばすようにした。
+  例：9292225（石川県羽咋郡宝達志水町 米出）が `""` → `"米出"`。
+* **同じ郵便番号の行が全部同じ町名でも町名が空になっていた**。
+  `_find_shared_name_from` が「差分が見つからない＝共通部分なし」と
+  判定していたのを修正。短い方の地名がまるごと共通部分になる場合
+  （例：東中島／東中島本町）もその地名を返すようにした。
+* 日本郵便へのリクエストがリダイレクト・エラーレスポンスを一切見ていな
+  かったのを修正。リダイレクトは最大 5 回まで追い、200 以外は
+  `JpAddress::Zipcode::DownloadError` を投げる。
+* zip の中から CSV エントリだけを取り出すようにした（従来は全エントリを
+  同じパスに展開しようとしていた）。
+* 存在しない CSV パスを `load_master_data` に渡すと、無駄にダウンロード
+  してから `Errno::ENOENT` になっていたのを `ArgumentError` に。
+* `row[8]` が空の行で `NoMethodError` にならないようにした。
+* `FileUtils` を require していなかったのを修正。
+
+### 変更
+
+* `load_master_data` を全面的に書き直し。行ごとの生 SQL を `insert_all!`
+  でのバッチ投入に変え、統合処理も CSV 読み込み中にメモリ上で行うように
+  したので、大幅に速くなった（本番でのロードが数分 → 数十秒程度）。
+  これに伴い内部メソッド `_merge_same_zip_addresses` は削除。
+* API のレスポンスを `render plain:` から `render json:` に変更。
+  Content-Type が `text/plain` から `application/json` になる。
+* 郵便番号の全角数字（`５３３－００３３`）を受け付けるようにした。
+* `JpAddress::ApplicationRecord` を追加し、`Zipcode` の親クラスにした。
+* Sprockets 前提の app/assets・レイアウト・空の ApplicationHelper を削除。
+  JSON しか返さない engine には不要で、propshaft / importmap 環境では
+  むしろ壊れるため。
+* テストを VCR カセット（2.5MB のバイナリ）から WebMock のスタブに置き換え。
+  コントローラスペックはリクエストスペックに変更。
+* GitHub Actions で Ruby × Rails のマトリクステストを追加。
+
+## 1.0.2 以前
+
+省略。

data/README.md

@@ -1,10 +1,10 @@
-# Ruby on Rails 6 で 郵便番号住所検索 な gem
+# Ruby on Rails で 郵便番号住所検索 な gem
 
 ## JpAddressとは
-日本郵便の「[郵便番号データ](https://www.post.japanpost.jp/zipcode/dl/oogaki-zip.html)」を Rails 6.1 で使用するための gem です。
+日本郵便の「[郵便番号データ](https://www.post.japanpost.jp/zipcode/download.html)」を用いて、あなたの Rails サイトに「郵便番号からの住所検索機能」を組み込むための gem です。
 以下の機能を提供します。
 
-* [[郵便番号データ](https://www.post.japanpost.jp/zipcode/dl/oogaki/zip/ken_all.zip "ken_all.zip")]をダウンロードして自前ＤＢのテーブル（jp_address_zipcodes）にロードするクラスメソッド。（```JpAddress::Zipcode.load_master_data```）
+* [[郵便番号データ](https://www.post.japanpost.jp/zipcode/dl/kogaki/zip/ken_all.zip "ken_all.zip")]をダウンロードして自前ＤＢのテーブル（jp_address_zipcodes）にロードするクラスメソッド。（```JpAddress::Zipcode.load_master_data```）
 * 郵便番号を受け取り都道府県名と住所をJSONで返却するAPI。
 （```jp_address/zipcodes#search```）
 
@@ -14,8 +14,17 @@ APIはお使いのRailsアプリケーションにマウントして使います
 あと必要なのは、戻ってくるJSONを加工してHTML要素にセットするJavaScriptの記述だけです。<br>
 （本記事下部にサンプルコードを掲載しています。）
 
+### 対応バージョン
+
+| | 対応 |
+|---|---|
+| Ruby | 3.1 以降 |
+| Rails | 7.1 / 8.0 / 8.1 |
+
+Rails 6.x 以前をお使いの場合は 1.0.2 をご利用ください。
+
 ### インストール
-GemFileに追記
+Gemfileに追記
 ```ruby
 gem 'jp_address'
   ```
@@ -36,13 +45,34 @@ $ bundle exec rails runner -e development 'JpAddress::Zipcode.load_master_data'
 $ bundle exec rails runner -e production 'JpAddress::Zipcode.load_master_data'
 ```
 
-環境にもよりますが、１～３分ぐらいかかると思います。
+APP_ROOT/tmp/ を作業ディレクトリに使用しています。<br>
+最初にテーブルをトランケートしますので、毎回「全件insert」になります。<br>
 
-※ APP_ROOT/tmp/ を作業ディレクトリに使用しています。<br>
-※ 最初にテーブルをトランケートしますので、毎回「全件insert」になります。
+すでに手元に ken_all.csv がある場合は、パスを渡せばダウンロードを省略できます。<br>
+（文字コードは日本郵便の配布どおり CP932 のままで構いません。）
+```ruby
+JpAddress::Zipcode.load_master_data('tmp/ken_all.csv')
+```
+
+同じ郵便番号を持つレコードは統合されます。<br>
+<br>
+例：9896712<br>
+```
+"宮城県","大崎市","鳴子温泉水沼"
+"宮城県","大崎市","鳴子温泉南山"
+"宮城県","大崎市","鳴子温泉山際"
+"宮城県","大崎市","鳴子温泉和田"
+```
+これらは先頭から共通する地名を探し、うまく見つかれば
+```
+"宮城県","大崎市","鳴子温泉"
+```
+として１つのレコードにします。
+共通する地名が抜き出せない場合は空の町名にします。
 
 ### APIのマウント
 Railsアプリの config/routes.rb に追記。
+at: に渡す値は /jp_address でなくても /service や /api などでも構いません。
 ```ruby
 mount JpAddress::Engine, at: "/jp_address"
 ```
@@ -61,6 +91,14 @@ http://localhost:3000/jp_address/zipcodes/search?zip=5330033
 {"id":84280,"zip":"5330033","prefecture":"大阪府","city":"大阪市東淀川区","town":"東中島"}
 ```
 
+該当する郵便番号がない場合は、各項目が null の JSON が返ります。
+```js script
+{"id":null,"zip":null,"prefecture":null,"city":null,"town":null}
+```
+
+半角ハイフン・空白・全角数字は自動的に取り除かれるので、`533-0033` や `５３３－００３３` を
+そのまま投げても構いません。
+
 ### APIを利用するサンプル JavaScript
 フォームに
 1. #zipcode （郵便番号を入力するテキストボックス）
@@ -68,15 +106,11 @@ http://localhost:3000/jp_address/zipcodes/search?zip=5330033
 3. #address （住所を表示するテキストボックス）
 
 の３要素があるとします。<br>
-#zipcodeに入れられた値を keyup イベントで拾ってAPIを叩き、都道府県プルダウンを選択し、住所をセットするサンプルです。
+#zipcodeに入れられた値を input イベントで拾ってAPIを叩き、都道府県プルダウンを選択し、住所をセットするサンプルです。<br>
 
 都道府県プルダウンは、戻ってくるJSONの "prefecture" すなわち都道府県名で選択します。<br>
 ですので、お持ちの都道府県マスターの各レコードがどのようなＩＤを持っていても構いません。
 
-※ JQuery の存在を前提にしています。<br>
-※ 郵便番号の半角ハイフンは自動でカットされます。<br>
-※ もともと CoffeeScript で書いてあったソースを decaffeinate したものですので冗長です。本質的な処理はAddressSearch が担っているだけで、他の関数は decaffeinate に必要なだけです。
-
 #### フォーム
 ```
 <form>
@@ -134,106 +168,73 @@ http://localhost:3000/jp_address/zipcodes/search?zip=5330033
 </form>
 ```
 
-#### application.js など共通に読み込まれるファイルに配置するJavaScript
+#### JavaScript
+jQuery などのライブラリは不要です。app/javascript 配下など、フォームのあるページで
+読み込まれる場所に置いてください。
 
 ```js script
-  function _classCallCheck(instance, Constructor) {
-    if (!(instance instanceof Constructor)) {
-      throw new TypeError("Cannot call a class as a function");
-    }
+class AddressSearch {
+  constructor(zipSelector, prefectureSelector, addressSelector, endpoint = '/jp_address/zipcodes/search') {
+    this.zip        = document.querySelector(zipSelector);
+    this.prefecture = document.querySelector(prefectureSelector);
+    this.address    = document.querySelector(addressSelector);
+    this.endpoint   = endpoint;
   }
 
-  function _defineProperties(target, props) {
-    for (var i = 0; i < props.length; i++) {
-      var descriptor = props[i];
-      descriptor.enumerable = descriptor.enumerable || false;
-      descriptor.configurable = true;
+  start() {
+    this.zip.addEventListener('input', () => this.execute());
+  }
 
-      if ("value" in descriptor)
-        descriptor.writable = true;
-      Object.defineProperty(target, descriptor.key, descriptor);
+  async execute() {
+    const zip = this.zip.value.replace(/[^0-9０-９]/g, '');
+    if (zip.length !== 7) return;
+
+    const url = `${this.endpoint}?zip=${encodeURIComponent(zip)}`;
+    const res = await fetch(url, { headers: { Accept: 'application/json' } });
+    if (!res.ok) return;
+
+    const json = await res.json();
+    if (json.id === null) {
+      this.clear();
+    } else {
+      this.setPrefecture(json.prefecture);
+      this.address.value = `${json.city}${json.town}`;
     }
   }
 
-  function _createClass(Constructor, protoProps, staticProps) {
-    if (protoProps)
-      _defineProperties(Constructor.prototype, protoProps);
-    if (staticProps)
-      _defineProperties(Constructor, staticProps);
-    return Constructor;
+  clear() {
+    this.prefecture.selectedIndex = 0;
+    this.address.value = '';
   }
 
-  var AddressSearch = function() {
-    "use strict";
-    function AddressSearch(zip_elem_id, prefecture_elem_id, address_elem_id) {
-      _classCallCheck(this, AddressSearch);
-      this.zip                = $(zip_elem_id);
-      this.prefecture         = $(prefecture_elem_id);
-      this.address            = $(address_elem_id);
-      this.prefecture_elem_id = prefecture_elem_id;
+  setPrefecture(name) {
+    for (const option of this.prefecture.options) {
+      if (option.text === name) {
+        option.selected = true;
+        return;
+      }
     }
+  }
+}
 
-    _createClass(AddressSearch, [{
-      key: "_remove_hyphen",
-      value: function _remove_hyphen() {
-        return this.zip.val(this.zip.val().replace(/-/, ''));
-      }
-    }, {
-      key: "_clear_current_value",
-      value: function _clear_current_value() {
-        $(this.prefecture_elem_id + ' >option:eq(0)').prop('selected', true);
-        return this.address.val('');
-      }
-    }, {
-      key: "_set_prefecture",
-      value: function _set_prefecture(json) {
-        return $(this.prefecture_elem_id + ' > option').each(function() {
-          if ($(this).text() === json['prefecture']) {
-            return $(this).prop('selected', true);
-          }
-        });
-      }
-    }, {
-      key: "_set_address",
-      value: function _set_address(json) {
-        return this.address.val(json['city'] + json['town']);
-      }
-    }, {
-      key: "_call_api",
-      value: function _call_api() {
-        var _this = this;
-        return $.getJSON('/jp_address/zipcodes/search', {zip: this.zip.val()}, function(json) {
-          if (json['id'] === null) {
-            return _this._clear_current_value();
-          } else {
-            _this._set_prefecture(json);
-            return _this._set_address(json);
-          }
-        });
-      }
-    }, {
-      key: "execute",
-      value: function execute() {
-        this._remove_hyphen();
-        if (this.zip.val().length === 7) {
-          return this._call_api();
-        }
-      }
-    }]);
+// #zipcode, #prefecture_id, #address を各自の環境に合わせて書き換えてください。
+document.addEventListener('DOMContentLoaded', () => {
+  new AddressSearch('#zipcode', '#prefecture_id', '#address').start();
+});
+```
+
+### 開発
 
-    return AddressSearch;
-  }();
+```
+$ bundle install
+$ bundle exec rspec
 ```
 
-#### フォームのあるページに配置するJavaScript
-```js script
-  // #zipcode, #prefecture_id, #address を各自の環境に合わせて書き換えてください。
-  $(function() {
-    var address_search = new AddressSearch('#zipcode', '#prefecture_id', '#address');
-    $('#zipcode').keyup(function() {
-      address_search.execute();
-    });
-  });
+複数バージョンの Rails で試す場合は gemfiles/ 配下の Gemfile を使ってください。
+
+```
+$ BUNDLE_GEMFILE=gemfiles/rails_7_1.gemfile bundle install
+$ BUNDLE_GEMFILE=gemfiles/rails_7_1.gemfile bundle exec rspec
 ```
 
 ##### 作者

data/Rakefile

@@ -4,23 +4,22 @@ rescue LoadError
   puts 'You must `gem install bundler` and `bundle install` to run rake tasks'
 end
 
-require 'rdoc/task'
-
-RDoc::Task.new(:rdoc) do |rdoc|
-  rdoc.rdoc_dir = 'rdoc'
-  rdoc.title    = 'JpAddress'
-  rdoc.options << '--line-numbers'
-  rdoc.rdoc_files.include('lib/**/*.rb')
+begin
+  require 'rdoc/task'
+
+  RDoc::Task.new(:rdoc) do |rdoc|
+    rdoc.rdoc_dir = 'rdoc'
+    rdoc.title    = 'JpAddress'
+    rdoc.options << '--line-numbers'
+    rdoc.rdoc_files.include('lib/**/*.rb')
+  end
+rescue LoadError
+  # rdoc is optional; skip the task when it is not installed.
 end
 
-APP_RAKEFILE = File.expand_path("../spec/dummy/Rakefile", __FILE__)
+APP_RAKEFILE = File.expand_path('spec/dummy/Rakefile', __dir__)
 load 'rails/tasks/engine.rake'
 
-
-load 'rails/tasks/statistics.rake'
-
-
-
 Bundler::GemHelper.install_tasks
 
 require 'rspec/core'
@@ -29,5 +28,5 @@ RSpec::Core::RakeTask.new(:spec) do |spec|
   spec.pattern = FileList['spec/**/*_spec.rb']
 end
 
-task :default => :spec
-task :test => :spec
+task default: :spec
+task test: :spec

data/app/controllers/jp_address/application_controller.rb

@@ -1,4 +1,5 @@
 module JpAddress
   class ApplicationController < ActionController::Base
+    protect_from_forgery with: :null_session
   end
 end

data/app/controllers/jp_address/zipcodes_controller.rb

@@ -1,10 +1,15 @@
-require_dependency "jp_address/application_controller"
-
 module JpAddress
   class ZipcodesController < ApplicationController
     def search
-      @zipcode = Zipcode.find_by(:zip => params[:zip].to_s.gsub(/[^0-9]/, '')) || Zipcode.new
-      render plain: @zipcode.to_json
+      @zipcode = Zipcode.find_by(zip: _normalized_zip) || Zipcode.new
+      render json: @zipcode
+    end
+
+    private
+
+    # 全角数字・ハイフン・空白などを取り除いて半角数字だけにします。
+    def _normalized_zip
+      params[:zip].to_s.tr('０-９', '0-9').gsub(/[^0-9]/, '')
     end
   end
 end

data/app/models/jp_address/application_record.rb

@@ -0,0 +1,5 @@
+module JpAddress
+  class ApplicationRecord < ActiveRecord::Base
+    self.abstract_class = true
+  end
+end

data/app/models/jp_address/zipcode.rb

@@ -1,82 +1,212 @@
 module JpAddress
-  class Zipcode < ActiveRecord::Base
+  class Zipcode < ApplicationRecord
 
-    MASTER_CSV_PATH = 'tmp/ken_all.csv'
+    # 日本郵便からダウンロードした CSV の一時保存先。
+    MASTER_CSV_PATH = 'tmp/ken_all.csv'.freeze
+
+    # 郵便番号データ（住所の郵便番号／小書き）の配布 URL。
+    MASTER_FILE_URL = 'https://www.post.japanpost.jp/zipcode/dl/kogaki/zip/ken_all.zip'.freeze
+
+    # insert_all で一度に流し込む件数。
+    INSERT_BATCH_SIZE = 1_000
+
+    # リダイレクトを追う上限。
+    MAX_REDIRECTS = 5
+
+    class DownloadError < StandardError; end
 
     def self.download_master_file_from_japanpost
       _setup_directory
-      zip = _save_zip(_request_to_japanpost.body)
-      _save_csv(zip.path)
+      tmp_zip = _save_zip(_request_to_japanpost)
+      begin
+        _save_csv(tmp_zip.path)
+      ensure
+        tmp_zip.close!
+      end
       :success if File.exist?(MASTER_CSV_PATH)
     end
 
     def self._setup_directory
-      FileUtils.mkdir_p 'tmp'
+      FileUtils.mkdir_p File.dirname(MASTER_CSV_PATH)
       _remove_csv
     end
     private_class_method :_setup_directory
 
-    def self._request_to_japanpost
-      res = nil
-      https = Net::HTTP.new('www.post.japanpost.jp',443)
-      https.use_ssl = true
-      https.start {
-        res = https.get('/zipcode/dl/kogaki/zip/ken_all.zip')
-      }
-      res
+    def self._request_to_japanpost(url = MASTER_FILE_URL, redirect_limit = MAX_REDIRECTS)
+      raise DownloadError, "too many redirects while downloading #{MASTER_FILE_URL}" if redirect_limit <= 0
+
+      uri = URI.parse(url)
+      res = Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |https|
+        https.get(uri.request_uri)
+      end
+
+      case res
+      when Net::HTTPSuccess
+        res.body
+      when Net::HTTPRedirection
+        location = res['location']
+        raise DownloadError, "got a redirect without a location from #{url}" if location.nil?
+
+        _request_to_japanpost(URI.join(url, location).to_s, redirect_limit - 1)
+      else
+        raise DownloadError, "failed to download #{url} (#{res.code} #{res.message})"
+      end
     end
     private_class_method :_request_to_japanpost
 
     def self._save_zip(binary)
-      tmp_zip = Tempfile.open('ken_all.zip') do |f|
-        f.binmode
-        f.write binary
-        f
-      end
+      tmp_zip = Tempfile.new(['ken_all', '.zip'])
+      tmp_zip.binmode
+      tmp_zip.write binary
+      tmp_zip.flush
+      tmp_zip.close
       tmp_zip
     end
     private_class_method :_save_zip
 
     def self._save_csv(zip_path)
       Zip::File.open(zip_path) do |zip_file|
-        zip_file.each do |entry|
-          entry.extract(MASTER_CSV_PATH)
-        end
+        entry = zip_file.find { |e| e.file? && File.extname(e.name).downcase == '.csv' }
+        raise DownloadError, "no CSV file found in the archive from #{MASTER_FILE_URL}" if entry.nil?
+
+        entry.extract(MASTER_CSV_PATH)
       end
     end
     private_class_method :_save_csv
 
+    # CSV を読み込んでテーブルを作り直します。
+    #
+    # 同じ郵便番号を持つ行は１レコードに統合してから insert するので、
+    # 郵便番号はテーブル内で一意になります。
     def self.load_master_data(csv_path = MASTER_CSV_PATH)
-      download_master_file_from_japanpost if !File.exist?(csv_path)
+      unless File.exist?(csv_path)
+        raise ArgumentError, "#{csv_path} does not exist" unless csv_path == MASTER_CSV_PATH
+
+        download_master_file_from_japanpost
+      end
+
       _clear_table
-      CSV.foreach(csv_path, encoding:'SJIS:UTF-8') do |row|
-        connection.execute(
-          sanitize_sql(
-            ["insert into jp_address_zipcodes (zip, prefecture, city, town)
-              values ('%s', '%s', '%s', '%s')", row[2], row[6], row[7], _remove_needless_words(row[8])
-            ]
-          )
-        )
+      transaction do
+        _each_record_batch_from(csv_path) { |batch| insert_all!(batch) }
       end
       _remove_csv
+      nil
+    end
+
+    # CSV を郵便番号ごとにまとめ、insert_all に渡せる Hash の配列を
+    # INSERT_BATCH_SIZE 件ずつ yield します。
+    def self._each_record_batch_from(csv_path)
+      _read_master_csv(csv_path).each_value.each_slice(INSERT_BATCH_SIZE) do |slice|
+        yield slice.map { |rec|
+          {
+            zip:        rec[:zip],
+            prefecture: rec[:prefecture],
+            city:       rec[:city],
+            town:       _find_shared_name_from(rec[:towns])
+          }
+        }
+      end
+    end
+    private_class_method :_each_record_batch_from
+
+    # 郵便番号をキーにして CSV の行をまとめます。
+    # 都道府県名・市区町村名は同じ郵便番号の最初の行のものを採用します。
+    #
+    # 例：9896712
+    #   "宮城県","大崎市","鳴子温泉水沼"
+    #   "宮城県","大崎市","鳴子温泉南山"
+    #   "宮城県","大崎市","鳴子温泉山際"
+    #   "宮城県","大崎市","鳴子温泉和田"
+    # これらは
+    #   "宮城県","大崎市","鳴子温泉" として１つのレコードにします。
+    #   共通する地名が抜き出せない場合は空の町名にします。
+    #
+    # ken_all.csv では町域名が長いと括弧の途中で改行され、次の行に続きます。
+    #   "米出（１２４、１２５、１２７、"
+    #   "１４８、１４９、１５４番地）"
+    # 続きの行は町域名ではないので、括弧が閉じるまでの行は読み飛ばします。
+    def self._read_master_csv(csv_path)
+      records = {}
+      # 郵便番号ごとの「閉じていない括弧の数」。0 より大きい間は継続行。
+      open_parens = Hash.new(0)
+
+      # ken_all.csv の文字コードは CP932（Ruby では SJIS が CP932 の別名）。
+      CSV.foreach(csv_path, encoding: 'CP932:UTF-8') do |row|
+        zip = row[2].to_s.strip
+        next if zip.empty?
+
+        raw_town = row[8].to_s
+        if open_parens[zip] > 0
+          open_parens[zip] += _paren_balance(raw_town)
+          next
+        end
+        open_parens[zip] = [_paren_balance(raw_town), 0].max
+
+        record = (records[zip] ||= {
+          zip:        zip,
+          prefecture: row[6].to_s.strip,
+          city:       row[7].to_s.strip,
+          towns:      []
+        })
+        town = _remove_needless_words(raw_town)
+        record[:towns] << town if town.present?
+      end
+      records
+    end
+    private_class_method :_read_master_csv
+
+    def self._paren_balance(text)
+      text.count('（') - text.count('）')
+    end
+    private_class_method :_paren_balance
+
+    # 引数に渡された地名群から、先頭から見て共通となる地名を返します。
+    #
+    # input = %w[鳴子温泉小身川
+    #            鳴子温泉川袋
+    #            鳴子温泉木戸脇
+    #         ]
+    #
+    # return => 鳴子温泉
+    #
+    # 共通部分が１文字しかない場合（例：大分／大阪）は、地名として意味を
+    # なさないので空文字を返します。ただし短い方の地名がまるごと共通部分に
+    # なる場合（例：東中島／東中島本町）は、その地名をそのまま返します。
+    def self._find_shared_name_from(names)
+      names = Array(names).reject { |name| name.blank? }
+      return '' if names.empty?
+
+      shortest      = names.min_by { |name| name.length }
+      shared_length = 0
+      shortest.each_char.with_index do |char, pos|
+        break unless names.all? { |name| name[pos] == char }
+
+        shared_length = pos + 1
+      end
+
+      return shortest if shared_length == shortest.length
+
+      shared_length > 1 ? shortest[0, shared_length] : ''
     end
 
     def self._clear_table
-      begin
-        connection.truncate 'jp_address_zipcodes'
-      rescue NotImplementedError
-        connection.execute 'delete from jp_address_zipcodes'
+      connection_pool.with_connection do |conn|
+        begin
+          conn.truncate table_name
+        rescue NotImplementedError
+          conn.delete "delete from #{conn.quote_table_name(table_name)}"
+        end
       end
     end
     private_class_method :_clear_table
 
     def self._remove_needless_words(base)
-      base.sub(/以下に掲載がない場合/, '').sub(/（.*/, '')
+      base.to_s.sub(/以下に掲載がない場合/, '').sub(/（.*/, '').strip
     end
     private_class_method :_remove_needless_words
 
     def self._remove_csv
-      FileUtils.rm_rf MASTER_CSV_PATH
+      FileUtils.rm_f MASTER_CSV_PATH
     end
     private_class_method :_remove_csv
 

data/db/migrate/20160312045953_create_jp_address_zipcodes.rb

@@ -1,4 +1,4 @@
-class CreateJpAddressZipcodes < ActiveRecord::Migration[5.0]
+class CreateJpAddressZipcodes < ActiveRecord::Migration[7.1]
   def change
     create_table :jp_address_zipcodes do |t|
       t.string :zip       , null: false, index: true