未定义的方法`to_f'用于#<activerecord :: relation:0x472d0a0> </activerecord :: relation:0x472d0a0>

时间:2012-10-11 17:54:28

标签: ruby-on-rails ruby

我正在尝试构建一个呼叫跟踪应用程序,以此来学习rails和twilio。现在,我想做的是制作一个图表,显示每天特定手机的电话数量。

我的数据结构设置方式是手机型号有多次调用。

在我的手机型号中,我有

def total_on(date)
  self.calls.where("placed_at >= ? AND placed_at <= ?", date.beginning_of_day, date.end_of_day)
end

在我看来,我有以下javascript:

<script type="text/javascript" charset="utf-8">
  $(function () {
    new Highcharts.Chart({
      chart: { renderTo: 'calls_chart' },
      title: { text: 'Calls Per Day' },
      xAxis: { type: 'datetime' },
      yAxis: {
        title: { text: ' Calls'}
      },
      series: [{
        name: "<%= @phone.name %>",
        pointInterval: <%= 6.days * 1000 %>,
        pointStart: <%= 1.month.ago.at_midnight.to_i * 1000 %>,
        data: <%= (1.month.ago.to_date..Date.today).map { |date| @phone.total_on(date).to_f}.inspect %>
      }]
    });
  });

重要的部分是数据:

<%= (1.month.ago.to_date..Date.today).map { |date| @phone.total_on(date).to_f}.inspect %>

当我尝试每天拨打电话数据时,我收到错误

undefined method `to_f' for #<ActiveRecord::Relation:0x472d0a0>

说实话,我不确定我是否在这里做任何事情。 stackoverflower的伙伴帮我制作了上面引用的Instance Method,所以我假设它比我写的要好很多。

那么,在视图中,我需要更改哪些内容才能拨打特定日期的电话号码?

更新

在Zack的回答之后,我现在得到了错误

SQLite3::SQLException: no such column: calls.placed_at: SELECT COUNT(*) FROM "calls"  WHERE "calls"."phone_id" = 44 AND ("calls"."placed_at" BETWEEN '2012-09-11 00:00:00.000000' AND '2012-09-11 23:59:59.999999')

可能我混淆了我所拥有的关联或其他东西。以下是具有数据结构的模型,如果它们有帮助 -

手机型号

# == Schema Information
#
# Table name: phones
#
#  id              :integer          not null, primary key
#  name            :string(255)
#  twilio_number   :integer
#  original_number :integer
#  user_id         :integer
#  created_at      :datetime         not null
#  updated_at      :datetime         not null
#

class Phone < ActiveRecord::Base

  attr_accessible :original_number, :user_id, :name, :twilio_number
  belongs_to :user
  has_many :calls, dependent: :destroy

  validates :name, presence: true
  validates :twilio_number, presence: true
  validates :original_number, presence: true
  validates :user_id, presence: true

  default_scope order: 'phones.created_at DESC'

  validate :check_phone_limit, :on => :create

  def check_phone_limit
    if User.find(self.user_id).at_max_phone_limit?
      self.errors[:base] << "Cannot add any more phones"
    end
  end

  def original_number=(value)
    num = value.to_s.gsub(/[^0-9+]/, "")
    write_attribute(:original_number, num.to_i)
  end

def total_on(date)
  self.calls.where(placed_at: date.beginning_of_day..date.end_of_day).count
end

end

致电型号

# == Schema Information
#
# Table name: calls
#
#  id               :integer          not null, primary key
#  AccountSid       :string(255)
#  From             :string(255)
#  To               :string(255)
#  CallStatus       :string(255)
#  ApiVersion       :string(255)
#  Direction        :string(255)
#  FromCity         :string(255)
#  FromState        :string(255)
#  FromZip          :string(255)
#  FromCountry      :string(255)
#  ToCity           :string(255)
#  ToState          :string(255)
#  ToZip            :string(255)
#  ToCountry        :string(255)
#  CallSid          :string(255)
#  DialCallSid      :string(255)
#  DialCallDuration :string(255)
#  DialCallStatus   :string(255)
#  RecordingUrl     :string(255)
#  phone_id         :integer
#  DialCallMinutes  :integer
#  created_at       :datetime
#  updated_at       :datetime
#

