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

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

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

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

Ruby on Rails

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Q&A

解決済

1回答

2481閲覧

whereメソッドの使い方について

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

Ruby on Rails

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

0グッド

1クリップ

投稿2017/08/19 15:05

編集2017/08/19 15:15

###該当のソースコード

def ranking product_ids = Review.group(:product_id).order('count_product_id DESC').limit(5).count(:product_id).keys @ranking = product_ids.map { |id| Product.find(id)} end

reviewsテーブルには作品のレビューが記載されており、カラムにはid,nickname,rate,review,product_idがあります。
productsは作品の一覧になります。
レビューの件数が多い順にproductsテーブルからレコードを取り出したいと思っています。
上記のソースコードであれば取得ができるのですがproduct_ids.map { |idid| Product.where(id: idid)}とするとエラーになってしまいます。なぜこれではダメなのでしょうか?

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

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

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

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

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

guest

回答1

0

ベストアンサー

推測ですが、

ruby

1<% @ranking.each do |ranking| %> 2 <li><%= ranking.name %></li> 3<% end %>

のようにviewで実装していて、そこでエラーが出ていると思います。
findとwhereの違いですが、findはProductモデルそのものが返ってきますので@rankingは

ruby

1@ranking = [Productモデル, Productモデル, Productモデル]

と言うかたちになります。

対してwhereは簡単に言うなら配列が返ってきます(本当は違いますが、配列のようなものです)
なぜならwhereはそもそも結果が何件返ってくるかわからないからです。
ですので1件しかないとわかっていてもプログラムにはわからないので配列のような形になります。ですので、書いているコードだと@rankingは

ruby

1@ranking = [[Productモデル], [Productモデル], [Productモデル]]

みたいな二次元配列になります。
で、[Productモデル].nameのように配列に対してnameを呼び出したりするのでエラーになります。
一つしかないなら

ruby

1@ranking = product_ids.map { |id| Product.find_by(id: id)}

とするか

ruby

1@ranking = product_ids.map { |id| Product.where(id: id).first}

としましょう。

投稿2017/08/19 15:37

chelsy7110

総合スコア596

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

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

退会済みユーザー

退会済みユーザー

2017/08/19 17:56

おっしゃられていました通り<%= ranking.name %>のnameはどういう意味?のエラーが出ていました。 推測していただいてありがとうございます。 ご回答ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問