Railsでのparamsの取り扱いについて

Question

#前提・実現したいこと
ProgateでRailsの学習をしております。postsデータベースからデータを取り出して表示するために、コントローラーの中で@postにデータを代入しました。更にルーティングで:idを設定し、パラメータを受け取れるようにし、それを@idに代入しました。

htmlの方でそれを取り出そうと思い、下記のようなコードを書いてみましたが、エラーになりました。アクセスしたurlは/posts/0です。
試しに、@idの部分を@post[0]で書くと正常に動作しました。

#発生している問題
下記のようなメッセージがブラウザに表示されました。

TypeError in Posts#show

@idが0と同じなら、@idでも同様の結果が出るはずだと考えたのですが、何が違うのでしょうか？

#該当のソースコード

get "posts/:id" =>"posts#show"

  def show
    @post =Post.all
    @id =params[:id]
  end

 <%=@post[@id].content%>

Comments

[fumiya0414]

showのビューにデータを表示させたいのであればコントローラーの記述は@post=Post.allではなく、@posts=Post.find(params[:id])ではないでしょうか？そして@idは渡す必要がない気がします。
そしてビュー（html)側では.each で回して変数postに代入し、post.contentで取り出すべきでないですか？

[tttkkm]

正解はそうだったのですが、もっと単純にできないのかなと思いやってみました。そしてなぜできないのかがわからなかったので質問させていただきました。

Answer 1

@idが0と同じなら、@idでも同様の結果が出るはずだと考えたのですが、何が違うのでしょうか？

id	content
0	1つ目の投稿

この場合、確かにPost.all[id]とPost.find(id)の結果は同じになります。

が、Post.allで取得したレコードの配列が必ずID順であるという保証はなく、
データが作成されたり削除されたりしていく中で毎回レコードの位置とIDが一致するとは限らないため、
Post.all[id]で目的のレコードが取得できるとは限りません。
ここが一番大きな違いになると思います。

<%=@post[@id].content%>

のエラーについてですが、clora さんの回答通り@idが文字列(String型)だから起きています。
配列(今回の場合はActiveRecord_Relationですが)のindex指定は必ず整数(Integer型)でなければいけません。

Comments

[tttkkm]

なぜPost.find(id)を使うのかがわからなかったんですが、そういう理由だったんですね！ありがとうございます！ベストアンサーにさせていただきます！

Answer 2

@id =params[:id]がstring型だからだと思います。
@id.to_iとしてint型に変換してあげるとできると思います。

ただ、fumiya0414さんが言っているように、showではコントローラで@post=Post.find(params[:id])として、ビューでは、@post.contentと記述した方が一般的です。

Comments

[tttkkm]

なるほど！string型だからだったんですね！ありがとうございます！