1

为了避免使最后一个问题充满编辑,我将提出一个关于调试的新问题。最初的问题是这个Neo4j gem - Plucking multiple nodes/relationships

这就是我结束的地方。当天检测存在一些缺陷,但作为查询,它现在确实有效。@collection 返回大量书面内容。

事件.rb

def self.reminder

    one_day = 1.day.to_i
    time = Time.zone.now.to_i
    @collection = Event.as(:e).where( "( ( e.date_start - {current_time} ) / {one_day_p} ) < {one_p} " ).users(:u, :rel).where(setting_reminder: true).rel_where(reminded: false ).params(one_day_p: one_day, current_time: time, one_p: 1).pluck(:e, 'COLLECT(u)', 'COLLECT(rel)')

    @collection.each do |event, users, rels|
      users.each_with_index do |user, i|
        UserMailer.reminder(event,user[i]).deliver
      end

      rels.each_with_index do |rels, i|
        rels[i].reminded = true
      end
    end
end

我最初和最后一个答案一样拥有它,但我认为我需要跟踪用户的索引和嵌套在每个块中的 rel 。

在rails c中运行,当我运行时Event.reminder我得到

TypeError: 0 is not a symbol

我的嵌套循环有什么问题?

4

2 回答 2

1

@collection是一个包含其他数组的大数组。你不能这样做@collection.each do |event, users, rels|,你需要返回其中的每个数组,然后遍历它们。两种方式:

@collection = Event.as(:e).where( "( ( e.date_start - {current_time} ) / {one_day_p} ) < {one_p} " ).users(:u, :rel).where(setting_reminder: true).rel_where(reminded: false ).params(one_day_p: one_day, current_time: time, one_p: 1).pluck(:e, 'COLLECT(u)', 'COLLECT(rel)')
@collection.each do |row|
  event = row[0]
  users = row[1]
  rels  = row[2]

  users.each_with_index do |user, i|
    UserMailer.reminder(event, user).deliver
    rels[i].reminded = true
    rels[i].save
  end
end

# OR

events = collection.map { |row| row[0] }
users  = collection.map { |row| row[1] }
rels   = collection.map { |row| row[2] }

events.each_with_index do |event, i|
  UserMailer.reminder(event, users[i]).deliver
  rels[i].reminded = true
  rels[i].save
end
于 2014-12-18T15:18:43.103 回答
0

我认为你只需要做一个正常的每个:

  users.each do |user|
    UserMailer.reminder(event,user).deliver
  end
于 2014-12-18T09:21:41.010 回答