外部キーを設定した他のモデルのデータをviewに表示する方法[showアクション]

Question

前提・実現したいこと

料理のレシピを管理するアプリを作っています。cookモデルを親、materialモデルを子としています。二つを紐付けてデータを登録するところまでは出来たのですが、showで表示する際にエラーが発生して詰まっています。

発生している問題・エラーメッセージ

ActiveRecord::RecordNotFound in CookController#show
Couldn't find Material without an ID

該当のソースコード

def new
  @cook = Cook.new
  @cook.materials.build
  @cook.works.build
end

def show
  @cook = Cook.find(params[:id])
  @materials = Material.find(params[:cook_id])
end

= @cook.title
= @cook.description
= @materials.material1

class CreateMaterials < ActiveRecord::Migration[5.2]
  def change
    create_table :materials do |t|
      t.string :material1
      t.string :material2
      t.string :material3
      t.string :material4
      t.string :material5
      t.string :material6
      t.string :material7
      t.string :material8
      t.string :material9
      t.string :material10
      t.string :material11
      t.string :material12
      t.string :material13
      t.string :material14
      t.string :material15
      t.string :material16
      t.string :material17
      t.string :material18
      t.string :material19
      t.string :material20
      
      t.references :cook, index: true, foreign_key: true

      t.timestamps
    end
  end
end

試したこと

@materials = Material.find(params[:cook_id])を消すと@cookのtitleとdescriptionはちゃんと表示されます。また、@cookから直接取り出せるのではないかと思い、@cook.material1などとviewに書いてみましたがダメでした。

補足情報（FW/ツールのバージョンなど）

追加で補足していきます。

Answer 1

…すいません、あってるかどうかわかりませんが…。
私も非エンジニアのトウシロウですので…。

まず、今回のエラー、解消自体は簡単です。以下を修正してください。

def show
  @cook = Cook.find(params[:id])
  @materials = Material.find_by(cook_id: params[:cook_id])
end

これだけです。変更点はfindをfind_byにしただけですね。
重要なのは**findメソッドとfind_byメソッドの役割の違いです。**
ご自身で調べたほうがいいと思いますのであえてここでは記述しませんが、カッコの中、引数の部分の記述が違いますね。これによってこの2つのメソッドは全く別の振る舞いをしてくれるわけです。
これでrauraujiさんの今回の問題は解決ですね、表面上は。

ただ今回のお話、もっと重要な問題が隠れています。
結論を申し上げます。
Active Recordのシステム、特に関連付けについて大きな誤解をされています。

…ごめんなさい。私もこの部分の理解には相応の時間を要しましたので、非難するつもりは全くありません。ただ、ここは明確にしておかないと沼から抜けられない用に思いました（経験者談）ので…。平にご容赦ください。

とりあえず、「動いてほしい多分あってると信じたい苦しいときの神頼みコード」は以下のような形になると思います。

class CreateMaterials < ActiveRecord::Migration[5.2]
  def change
    create_table :materials do |t|
      t.string :material_name, null: false, default: ""
      t.integer :procedure, null: false, default: 0

      t.references :cook, foreign_key: true

      t.timestamps
    end
  end
end

class Cook < ApplicationRecord
  has_many :materials
end

class Material < ApplicationRecord
  belongs_to :materials
end

def show
  @cook = Cook.find(params[:id])
  @materials = @cook.materials.order("procedure ASC")
end

おそらく何が書いてあるのかイメージがしにくいと思います。
そんなrauraujiさんには**Railsガイドを読み込むことを強く推奨します。**
また、ここで触れた内容は**Active Record の関連付け**という部分に詳細な説明が掲載されています。
こんなポンコツの説明と比較するのもおこがましい、明瞭な記述です。
感覚的にまとめるなら、どのデータがどのデータにぶら下がっているのかっていうイメージを必要なところに書こうねといったところでしょうか。

例えば今回であれば、料理のレシピはたくさんの手順を持っていますよね。ですから、このイメージを文章にするとCook has many materials. また、Materials belongs to cook. になるわけです。
modelを見てみるとそのイメージが書いてありますよね。
また、こんな作りにするとfindとかfind_byはいらなくなって、whereが日の目を見ることになります。
が、これはまた別のお話…ということで。

さて、ここまでの説明で薄々お気づきとは思いますが、**外部キー制約云々はこのあたりの関連性を理解した後のお話で、今回の問題とは無関係です。**やらないほうがいいですが、外部キーってなくても動きますし…。

ということで、まずはActive Recordを見直そうというお話でした。
長々と書いてますが、所詮は素人に毛が生えたような人間の戯言です。
情報の正確性は全く担保できませんので、必ずご自身で情報収集をお願いします。
rauraujiさんのお力になれていれば幸いです。
長文失礼しました。

追伸：Ruby及びRailsガチ勢の皆様。浅瀬にも入らず砂山を作っているような小生の発言、問題がありましたらぜひ、ボッコボコのタコ殴りにし糾弾した上で、修正いただけますと幸いです。よろしくお願いいたします。

Comments

[raurauji]

とても丁寧に回答していただきありがとうございます！！
material.rbとcook.rbには一応アソシエーションを書いていたのですが、これがどこで役に立つものかをあまり意識せずに外部キーだけで関連付けさせようとしていたみたいです。
少しばかりですが、ActiveRecordに関する知識が深まった気がします。今後もActiveRecordの理解に注力していこうと思います！！

[raurauji]

一つ質問なのですが、外部キーを使わずにwhereでデータを取得する際は、どのような記述になるのでしょうか？

[take77]

取得の方法自体は変わりません。
外部キー制約というのは、そもそも「データベースの安全性を担保するための制約」ですので…。
データベース（railsの場合はmigration）のindexの貼り方を変更すれば制約は外れます。
ただし、「親のidが取得できないことによる謎のエラー」的なものが発生したり色々と問題が出る…んだったと思います。
違ってたらすいません…

[raurauji]

なるほど！丁寧に説明していただき本当にありがとうございました！！