テーブル内にオブジェクトを作成または更新する

Question

新しいオブジェクトが存在しない場合は作成します。または、データベースに既に存在する場合は、その電子メールを更新してください。 CSVファイルからデータをインポートします。

私はfind_or_create_byメソッドを使用しようとしましたが、すでにオブジェクトが存在するときにデータベース内のオブジェクトを複製しました。単に更新するだけではありません。

Person.rb

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

    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.find_or_create_by(firstname: row['firstname'], lastname: row['lastname'], home_phone_number: row['home_phone_number'], mobile_phone_number: row['mobile_phone_number'], email: row['email'], address: row['address']) 
    end 
    end 
end 

あなたは間違って何をやっていたこの

Person.find_or_create_by(email: row['email']) do |person| 
    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']) 
end 

を行う必要がありschema.rb

create_table "people", force: :cascade do |t| 
t.string "email" 
t.string "home_phone_number" 
t.string "mobile_phone_number" 
t.string "firstname" 
t.string "lastname" 
t.string "address" 
t.datetime "created_at",   null: false 
t.datetime "updated_at",   null: false 
end 

Answer 1

これが解決策です：

person = Person.find_or_create_by(firstname: row["firstname"]) 
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']}) 

このソリューションは、あなたが私に提案したものを仕事とされていない理由を私は説明できません。 あなたの助けを借りて@sohail_khalilと@abhilashに感謝！

Answer 2

あなたが複数の列にfind_or_create_by適用されます。それを1つだけに適用します。

私はこれがあなたを助け、あなたの問題を解決することを望みます。

Answer 3

@sohailに何が追加されたかを説明します。複数のパラメータを使用してレコードを見つけて更新または作成するには、

Person.where(<conditions>).first_or_initialize do |person| 
    <initialize code> 
end 

これが適切です。

Answer 4

あなたがRailsの3.2以上を持っている場合は、交換してください：

Person.find_or_create_by(firstname: row['firstname'], lastname: row['lastname'], home_phone_number: row['home_phone_number'], mobile_phone_number: row['mobile_phone_number'], email: row['email'], address: row['address']) 

をして：

Person.where(:email => row['email']).first_or_create(:firstname => row['firstname'], :lastname => row['lastname'], :home_phone_number => row['home_phone_number'], :mobile_phone_number => row['mobile_phone_number'], :email => row['email'], :address => row['address'])