値がうまく代入されません。。

Question

csvからインポートしたもので、Userモデルにすでに登録しているレコードと比較して当てはまるものを@update_usersに代入したいです。
@update_userにうまく入るかどうか確かめるために、

logger.info "update値を見たい #{@update_user.inspect}"

と記述して確かめているのですが、@update_userには値は入っていませんでした。

find_byの使い方なのでしょうか？

def import
        csv_users = []
        CSV.foreach(params[:file], headers: true, encoding: 'Shift_JIS:UTF-8') do |row|
            values = row.to_h
            csv_users << User.new(
                id: values["id"],
                name: values["name"],
                age: values["age"],
                note: values["note"],
                text: values["text"],
                number: values["number"],
                )
        end
        csv_users.each do |i|
###ここです！！
        @update_user = User.find_by('name = ? AND text = ? AND number = ?', i.name, i.text, i.number)
        if @update_user.present?
          @update_user.update(name: i.note)
        else
          @new_users = User.create(
            name: i.name,
            age: i.age,
            note: i.note,
            text: i.text,
            number: i.number,
          )
        end

      end

Comments

[maisumakun]

単純に、条件を満たすものがないだけ、ということはないですか？

[oiykram]

csvファイルの中身を変えてみてはいるのですが、うまくいきませんでした。。

[winterboum]

念の為
1. @update_user.update(name: i.note)
は間違いではないですね？　既存のデータの場合は　name を書き換えるのでOK?
2. 既存のデータの場合は、age、 note がCSVの値と異なっていても、そこを更新する必要はない？

[oiykram]

すみません。nameにnameです。間違えてました。
2は、name,text,numberが同じものを取り出して、それ以外のカラムを更新にしたいです。

Answer 1

念の為　の

1. @update_user.update(name: i.note)

は間違いではないですね？　既存のデータの場合は　name を書き換えるのでOK?
2. 既存のデータの場合は、age、 note がCSVの値と異なっていても、そこを更新する必要はない？

１も２も NO　ですね？
つまり
「if @update_user.present?　のときも nameだけではなく全部更新する。」
でしたら、前の質問で私が回答したような方法

CSV.foreach(params[:file], headers: true, encoding: 'Shift_JIS:UTF-8') do |row|
  values = row.to_h
  user = User.find_or_create_by(
                    name: values["name"],
                    text: values["text"],
                    number: values["number"],
                   )
 user.attributes(age: values["age"],note: values["note"])
 user.save
end

でよいと思うのですが。
これではなにかまずいことがありますか？

Comments

[oiykram]

連絡ありがとうございます。
＞user.attributes（age：values ["age"]、note：values ["note"]）
wrong number of arguments (given 1, expected 0)
のArgumentError in UsersController#import　エラーが出てしまいますね。。

[winterboum]

間違えた
user.attributes　= {....} でした。
# この程度のエラーは解決して欲しい