0

Rails と twilio を学習する方法として通話追跡アプリを構築しようとしています。今、私がやりたいことは、特定の電話が毎日受け取る電話の数を示すグラフを作成することです.

私のデータ構造が設定されている方法は、電話モデルが has_many 呼び出しであることです。

私の電話モデルでは、私は持っています

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 が、上で参照したインスタンス メソッドを作成するのを手伝ってくれたので、それは私が書いたものよりもはるかに優れていると思います。

では、ビューで特定の日付に行われた電話の数を呼び出すことができるようにするには、何を変更する必要がありますか?

アップデート

ザックの答えの後、エラーが発生しました

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

>

4

1 に答える 1

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

アップデート:

元の「placed_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
于 2012-10-11T18:00:03.123 に答える