undefined method `/' for [2, 20001]:Arraynita

Question

undefined method `/' for [2, 20001]:Array

@profit_changes << profit_month/@user.notes.sum(:profit_and_loss)*100

下の部分で上のようなエラーが起きています。

これはなぜなのでしょうか？

データが存在しない場合は分岐で避けているはずなのですが…割り算が使えないです。。

以下はindexアクション全てのコードです。

対策がわかる方ご教授の方よろしくお願い申し上げます。

class UsersController < ApplicationController
  
  def index
    @user=current_user
    @notes=@user.notes.all.order("id DESC").page(params[:page]).per(30)
    @comments=[]
    if @user.notes.present?
      @notes.each do |note|
        @comments << note.comments.count
      end
    end
    @profit_symbol=@user.notes.group(:symbol).sum(:profit_and_loss)
    profit_month=@user.notes.group("MONTH(created_at)")
    @profit_changes=[]
    if @user.notes.present?
      profit_month.sum(:profit_and_loss).each do |profit_month|
        @profit_changes << profit_month/@user.notes.sum(:profit_and_loss)*100
      end
    end
    @profit_changes_symbol=[]
    if @user.notes.present?
      @user.notes.group(:symbol,"MONTH(created_at)").each do |profit_changes_symbol|
        @profit_changes_symbol << (profit_changes_symbol.sum(:profit_and_loss)/@profit_symbol*100)
      end
    end
    @victory_rates=[]
    if @user.notes.present?
      @user.notes.where(:profit_and_loss > 0).group("MONTH(created_at)").each do |victory_rates|
        @victory_rates << (victory_rates.count/profit_month.count)
      end
    end
    this_month = Date.today.all_month
    before_month=Date.today.last_month
    profit_this_month = []
    profit_before_month = []
    @user.notes.each do |note| 
      if this_month.include?(Date.parse(note[:created_at].to_s))
        profit_this_month << note
      elsif before_month.include?(Date.parse(note[:created_at].to_s))
         profit_before_month << note
      end
    end
    @profit_this_month= profit_this_month.sum(:profit_and_loss)
    if @user.notes.present?
      @profit_this_month_average = @profit_this_month / profit_this_month.length
      @victory_rate_this_month = profit_this_month.where(:profit_and_loss > 0).count/profit_this_month.count
    end
    @profit_before_month= profit_before_month.sum(:profit_and_loss)
    if @user.notes.present?
      @profit_before_month_average= @profit_before_month.to_i / profit_before_month.length
      @victory_rate_before_month=profit_before_month.where(:profit_and_loss > 0).count/profit_before_month.count
    end
  end

Answer 1

これはなぜなのでしょうか？

エラーメッセージのとおりです。profit_monthの値が配列なので、そのままでは計算できません。

Comments

[ganbarou_nippon]

配列なのでそれをeachメソッドで一つずつ取り出して、上で宣言した殻の配列に入れているはずなのですが…それでも計算できませんか？

[maisumakun]

> 配列なのでそれをeachメソッドで一つずつ取り出して、上で宣言した殻の配列に入れているはずなのですが

そうなっていないからエラーになっています。

[taichi730]

`profit_month.sum(:profit_and_loss).each`
が意図した記述でないと予想

[ganbarou_nippon]

'''profit_month.sum(:profit_and_loss).each do |profit_month|
@profit_changes << profit_month/@user.notes.sum(:profit_and_loss)*100
end'''
この部分が配列になっていないということですよね？？
上で月ごとにグループ化したものを'''profit_month'''という変数に入れて、そして月ごとに出した'''（:profit_and_loss）'''の合計を全ての合計で割って、その月ごとの値を'''@profit_changes'''に入れたいのですがどうすれば良いでしょうか？？

[maisumakun]

> この部分が配列になっていないということですよね？？

そうですね、group化したものにsumをかけているので、キーが月のハッシュで来ているかと思います。

[ganbarou_nippon]

なるほど。。。承知しました！教えていただきありがとうございます。

もう一つ質問があるのですが、
'''@user.notes.group(:symbol,"MONTH(created_at)").sum(:profit_and_loss).values.each do |profit_changes_symbol|'''
のように二つの要素でグループ化している場合、:symbolごとに月単位(MONTH(created_at))の値に対して
"""@profit_changes_symbol << profit_changes_symbol/@profit_symbol*100""""
という処理を行いたいです。

その際にHash can't be coerced into Integerというエラーが出てしまうのですが、どうすれば良いでしょうか？

この時、:symbol、と"MONTH(created_at)"のインデックスをkeysで別々に取得して、それぞれを別の配列に入れたいのですがどうすれば良いでしょうか？？

何度もすみません。。