railsを利用してWEBサービスを作っています。モデルにメソッドを書いて、そのメソッドをviewに呼び出すところで詰まっています。

現在、口コミサイトを制作しています。

商品を登録し、その商品に点数評価とコメントを残していく機能です

Aという商品に３つ評価がつき、それぞれの点数が３点、４点、５点だったとします。

全ての評価を足して、その評価の数だけ割った数値を表示させたいです。

この場合、３＋４＋５＝１２　　１２/3＝４です

Itemモデルに

def self.total_taste
  item = Item.find(params[:id])
  comments = item.comment
  @sum = 0
  comments.each do |a|
    @sum += (a.value.round(1))/comments.count.to_i
  end
  return @sum
end

item.show.erbに

<% Item.total_taste %>

このように書いたところ

undefined local variable or method `params'

このようなエラーが出ます

URLにparamsのidが表示されるのですが、modelのメソッドはそこを読んでくれないのでしょうか？

解決策を教えて欲しいです

行動規範の内容に同意します

回答2件

def self.total_taste
ですと、Item全体を相手にしてしまいます。

item.show.erb　ということは　@item とかを表示するようになっていると思うのでこういう場合は
<%= @item.total_taste %> とし(=忘れないでね）
def total_taste return unless comments.size > 0 (comments.inject(0.0){|s,comment| s+comment.value }/comments.size).round(1) end
などと。
ただし、comments を使うには has_many :commentsが定義されてること。

投稿2020/04/24 07:08

編集2020/04/24 07:09

winterboum

総合スコア23567

mnhy613

2020/04/24 11:48

ご回答ありがとうございます。 def total_taste return unless comments.size > 0 (comments.inject(0.0){|s,comment| s+comment.value }/comments.size).round(1) end このコードはモデルに書くのですか？それともviewに書くのでしょうか？

winterboum

2020/04/24 12:06

viewでは普通methodは書きません。書けないこともないなぁ、と今気が付きました。helperに書くのとどう違うだろう。。。まぁ、読みにくいviewになることは間違いなさそう。 modelに書いてください

行動規範の内容に同意します

ベストアンサー

paramsを利用できるのはコントローラのアクション内だけです。
ここでは単にidを引数で受け取れる事が出来れば良いので、メソッドに引数を追加します。
また、find(id)ではレコードがない時に例外が発生してしまうので、
find_by()を使ってレコードが見つからない場合は例外ではなくnilを受け取れるようにしたほうが良いと思います。

ruby
1def self.total_taste(id)
2  item = Item.find_by(id: id)
3
4  # レコードが見つからない場合は規定値を返す
5  unless item
6    return 0 
7  end
8
9  comments = item.comments
10  sum = 0
11  comments.each do |a|
12    sum += (a.value.round(1))/comments.count.to_i
13  end
14
15  return sum
16end