Ruby on Rails5でのimgを使った画像表示について

Question

Ruby on Rails5でのimgを使った画像表示について質問です。

シンプルな投稿サイトを作っている際に、ユーザー登録で画像も一緒に表示させようとしたのですが、上手くいきません。
画像は以下のディレクトリに保存しています。
public/user_images/default_user.jpg

それを下記2ページに表示させたいとします。
app/views/users/index.html.erb

<img src="<%= "/user_images/#{user.image_name}" %>">

app/views/users/show.html.erb

<img src="<%= "/user_images/#{@user.image_name}" %>">

コントローラーはこちらです
app/controllers/users_controller.rb

def create
  @user = User.new(image_name: "default_user.jpg")

上記の例にて、show.html.erbでは「@user.image_name」でエラーが出ないのですが、index.html.erbでは「user.image_name」でないとエラーが出てしまいます。
私としてはどちらも@userが正解に見えるのですが...

すみませんが、なぜこのような違いが出るのか、ご存知の方ご教授お願いします。

Comments

[bamboo-nova]

全体を見ていないので分かりませんが、indexのビューではeach doの構文が入っていて、
```
<% @users.each do |user| %>
```
とか入れているパターンとかありませんか？

[winterboum]

質問文から見て大体の予想はつくのですが、思い込みがあると嫌なのでcodeを追加してください。
今回は Userについての、indexとshowでの問題ですので、
users_controller.rb と users/index.html.*, users/show.html.* の３つ

Answer 1

まだ未解決でしょうか？解決済でしたら読み飛ばしてください。
ソースが追加されるのを待ってるうちに忘れてました。ソースがないので、「多分こうなってるだろう」という予測で書きます。

まず、@userですが、「@userと書くと、必要なときに期待するデータが手に入る万能の召喚呪文」ではありません。使う前に予め必要な値を代入しておく必要があります。その意味では @userもuserも違いはありません。
違いは二つ。
userは、未定義で使うと Undefinedの例外で止まります。またそれが定義されたmethodの中でのみ有効です。
@userは未定義の場合 nil として扱われるます。またそのクラス（のそのインスタンス）の中ならどのmethodでも使えます。
加えてcontrollerで定義した時は「そのアクションで呼びだされた」viewでも使えます。

ですから @userが正しいのかuserが正しいのか、は「どちらに必要な値を代入してあるか」で決まります。

view show では controllerのshowで@user=User.find(params[:id])としていると思います。ですから @userが正解です。
view indexでは controllerのindexで @users=Usr.all の様に『複数形の』@usersに代入し、viewの方で @users.each do |user| としているとおもいます。@user は定義されておらず、user が定義されているので user が正解です。

Comments

[退会済みユーザー]

返事が遅れてすみません。一旦全体を消して書き直したら上手くいってしまったので、自分で何が分からなかったのかが分からくなっていました。
winterboum様の回答を見ると、書き直したViewでは確かに@users.each do |user|としているので、この部分の不理解が原因だったのだと思います。@usersの各項目を1こずつ出すのだから@userで良さそうに見えなくもないですが、@usersと@userは全く別の変数なので、エラーになるだろうなと今では理解できます。
ご教授ありがとうございました。

Answer 2

<img src="<%= "/user_images/#{@user.image_name}" %>">
の代わりに
<%= image_tag @user.image_name.url %>
にするとエラーが出ないはずです