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

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

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

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

Q&A

解決済

1回答

139閲覧

Rails APIが思ったような挙動になりません

Khy

総合スコア118

Ruby on Rails

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

0グッド

0クリップ

投稿2020/01/01 08:44

RailsでAPIを作成しております(Rails初心者です)。

ProductテーブルとReviewテーブルの2つのテーブルがあり、1対多の関係になっております。(1つの製品に複数のレビューがある)

http://localhost:3000/api/v1/products/【product_id】/reviewsで製品ごとのレビューを取得したいのですが、【product_id】の値に関わらず全てのレビューが帰ってきてしまいます。

http://localhost:3000/api/v1/products/1/reviewsの時もhttp://localhost:3000/api/v1/products/2/reviewsの時もどちらも全てのReviewテーブルの値が帰ってきてしまう状態です。

routes.rbproducts_controller.rbreviews_controller.rbproduct.rbreview.rbは以下のようになっているのですが、製品ごとに関連するレビューだけを取得できるようにするにはどうしたらいいのでしょうか?

rb

1# config/routes.rb 2 3Rails.application.routes.draw do 4 namespace :api do 5 namespace :v1 do 6 resources :products do 7 resources :reviews 8 end 9 end 10 end 11end

rb

1# controllers/products_controller.rb 2 3module Api 4 module V1 5 class ProductsController < ApplicationController 6 before_action :set_product, only:[:show, :update, :destroy] 7 8 def index 9 products = Product.order(created_at: :desc) 10 render json: { status: 'SUCCESS', message: 'Loaded products', data: products} 11 end 12 13 def show 14 render json: { status: 'SUCCESS', message: 'Loaded products', data: @product} 15 end 16 17 def create 18 product = Product.new(product_params) 19 if product.save 20 render json: { status: 'SUCCESS', data: product} 21 else 22 render json: { status: 'ERROR', data: product.errors} 23 end 24 end 25 26 def destroy 27 @product.destroy 28 render json: { status: 'SUCCESS', message: 'Deleted the product', data: @product} 29 end 30 31 def update 32 if @product.update(product_params) 33 render json: { status: 'SUCCESS', message: 'Updated the product', data: @product} 34 else 35 render json: { status: 'SUCCESS', message: 'Not updated', data: @product.error} 36 end 37 end 38 39 private 40 41 def set_product 42 @product = Product.find(params[:id]) 43 end 44 45 def product_params 46 params.require(:product).permit(:name) 47 end 48 end 49 end 50end 51

rb

1# controllers/reviews_controller 2 3module Api 4 module V1 5 class ReviewsController < ApplicationController 6 before_action :set_review, only:[:show, :update, :destroy] 7 8 def index 9 reviews = Review.order(created_at: :desc) 10 render json: { status: 'SUCCESS', message: 'Loaded reviews', data: reviews} 11 end 12 13 def show 14 render json: { status: 'SUCCESS', message: 'Loaded reviews', data: @review} 15 end 16 17 def create 18 review = Review.new(review_params) 19 if review.save 20 render json: { status: 'SUCCESS', data: review} 21 else 22 render json: { status: 'ERROR', data: review.errors} 23 end 24 end 25 26 def destroy 27 @review.destroy 28 render json: { status: 'SUCCESS', message: 'Deleted the review', data: @review} 29 end 30 31 def update 32 if @review.update(review_params) 33 render json: { status: 'SUCCESS', message: 'Updated the review', data: @review} 34 else 35 render json: { status: 'SUCCESS', message: 'Not updated', data: @review.error} 36 end 37 end 38 39 private 40 41 def set_review 42 @review = Review.find(params[:id]) 43 end 44 45 def review_params 46 params.require(:review).permit(:title, :description, :product_id) 47 end 48 end 49 end 50end 51

rb

1# models/product.rb 2 3class Product < ApplicationRecord 4 has_many :review, dependent: :destroy 5end 6

rb

1# models/review.rb 2 3class Review < ApplicationRecord 4 belongs_to :product 5end 6

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

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

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

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

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

guest

回答1

0

ベストアンサー

ruby

1# Api::V1::ReviewsController#index内 2reviews = Review.order(created_at: :desc)

コードに書いてあるとおり、ここではProductによる絞り込みは全く行われていません。Review.where(product_id: params[:product_id])のように絞り込みを行ってください。

投稿2020/01/01 08:49

maisumakun

総合スコア145121

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

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

Khy

2020/01/01 08:55

ご回答ありがとうございます。 お陰様で理解し、解決することができました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問