キャラクターを選択すると、そのキャラクターの全ての技が表示されるように実装したい！

Question

#前提
CharactersテーブルとMovesテーブルの二つのテーブルを用意し、Characterｓの主キーをMovesで受け取り、そのidをもとにBにあるデータを全て取得し、表示させる、というような設計にしております。

#試してみたこと
Moveｓテーブルに外部キー（Charactersの主キー）を設定し、保存しています。
それをfindメソッドで取得してeachメソッドで表示させられるのではと思い、記述してみたものの、うまく行きません。
findの引数に数字を入れると取得はできるのですが、これではキャラクターごとに技データを取得することができないです。

こういった場合、
どのように記述できるでしょうか？
また、現時点でのコードでも間違いがありましたらご指摘いただけたらと思います。

-controller/characters

class CharactersController < ApplicationController
  def index
    character = Character.all
  end
end

• views/character/index

<div class = "main">
  <div class="title-contents">
    <h1>フレームデータ</h1>
  </div>
  <div class="main-contents">
    <div class = "select-character">
      <div class = "contents-name">
        キャラクター選択
      </div>
      <div class='characters'>
        <ul>
         <li><%= link_to "キャラクター1", character_moves_path(:character_id) %></li>
         <li><%= link_to "キャラクター2", character_moves_path(:character_id) %></li>
         <li><%= link_to "キャラクター3", character_moves_path(:character_id) %></li>
        </ul>
      </div>
    </div>
  </div>
</div>

• controller/moves

class MovesController < ApplicationController

  def index
    @moves = Move.find(1)
  end
end

• views/move/index

<table>
  <tr>
    <th>技名</th>
    <th>コマンド</th>
    <th>ダメージ</th>
    <th>発生F</th>
    <th>ガードF</th>
  </tr>
  <% @moves.each do |moves| %>
  <tr>
    <td><%= moves.moves_name %></td>
    <td><%= moves.command %></td>
    <td><%= moves.damage %></td>
    <td><%= moves.startup %></td>
    <td><%= moves.block %></td>
  </tr>
  <% end %>
</table>

Answer 1

app/models/character.rb と move.rb が以下のようになっているか確認してください。

class Character < ApplicationRecord
  has_many :moves
end

class Move < ApplicationRecord
  belongs_to :character
end

このとき、id が 1 の Character の moves は以下のようにして参照できます。

Character.find(1).moves # => [move(character_id=1), move(character_id=1), move(character_id=1), ...]

コントローラーでは以下のようになります。

# GET /characters/1
def show
  @character = Character.find(params[:id])
  @moves = @character.moves
end

コードの問題は、

character = Character.all

@characters = Character.all

コントローラーの変数をビューで参照するにはインスタンス変数(@からはじまるやつ)の必要があります。
また、all が返すのはコレクションですので変数名は複数形にしましょう

@moves = Move.find(1)

@move = Move.find(1)

find が返すのは単一のオブジェクトです。変数の名前は単数形にしましょう。
これは each に応答しないので今のビューではエラーになるでしょう。

以下コメントへの返信です

ルーティングを rails の一般的な記述にしている事を前提にしています。

コントローラーが以下のようになっているのなら

def index
  @characters = Character.all
end

次のように書けます。

<ul>
<% @characters.each do |character| %>
  <% # character に name というカラムがあると推定 %>
  <li><%= link_to character.name, character_moves_path(character) %></li>
<% end %>
</ul>

生成されるパスは /charcters/:id/moves です。
次のようなルーティングが設定されていないとエラーになります。
対応するコントローラー、アクションは moves#index です。

resources :characters do
  resources :moves
end

character_path(character) に変更すれば /characters/:id になります。
対応するコントローラー、アクションは characters#show です。

Comments

[hikaru-udon]

レビューありがとうございます！！
アソシエーション確認しました、組めました！
命名規則など、本当に勉強になります！

ご指摘していただいた通りに記述したのですが、
ActiveRecord::RecordNotFound in MovesController#index
Couldn't find Character without an ID
のエラーが出ます。

おそらく画面遷移時にcharacter_idが渡せていないのが原因だということが推測できました。
現在キャラクターの選択をimage_tagを用いたボタン画像で行えるよう実装しているのですが、ここにそれぞれのcharacterのidを持たせるにはどうすれば良いでしょうか？（上記コードの<li>要素のところです）

[hikaru-udon]

ルーティングは提示していただいたものと同じでした！

charactersコントローラでindexを記述し、html.erbを編集し、「link_to インスタンス変数名.name」でキャラクター名を表示できるようになりました。
ですが、クリックして飛ぼうとしたところ、先ほどと同じくエラーが出ました。

エラー文：Couldn't find Character with 'id'=character_id
指摘コード：
def index
@character = Character.find(params[:character_id])
@moves = @character.moves
end

binding.pryで止めて確認したところ、
「<ActionController::Parameters {"controller"=>"moves", "action"=>"index", "character_id"=>"character_id"} permitted: false>」
と出ました。falseが出ている理由がわからないです。
characterのerbかmovesコントローラのどちらかが記述ミスしているという解釈でよろしいのでしょうか？

[neko_daisuki]

link_to character.name, character_moves_path(:character_id) となっていませんか？
:character_id ではなく、 character です。

[neko_daisuki]

xxx_path について
character_path("ここ")　や、character_moves_path("ここ") の ここに指定するのは通常 ID です。
前者では/characters/:id の :id、後者は/characters/:character_id/moves の :character_id の部分になります。

params[:character_id] について
params[:character_id]は URL の一部です。
いまは、/characters/:character_id/moves の:character_idの部分がparams[:character_id]になります。
つまり xxx_path("ここ") で指定した値になります。

find について
find は ID を検索します。見つからないとエラーになります。
params[:character_id] の値は "character_id" になっています。
"character_id" という ID がデータベースに存在しないのでエラーになっています。
通常、ID は数値です。

permitted: falseについて
これはストロングパラメーターと言うやつで、データベースを更新するときのためのやつです。
なので今はfalseで問題ありません。

[hikaru-udon]

なるほど！パスの引数はそういう仕組みなのですね！勉強になります！！
変更点を変更したところ、parameterに{"character_id"=>"1"}と表示されるようになりました！ありがとうございます！！
ですが、「Couldn't find Character without an ID」とでます。。
movesテーブルには外部キーのcharacter_idカラムを作っているので技データをとって来れると思うのですが、このエラー文の場合はそれ以前お問題ですかね？

[neko_daisuki]

@character = Character.find(params[:character_id])
ここでそのエラーが出るのでしょうか？

[hikaru-udon]

そうなのでしたが、すみません、paramsの後のidがcharacter_idにできてませんでした。
修正したところ、無事表示されました！！すごいです！！
教えていただいたパスと引数の仕組みに関してもっと理解を深める必要があると学ぶことができました。
解決のため、たくさんご教授いただき本当にありがとうございます！
とても勉強になりました！