質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

239閲覧

ヘルパーメソッド化に直したい

Amagin

総合スコア11

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2022/01/27 04:09

他の記事を参考にメソッド化をしたのですが、インスタンスではなくヘルパーでメソッド化したいです。
理由として、true/falseで返ってくるものではないのでヘルパーのほうが合っているのではないかと言われ、自分で直そうと思いましたがうまく出来ません。
ご教授いただけたらと思います。
※メソッド化はまだまだ不勉強なので参考になる記事などありましたら、とても嬉しいです。

概要

recipeとrecipe_ratyモデルがあり、各レシピの評価の平均値を出そうとしています。
recipe_tasteはrecipe_ratyのカラムです。

model(recipe)

1 def avg_taste ←こちらをヘルパーメソッド化したい 2 if recipe_raties.empty? 3 0 4 else 5 recipe_raties.average(:recipe_taste).round(1) 6 end 7 end

controller(recipe)

1@recipes = Recipe.all

view

1= render partial: "index", collection: @recipes, as: 'recipe'

view(_index)

1= recipe.avg_taste

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

true/falseで返ってくるものではないのでヘルパーのほうが合っているのではないか

その判断基準がおかしいです。モデルのメソッドにするのが妥当と考えます。

投稿2022/01/27 04:14

maisumakun

総合スコア145184

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maisumakun

2022/01/27 04:15

「平均評価点」は、recipeの属性として存在しても何ら不自然ではないデータですので、モデルにメソッドを追加する形で問題ないと考えます。
Amagin

2022/01/27 04:34

早いご回答ありがとうございます。ヘルパーの方が良いと言われたため、判断基準もそういうものだと思っていましたが違うのですね。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問