comparison of Symbol with 0 failedというエラーに対応したいです。

Question

 @victry_times=notes_all.where(:profit_and_loss > 0).group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").count.values.map{|money| money/notes_all.group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").count*100}

というコードに対して

comparison of Symbol with 0 failed

というエラーが出ます。
そもそもこのエラーがどういったものかがわからないのですが、どうすれば解決できますでしょうか？

以下はこの問題に関わる全てのコードです。

def index
    @user=current_user
    @notes=@user.notes.all.order("id DESC").page(params[:page]).per(30)
    notes_all=@user.notes
    @comments=[]
    if notes_all.present?
       @notes.each do |note|
         @comments << note.comments.count
       end
      @profit_symbol=notes_all.group(:symbol).sum(:profit_and_loss)
      profit_month=notes_all.group("MONTH(created_at)")
      @profit_changes = profit_month.sum(:profit_and_loss).values.map{|money| money/notes_all.sum(:profit_and_loss)*100}
      @profit_times=notes_all.group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").sum(:profit_and_loss).values.map{|money| money/notes_all.sum(:profit_and_loss)*100}
      @victry_times=notes_all.where(:profit_and_loss > 0).group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").count.values.map{|money| money/notes_all.group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").count*100}
      @victory_rates= notes_all.where(:profit_and_loss > 0).group("MONTH(created_at)").count.values.map{|money| money/profit_month.count*100}
      this_month = Date.today.all_month
      before_month=Date.today.last_month
      profit_this_month = []
      profit_before_month = []
      notes_all.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)
      @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*100
      @profit_before_month= profit_before_month.sum(:profit_and_loss)
      @profit_before_month_average= @profit_before_month / profit_before_month.length
      @victory_rate_before_month=profit_before_month.where(:profit_and_loss > 0).count/profit_before_month.count*100
    end
  end

Answer 1

そもそもこのエラーがどういったものかがわからないのですが

エラーメッセージのとおりです。シンボルと0を>で比較しようとしたのでエラーとなっています。

どうすれば解決できますでしょうか？

.where(:profit_and_loss > 0)が問題となっています。とりあえず.where('profit_and_loss > 0')と文字列で渡せば回避できます。

Comments

[ganbarou_nippon]

教えていただきありがとうございます。
'''@victry_times=notes_all.where('profit_and_loss > 0').group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").count.values.map{|money| money/notes_all.group("HOUR(CONVERT_TZ(created_at, '+00:00', '+09:00'))").count*100}'''
としたところ
'''Hash can't be coerced into Integer''''
というエラーになり、文字列は受け付けないようなのですが、、どうすれば良いでしょうか？

[maisumakun]

それはおそらく、この文字列指定とは関係ないエラーかと思います。

[ganbarou_nippon]

承知しました！ありがとうございました！