Rails4 undefined method `each' for 1:Fixnum (NoMethodError)

Question

このエラーはどういう意味ですか？
no_check_itemを確認しましたがレコードが入っているので
動くはずなのに動きません

lib/tasks/item.rb:14:in formation_check': undefined method each' for 1:Fixnum (NoMethodError)

require "#{Rails.root}/app/models/item"
require "#{Rails.root}/app/models/navi"

class Tasks::Item


  def self.formation_check#リミットの指定が悪い可能性

    no_check_item = Item.where(limit_day: Time.mktime(2014, 12, 24, 00, 00, 00).prev_year..Time.now, limit_check: false)

    unless no_check_item.count == 0
      checked_item = no_check_item.update_all({limit_check: true})
      checked_item.each do |item|
        Navi.create(exhibitor: item.user_id,  item_id: item.id, status: 1)
      end
    end
  end


end

#修正後

class Tasks::Item

  def self.formation_check

    no_check_item = Item.where(limit_day: Time.mktime(2014, 12, 24, 00, 00, 00).prev_year..Time.now, limit_check: false)

    unless no_check_item.count == 0
      no_check_item.each do |item|
        Navi.create(exhibitor: item.user_id, item_id: item.id, status: 1)
        item.update({limit_check: true})
      end
    end
  end


end

Answer 1

update_allは更新したデータの行数を返すため、checked_itemがFixnumになっています。

---

追記：

些事ですが

• データ数がゼロなら、eachで回しても問題ありません。
• データが多くなることを考えると、eachよりfind_eachを使う方がよいです。
• 複数のデータを変更するばあいは、transactionで囲った方がよいでしょう。

ですので

# before
unless no_check_item.count == 0
  no_check_item.each do |item|
    Navi.create(exhibitor: item.user_id, item_id: item.id, status: 1)
    item.update({limit_check: true})
  end
end

は、

#after
Item.transaction do
  no_check_item.find_each do |item|
    Navi.create(exhibitor: item.user_id, item_id: item.id, status: 1)
    item.update(limit_check: true)
  end
end

とするのがよいでしょう。

また、これ以降no_checked_itemのインスタンスには触れない（※）のであれば最初のようにupdate_allを使ってもよいと思います。
※）update_allはデータベースを直接変更するので、Railsから見えるno_check_itemの中身はチェックされていない状態が続きます。チェックされた状態のインスタンスが必要であればリロードする必要があります。

Item.transaction do
  no_check_item.find_each do |item|
    Navi.create(exhibitor: item.user_id, item_id: item.id, status: 1)
  end
  no_check_item.update_all(limit_check: true) # 戻り値は読まなくてOK
end

ループ中のcreateのパフォーマンスも気になるようであれば、activerecord-importというgemを利用するのがよいでしょう。

Comments

[smith]

すべてのメソッドで試しましたがうまくいかなかったので
上記のように修正しました。２週間ほど詰まっていて悶絶していましたが
ヒントを頂いたおかげでひらめくことができました。
ありがとうございます！もっといい書き方あれば教えてください

[hello-world]

> もっといい書き方あれば教えてください
一応追記しました。

[smith]

感謝します！