DBカラム同士での計算方法

Question

前提・実現したいこと

Railsで簡単な燃費計算ができるアプリを作っています。
ユーザーが入力した情報は「infomationテーブル」に保存されている。

ユーザーが走行距離（range）と給油量（refuel）を入力したら、自動的に燃費（fuel_economy）が計算され,
それぞれのカラムに保存されるようにしたい。

:fuel_economy = :range / :refuel
ができるようにしたい。

発生している問題・エラーメッセージ

どこで計算させればいいか調べたがわからず、記載方法もわからない。
コントローラー、モデル？

試したこと

最初は、コントローラーに記載し、

private
  def infomation_params
　　:fuel_economy = :range / :refuel
    params.require(:infomation).permit(:refuel, :cost, :refuelday, :range, :fueltype_id, :fuel_economy).merge(user_id: current_user.id)
  end

としたが、エラーがでた。

現状はテーブルのfuel_economyではなく、view（〇〇.html.haml）上で、

• fuel_economy = range / refuel

を計算させ表示させている。
しかしながら、これでは将来改修など行った時に不都合になる。

Answer 1

ユーザーが走行距離（range）と給油量（refuel）を入力したら、自動的に燃費（fuel_economy）が計算され,

それぞれのカラムに保存されるようにしたい。

燃費自体を集計関数にかけたいなどの扱いをする必要があるなら別ですが、そうでないなら「他の列から算出できる値」をデータベースに保存する必要はありません。

他の列から算出可能な値を保存してしまうと、「データ容量を不必要に消費する」「矛盾するデータを投入できてしまう」など、そちらのほうが不都合です。

Comments

[maisumakun]

> テーブルのfuel_economyではなく、view（〇〇.html.haml）上で

モデルに計算用メソッドを作る、ではだめでしょうか？

[mk12-8078]

燃費を集計関数にかけ、最小値、最大値を表記したり、グラフ化できるようにする必要があるため、データベースに保存する必要があります。
今は、フロントのイメージを掴むためにviewで計算させ表示させていますが、誤魔化している状態です。

Answer 2

infomation_paramsでやる場合
:fuel_economy = :range / :refuel
⇓
params[:infomation][:infomation] = params[:infomation][:range].to_f/params[:infomation][:refuel].to_f
見苦しいし、refuelがnilやゼロの時の対策が厄介

def create でやる場合
@information = Information.new(infomation_params) の後
@information.fuel_economy = @information.range / @information.refuel if @information.refuel > 0

だがこういうことはmodelでやらせたい

before_save :set_fuel_economy
def set_fuel_economy
    fuel_economy = range / refuel if refuel > 0
end

Comments

[mk12-8078]

連絡ありがとうございます。
modelにコードを打ちましたが、値が入力されていませんでした。
また、def create　でもやってみましたが、"undefined method `refuel' for nil:NilClass"と表示されてしまい、保存そのものができませんでした。
ご教授いただいたコード以外に入力しないといけないコードが存在するのでしょうか？

Answer 3

データベースへの保存は諦め、
array = []
で計算結果を収納し、
array.max
array.min
で取得することにしました。