arrayの中から０以上のものを取り出したい。

Question

  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.profit_and_loss
        elsif before_month.include?(Date.parse(note[:created_at].to_s))
          profit_before_month << note.profit_and_loss
        end
      end
      @profit_this_month= profit_this_month.sum
      @profit_this_month_average = @profit_this_month / profit_this_month.length
      @victory_rate_this_month = profit_this_month.where(values > 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(values > 0).count/profit_before_month.count*100

というコードです。

@victory_rate_this_month = profit_this_month.where(values > 0).count/profit_this_month.count*100
@victory_rate_before_month=profit_before_month.where(values > 0).count/profit_before_month.count*100

この部分でprofit_this_monthの中に入っている値の内０以上のものだけ取り出したいです。valuesではうまくいきませんでした。（当然ですが…）どのように条件指定すれば良いでしょうか？？

Comments

[winterboum]

profit_this_month.select{|v| v > 0 }

Answer 1

p profit_this_month.a.select{|e| e >= 0}

で　とりだせませんか？

Answer 2

profit_this_month.select{|v| v > 0 }

nil が入っている可能性があるなら
profit_this_month.select{|v| v.to_i > 0 }

Comments

[ganbarou_nippon]

ありがとうございます！　解決しました。