ActiveRecordの結合検索の表示
解決済
回答 1
投稿
- 評価
- クリップ 0
- VIEW 1,450
前提・実現したいこと
RailsでActiveRecord使って複数のテーブルの結合をしようとしております。
多言語のレストランガイドのイメージです。どの言語でも共通のテーブルと言語ごとの情報のファイルを結合しております。
従属する方のテーブルに条件つけて取りたい(英語ならlangはen、日本語ならja)のですが、Viewで取り出す際にエラーになってしまいます。
Associationの設定がダメなのか、取り出し方の問題なのか判別つかないのでご教示いただければ幸いです。
発生している問題・エラーメッセージ
以下は出力されているエラーです。
undefined local variable or method `restaurant'
](affbcadb170ad618afe4c3bc163617cd.png)
該当のソースコード
テーブル定義はこんな感じです。
create_table "restaurant_translations", id: false, force: :cascade do |t|
t.integer "id", limit: 4, default: 0, null: false
t.integer "restaurant_id", limit: 4
t.string "restaurantname", limit: 255
t.string "address", limit: 255
t.string "tel", limit: 255
t.text "description", limit: 65535
t.string "lang", limit: 255, default: "", null: false
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end
create_table "restaurants", force: :cascade do |t|
t.string "restaurant_type", limit: 255
t.string "genre", limit: 255
t.string "url", limit: 255
t.string "fb", limit: 255
t.string "mailaddr", limit: 255
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end
こっちはController。
class RestaurantController < ApplicationController
def list
@restaurants = Restaurant.all
@restaurants = @restaurants.restaurant_translation.find_by(lang: "en")
end
end
こちらはModelファイル。
class Restaurant < ActiveRecord::Base
has_many :restaurant_translations
end
class RestaurantTranslation < ActiveRecord::Base
self.table_name = 'restaurant_translations'
belongs_to :restaurant
end
こちらはViewファイルです。
View(.slim)
table
thead
tr
th = t :restraunt_list_type
th = t :restraunt_list_name
th = t :restraunt_list_url
th = t :restraunt_list_genre
th = t :restraunt_list_addr
tbody
- @restaurants.each do |restaurant|
tr
td = restaurant.restaurant_type
td = restaurant.restaurant_translations.first.restaurantname
td = link_to 'here', restaurant.url
td = restaurant.genre
td = restaurant.restaurant_translations.first.address
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
0
ちょっとエラーの出方が不可解で、どうしてこのエラーが出ているのかよくわかりません。
しかし、listアクションのメソッドの2行目、
@restaurants = @restaurants.restaurant_translation.find_by(lang: "en")
だと、
RestrauntTranslationクラスのインスタンスが@restrauntsに入るような気がします。
おっしゃっている条件付きの関連取得は、おおかた2種類の方法で取得可能かとおもいます。
その1
Viewで取得し直す
tbody
- @restaurants.each do |restaurant|
tr
td = restaurant.restaurant_type
td = restaurant.restaurant_translations.where(lang: "en").first.restaurantname
その2
scopeを使う
class RestaurantTranslation < ActiveRecord::Base
self.table_name = 'restaurant_translations'
belongs_to :restaurant
scope with_lang: ->(lang) { where lang: lang }
end
# view
tbody
- @restaurants.each do |restaurant|
tr
td = restaurant.restaurant_type
td = restaurant.restaurant_translations.with_lang("en").first.restaurantname
こんな感じでしょうか。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう!
- ただいまの回答率 88.35%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
2016/06/13 20:01
2016/06/14 06:05
wrong number of arguments (1 for 2)
と出てしまいました。scopeのところです。何が悪いのでしょうか?見た目引数は合ってそうに見えます・・・。
2016/06/14 08:57
restraurant.restraurant_translaations.with_lang("en")でした。
scopeのlambda式の引数は、langに入れる文字列です。
2016/06/14 10:00
2016/06/14 11:00
間違った情報、すみませんでした。
2016/06/14 15:44