Ruby on Rails Ruby の計算式の記述方法

Question

##実現したいこと
Ruby on Rails　でアプリ制作をしています。
ユーザーに数字を入力してもらい、その数字をもとに以下の計算をして結果を出力したいです。

（最大重量）＝（使用重量）×｛1＋（回数÷40）｝

ユーザーが入力するのは　使用重量、回数 の2つです。
それぞれ、使用重量は　@rm.use_weight　　、回数は　@rm.reps　です。

データベースに保存し値を持ってくることは出来ています。

お手数ですがご教授いただけますでしょうか。
よろしくお願い致します。

##該当のソースコード

以下に試したコードを記載します。
いずれも思う結果が表示出来ませんでした。
（計算式はビュー画面に記述しています）

%p #{((@rm.reps / 40) + 1) * @rm.use_weight .to_f}

以下のコードの場合は　undefined method `+' for nil:NilClass　とエラーが出てしまいます。

%p a = #{@rm.reps / 40 .to_f}
%p b = #{a + 1 .to_f}
%p #{b * @rm.use_weight .to_f}

##補足情報　（ツールのバージョンなど）
MacBook Pro 13
macOs 10.15.4
rails 5.2.3
PostgreSQL. Versions 9.1

Answer 1

viewに ruby 演算を書くのはあまり好きでありません。
この課題のようにモデルの状態そのものを求める場合は特に。

class Rm
  def max_weight
     use_weight * (1 + reps / 40 )
  end
end

しておいて、view では @rm.max_weight で表示する、のが私流です。

Comments

[takeshi075]

ご返信ありがとうございます！
せっかくご返信頂いたのですが、今回の課題については１番目の回答して頂いた方を採用させていただきました。
申し訳ありません。

ちなみに、viewに演算を書くのが良くないのはどういった理由なのでしょうか？
可読性が悪くなる。
後々のメンテナンス性が悪くなる。
などですか？
知識が浅く想像が及ばないので、良ければ教えていただけますでしょうか？

[winterboum]

一番ベースとなるのは、　「その処理はどのobjectが責任を持つべきか」というオブジェクト指向の考え方からです。 最大重量 というのは　コントローラでもviewでもなく、modelのオブジェクトの特性なので。モデルに責任を取ってもらう。
で、なせそういう考え方になったのかというと
　可読性が悪くなる。
　後々のメンテナンス性が悪くなる。
などもその一つです

[takeshi075]

丁寧に解説して頂き、ありがとうございます！
また、何かありましたら、よろしくお願いします！

Answer 2

hamlでしょうか?

hamlの場合、式を分けるには

- a = @rm.reps / 40.0
- b = a + 1
%p #{b * @rm.use_weight}

ただ気になるのがそれぞれの型が数値になっているかです。
@rm.reps.classなどを確認してみるといいでしょう。

Integerならば数値ですが、Stringですと文字列なのでto_iもしくはto_fする必要があります。

Comments

[takeshi075]

ご返信ありがとうございます！
上記のコードで思う様に表示することが出来ました！

ちなみに、Integer　で登録しているので　to_iもしくはto_f　を使用しなくてもOKでした！
今後質問する際はそういった情報も記載する様にします！
ありがとうございました！