class Call < ActiveRecord::Base
  attr_accessible :AccountSid, :From, :To, :CallStatus, :ApiVersion, :Direction, :FromCity, :FromState, :FromZip, :FromCountry, :ToCity, :ToState, :ToZip, :ToCountry, :CallSid, :DialCallSid, :DialCallDuration, :DialCallStatus, :RecordingUrl, :DialCallMinutes
  belongs_to :phone


  def self.create_from_incoming_call(params)

   user_phone = Phone.find_by_twilio_number(params['To']) #Finds the phone number in the database based on what phone Twilio is calling

    twilio_request_params = {
      :CallSid => params['CallSid'],
      :AccountSid => params['AccountSid'],
      :From => params['From'],
      :To => params['To'],
      :CallStatus => params['CallStatus'],
      :ApiVersion => params['ApiVersion'],
      :Direction => params['Direction'],
        :FromCity => params['FromCity'],
        :FromState => params['FromState'],
      :FromZip => params['FromZip'],
      :FromCountry => params['FromCountry'],
      :ToCity => params['ToCity'],
      :ToState => params['ToState'],
      :ToZip => params['ToZip'],
      :ToCountry => params['ToCountry']


    }

    call = Call.new(twilio_request_params)
    call.save  
    return call

  end

  def Call.update_dial_call(params)

    twilio_request_params = {
        :DialCallSid => params['DialCallSid'],
        :DialCallDuration => params['DialCallDuration'],
        :DialCallStatus => params['DialCallStatus'],
        :RecordingUrl => params['RecordingUrl'],
      :DialCallMinutes => (params['DialCallDuration'].to_f/60.to_f).ceil
    }

    call = Call.where( :CallSid => params['CallSid'] ).first
    call.update_attributes twilio_request_params
    call.save

  end


end

电话显示操作

  def show
  @user = current_user
  @phone = Phone.find_by_id(params[:id])
  @calls = @phone.calls.paginate(page: params[:page])
  end

并且,视图中的javascript(重复以便于阅读)

<script type="text/javascript" charset="utf-8">
  $(function () {
    new Highcharts.Chart({
      chart: { renderTo: 'calls_chart' },
      title: { text: 'Calls Per Day' },
      xAxis: { type: 'datetime' },
      yAxis: {
        title: { text: ' Calls'}
      },
      series: [{
        name: "<%= @phone.name %>",
        pointInterval: <%= 6.days * 1000 %>,
        pointStart: <%= 1.month.ago.at_midnight.to_i * 1000 %>,
        data: <%= (1.month.ago.to_date..Date.today).map { |date| @phone.total_on(date).to_f}.inspect %>
      }]
    });
  });
</script

&GT;

1 个答案:

答案 0 :(得分:2)

您的total_on方法正在返回关联,而不是数字。

尝试追加.count

def total_on(date)
  calls.where("placed_at >= ? AND placed_at <= ?", date.beginning_of_day, date.end_of_day).count
end

你也可以给它一个日期范围来缩短方法(实际上相同,但更容易阅读):

def total_on(date)
  calls.where(placed_at: date.beginning_of_day..date.end_of_day).count
end

<强>更新

原始的“placement_at”查询是否有效,或者由于NoMethodError而未实际调用过它?

您的模型中似乎没有placed_at列,但我认为您也可以使用created_at

如果您想使用placed_at,可以定义此方法(我将名称更改为placed_on的样式):

class Call < ActiveRecord::Base

  def self.placed_on(date)
    where(created_at: date.beginning_of_day..date.end_of_day)
  end

end

您可以将其链接到total_on方法:

class Phone < ActiveRecord::Base

  def total_on(date)
    calls.placed_on(date).count
  end

end