【问题标题】:Rails undefined method `id'Rails 未定义的方法“id”
【发布时间】:2014-10-05 19:30:40
【问题描述】:

我有两个模型:

电话和结果:

class Phonecall < ActiveRecord::Base
  has_many :results
end

class Result < ActiveRecord::Base
  belongs_to :phonecall
end

一切都很简单,如果我执行此代码:

class CallsController < ApplicationController
  def index
    @phonecalls = Phonecall.order('datetime DESC').all

  end

end

在视野中:

<% @phonecalls.each do |ph| %>
  <%= ph.results.id %>
<% end %>

我有一个错误:

undefined method `id' for #<ActiveRecord::Associations::CollectionProxy []>

我做错了什么?我尝试获取与我的 Phonecall 对象相关的 Result 对象的属性。

【问题讨论】:

  • 结果是一个关系。如果你想要 ids 列表,你可以做 ph.result_ids 或者如果你想要第一个结果的 Id,你可以使用 ph.results.first.id 否则你可以像任何其他集合/关系一样迭代它
  • 你用的是rails 3还是rails 4?
  • 我想用一个 id 来表示电话和结果之间的关系
  • 你可以把 order 写得更漂亮:order(datetime: :desc)。而且您还需要通过使用 include 来避免 N+1 查询。最后它看起来像:@phonecalls = Phonecall.include(:results).order(datetime: :desc).to_a

标签: ruby-on-rails ruby ruby-on-rails-3 ruby-on-rails-4


【解决方案1】:

当你在做 ph.results 它返回一个 ActiveRecord 关系而不是一条记录,所以你不能在它上面调用方法 id。你需要使用一些从这个关系中选择单个结果记录的逻辑。

如果您想显示结果的所有 id,那么您必须遍历所有结果,然后像这样单独显示它们:

- @phonecalls.each do |ph|
  - ph.results.each do |r|
    = r.id

【讨论】:

  • 但是我可以记录一下与当前电话对象有关的情况吗?或者我需要做 .each 循环并控制这条记录 = 我当前的对象关系记录吗?
  • @user2051503 如果要全部显示,则必须遍历集合中存在的所有记录
  • 不,我只想显示与当前 Phonecalls 对象相关的Resultst id
  • 这正是 Mandeep 正在做的事情。试试看吧。
  • @user2051503 当您执行 ph.results 时,它会返回仅属于该特定电话的结果集合。它是记录的集合,因此您不能只显示单个值。您要么必须遍历所有这些,要么使用一些逻辑来选择单个结果,例如 .first 或 .last 但它只会给出第一条或最后一条记录,因此您必须找到该逻辑来选择一个该集合中的单个记录
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2018-04-23
  • 1970-01-01
  • 2014-04-04
相关资源
最近更新 更多