モデルでより効率的なコードを作成するには？

Question

メソッドのコードをリファクタリングする考えですか？self.import_data？これは、アプリケーションがCSVファイルをデータベースに保存できるようにする方法です（ユーザーの電子メールには制限があります）。正午に毎日走るはずだから、速くなければならない。 現在、大きなCSVファイルを持っているときに実行するのに非常に時間がかかります。私はこのコードをより効率的にして、ある時間を勝ち取る（またはループを回避する、または要求を少なくする）方法があるのだろうかと思います。私は本当にプロセスがどれほど長くなるか、そしてそれをどう修正するのか本当に分かっていません。ここで

は私のモデルである：

class Person < ActiveRecord::Base 
    has_paper_trail 
    validates :email, uniqueness: true 
    require 'csv' 

    def is_former_email?(update_email) 
    self.versions.each do |version| 
     next if version.object.nil? 
     return true if version.object.include?(update_email) 
    end 
    end 

def self.import_data 
    filename = File.join Rails.root, '/vendor/people.csv' 

    CSV.foreach(filename, headers: true, col_sep: ',') do |row| 
    firstname, lastname, home_phone_number, mobile_phone_number, email, address = row 

    person = Person.find_or_create_by(firstname: row["firstname"], lastname: row['lastname'], address: row['address']) 
    if person.is_former_email?(row['email']) == true 
     puts "not allowed" 
    else 
     person.update_attributes({firstname: row['firstname'], lastname: row['lastname'], home_phone_number: row['home_phone_number'], mobile_phone_number: row['mobile_phone_number'], address: row['address'], email: row['email']}) 
    end 
    end 
    end 
end 

Answer 1

私は少しはあなたのコードをリファクタリングしましたが、より効率的にするために、私は宝石activerecord-importを使用すると、検索以前のメールのバージョンのモデルを最適化することをお勧めします。

class Person < ActiveRecord::Base 
    require 'csv' 

    FILE_NAME = File.join Rails.root, '/vendor/people.csv' 

    validates :email, uniqueness: true 

    has_paper_trail 

    def self.import_data 
    people = CSV.new(File.new(FILE_NAME), headers: true, header_converters: :symbol, converters: :all).to_a.map(&:to_hash) 
    versions_by_item_id = Version.where(item_type: 'Person').select('item_id, object').group_by(&:item_id) 

    people.each do |person_params| 
     person = Person.find_or_create_by(person_params.slice(:firstname, :lastname, :address)) 
     if versions_by_item_id[person.id] && versions_by_item_id[person.id].sum { |v| v.object.to_s }.include?(person_params[:email]) 
     puts 'not allowed' 
     else 
     person.update_attributes(person_params.slice(:home_phone_number, :mobile_phone_number, :email)) 
     end 
    end 
    end 
end