5

我正在编写一个应用程序，允许用户相互发送有关“报价”的消息。

我想我会为自己节省一些工作并使用Mailboxer gem。

我正在使用 RSpec 遵循测试驱动的开发方法。我正在编写一个测试，以确保Conversation每个报价只允许一个。一个要约belongs_to两个不同的用户（提出要约的用户和收到要约的用户）。

这是我失败的测试：

describe "after a message is sent to the same user twice" do
  before do 
    2.times { sending_user.message_user_regarding_offer!  offer, receiving_user, random_string }
  end
  specify { sending_user.mailbox.conversations.count.should == 1 }
end

因此，在测试运行之前，用户sending_user向接收用户发送了两次消息。message_user_regarding_offer!看起来像这样：

def message_user_regarding_offer! offer, receiver, body
    conversation = offer.conversation
    if conversation.nil?
      self.send_message(receiver, body, offer.conversation_subject)
    else  
      self.reply_to_conversation(conversation, body)
      # I put a binding.pry here to examine in console
    end
    offer.create_activity key: PublicActivityKeys.message_received, owner: self, recipient: receiver
end

因此，在测试的第一次迭代中（发送第一条消息时），conversation变量是nil一条消息，并在两个用户之间创建了对话。

在第二次迭代中，返回在第一次迭代中创建的对话，用户回复该对话，但不会创建新对话。

这一切都有效，但是测试失败了，我不明白为什么！

当我在上面指定位置的代码中放置一个 pry 绑定时，我可以检查发生了什么......现在让我困惑：

wtf 红宝石

self.mailbox.conversations[0]返回一个Conversation实例

self.mailbox.conversations[1]返回nil

self.mailbox.conversations清楚地显示了一个包含一个对象的集合。

self.mailbox.conversations.count返回2？！

那里发生了什么？方法不正确，我的count测试失败了......

我错过了什么？或者这是一个错误？！

编辑

offer.conversation看起来像这样：

  def conversation
    Conversation.where({subject: conversation_subject}).last
  end

和offer.conversation_subject：

  def conversation_subject
    "offer-#{self.id}"
  end

编辑 2 - 在 pry 中显示第一次和第二次迭代

wtf 红宝石 2

还...

Conversation.all.count返回 1！

和：

Conversation.all == self.mailbox.conversations返回true

和

Conversation.all.count == self.mailbox.conversations.count返回false

如果数组相等怎么办？我不知道这里发生了什么，现在已经花了好几个小时了。认为这是一个错误？！

编辑 3

从 Mailboxer gem 的来源...

def conversations(options = {})
  conv = Conversation.participant(@messageable)

  if options[:mailbox_type].present?
    case options[:mailbox_type]
    when 'inbox'
      conv = Conversation.inbox(@messageable)
    when 'sentbox'
      conv = Conversation.sentbox(@messageable)
    when 'trash'
      conv = Conversation.trash(@messageable)
    when  'not_trash'
      conv = Conversation.not_trash(@messageable)
    end
  end

  if (options.has_key?(:read) && options[:read]==false) || (options.has_key?(:unread) && options[:unread]==true)
    conv = conv.unread(@messageable)
  end

  conv
end

该reply_to_convesation代码可在此处获得 -> http://rubydoc.info/gems/mailboxer/frames。

只是看不到我做错了什么！可能会修改我的测试来解决这个问题。或者放弃宝石并写我自己的。

4

5 回答 5

9

看到这个Rails 3：Relation.count 和 Relation.all.count 之间的区别

简而言之，select当您对查询应用计数时，Rails 会忽略列（如果多于一个）。这是因为

SQL 的 COUNT 只允许一列或更少的列作为参数。

从Mailbox 代码

 scope :participant, lambda {|participant|
    select('DISTINCT conversations.*').
      where('notifications.type'=> Message.name).
      order("conversations.updated_at DESC").
      joins(:receipts).merge(Receipt.recipient(participant))
  }

self.mailbox.conversations.count忽略select('DISTINCT conversations.*')并用 计算join表格receipts，本质上计算receipts其中有重复conversations的数量。

另一方面，self.mailbox.conversations.all.count首先获取应用 的记录select，它是唯一的conversations，然后对其进行计数。

self.mailbox.conversations.all == self.mailbox.conversations因为他们都使用 select 查询数据库。

要解决您的问题，您可以使用sending_user.mailbox.conversations.all.count或sending_user.mailbox.conversations.group('conversations.id').length

于 2013-09-22T22:31:51.647 回答
3

我倾向于size在我的代码中使用该方法。根据 ActiveRecord 代码，size 将使用缓存的计数（如果可用），并在通过关系创建模型但尚未保存时返回正确的数字。

# File activerecord/lib/active_record/relation.rb, line 228
def size
  loaded? ? @records.length : count
end

这里有一个关于此的博客。

在 Ruby 中，#length 和 #size 是同义词，它们都做同样的事情：它们告诉你数组或散列中有多少元素。从技术上讲，#length 是方法，#size 是它的别名。

在 ActiveRecord 中，有几种方法可以找出关联中有多少条记录，它们的工作方式有一些细微的差别。

post.comments.count - 使用 SQL COUNT 查询确定元素的数量。您还可以指定条件以仅计算关联元素的子集（例如：conditions => {:author_name => "josh"}）。如果您在关联上设置计数器缓存，#count 将返回该缓存值而不是执行新查询。

post.comments.length - 这总是将关联的内容加载到内存中，然后返回加载的元素数。请注意，如果先前已加载关联，然后通过另一种方式创建新评论（例如 Comment.create(...) 而不是 post.comments.create(...)），这不会强制更新。

post.comments.size - 这是前面两个选项的组合。如果集合已经加载，它将返回它的长度，就像调用#length一样。如果还没有加载，就像调用#count。

如果您不是通过关联创建模型，还值得一提的是要小心，因为相关模型不一定在其关联代理/集合中具有这些实例。

# do this
mailbox.conversations.build(attrs)
# or this
mailbox.conversations << Conversation.new(attrs)
# or this
mailbox.conversations.create(attrs)
# or this
mailbox.conversations.create!(attrs)

# NOT this
Conversation.new(mailbox_id: some_id, ....)
于 2013-09-25T05:58:41.947 回答
0

我不知道这是否解释了发生了什么，但 ActiveRecordcount方法会查询数据库以获取存储的记录数。可能会有所不同，如 http://archive.railsforum.com/viewtopic.php?id=6255 中所述，尽管length在该示例中，数据库中的记录数少于Rails 中的项目数数据结构。Relation

于 2013-09-19T23:04:26.303 回答
0

尝试

self.mailbox.conversations.reload; self.mailbox.conversations.count

也许

self.mailbox.reload; self.mailbox.conversations.count

或者，如果这些都不起作用，请尝试reload尽可能多的对象，看看您是否可以让它工作（self, mailbox,conversations等）。

我的猜测是内存和数据库之间出现了问题。不过，这绝对是一个非常奇怪的错误，可能想在 Rails 上提出一个问题，看看为什么会这样。

于 2013-09-21T22:33:25.330 回答
0

的结果mailbox.conversations在第一次调用后被缓存。重新加载它写mailbox.conversations(true)

于 2013-09-23T08:04:55.307 回答