配列の操作方法について

Question

前提・実現したいこと

whereで複数のデータを引っ張って来た後に
特定カラムを変更し保存したい

発生している問題・エラーメッセージ

whereで複数のデータを引っ張って来た後にfind_or_initilize_byで特定したあとに
特定カラムを変更し保存する処理で詰まっています。

該当のソースコード

def update
 line = human.line.where(id: messages_ids)
 kidoku = line.kidoku_messages.find_or_initialize_by(line_id: message_ids, user_id: user)
 kidoku.read_time = Time.now
 kidoku.save!
end

message_idsには配列で複数の値がはいることを想定しています
lineとkidoku_messageの関係は1対多になります

試したこと

line = human.line.find_by!(id: messages_id)で1つに限定したときには
成功しています。

def update
  line = human.line.find_by!(id: messages_id)
  kid = line.kidoku_messages.find_or_initialize_by(line_id: message_id, user_id: user)
  kid.read_time = Time.now
  kid.save!
end

以下のようにeachを使った場合はread_timeがないと言われてしまい処理が最後まで走りません。

def update
 line = human.line.where(id: messages_ids)
 line.each do |kidoku|
  kid = kidoku.kidoku_messages.find_or_initialize_by(line_id: message_ids, user_id: user)
  kid.read_time = Time.now
  kid.save!
 end
end

Comments

[mather]

> line = human.line.find_by!(id: messages_id)で1つに限定したときには成功しています。
成功したときのコードを提示してください。

[退会済みユーザー]

先程修正いたしました

Answer 1

成功したときと失敗したときで find_or_initialize_by の呼び出し方が変わっています。

- kidoku_messages.find_or_initialize_by(line_id: message_id, user_id: user)
+ kidoku_messages.find_or_initialize_by(line_id: message_ids, user_id: user)

message_ids は配列でしょうか？

追記

# Time.now を何度も呼び出すと処理中にずれていくので、処理開始時に取得しておく
CURRENT_TIME = Time.now

messages = kidoku_messages.find_or_initialize_by(line_id: message_ids, user_id: user)
messages.each do |message|
  message.read_time = CURRENT_TIME
  massage.save!
end

Comments

[退会済みユーザー]

はい。配列として受け取るように変更しようと思っていまして、message_idsとして配列として複数の値が入ります。

[mather]

kid 変数を確認してほしいのですが、find_or_initialize_by に配列を渡したので結果も配列になっているのではないでしょうか。

[退会済みユーザー]

はい。結果も配列になっております。

[mather]

kid が配列になっていれば、 kid.read_time には当然アクセスできないですよね。

[退会済みユーザー]

確かに、できませんでした。
今まではデータ1つに絞って処理をしていたのですが、複数のデータをまとめて受け取りまとめて処理しようと思っているのですが、 kid.read_timeにアクセスするのに配列からのeachが使えないとなると他に何かアクセスする方法ありますでしょうか？

[mather]

とりあえずエラーを回避する（はず）の方法を追記しますが、探すべきは「エラーなく実行できる方法」ではなく、「本来の目的を達成できる方法」だと思います。
僕の方では各モデルが何を表現していて何をするためのコードなのかわからないので、現時点では「これでうまくいく」という方法は提示できません。

[退会済みユーザー]

ありがとうございます！