railsのNoMethodErrorとundefined methodのエラー

Question

cloud9でrailsを使っているのですが、初期画像を設定する時にエラーが出ます。

ローカルサーバーを開くと

<div class="container">
  <div class="user">
  #↓が赤字
    <img src="<%= "/user_images/#{@user.image_name}" %>">
    <h2><%= @user.name %></h2>
    <p><%= @user.email %></p>
    <%= link_to("編集", "/users/#{@user.id}/edit") %>

class UsersController < ApplicationController
  def index
    @users = User.all
  end
  
  def show
    @user = User.find_by(id: params[:id])
  end
  
  def new
    @user = User.new
  end
  
  def create
    @user = User.new(
      name: params[:name],
      email: params[:email],
      image_name: "default_user.jpg"
      )
    if @user.save
      flash[:notice] = "ユーザー登録が完了しました"
      redirect_to("/users/#{@user.id}")
    else
      render("users/new")
    end
  end
  
  def edit
    @user = User.find_by(id: params[:id])
  end
  
  def update
    @user = User.find_by(id: params[:id])
    @user.name = params[:name]
    @user.email = params[:email]
    if @user.save
      flash[:notice] = "ユーザー情報を編集しました"
      redirect_to("/users/#{@user.id}")
    else
      render("users/edit")
    end
  end
  
end

<div class="main user-show">
  <div class="container">
    <div class="user">
      <img src="<%= "/user_images/#{@user.image_name}" %>">
      <h2><%= @user.name %></h2>
      <p><%= @user.email %></p>
      <%= link_to("編集", "/users/#{@user.id}/edit") %>
    </div>
  </div>
</div>

コード

です.
解決策をご存知であれば教えてください
<%= image_tag(@user.image_name}) %>を使うと。。。
![イメージ説明](b6dc375c637e8ccfdae61a3789e873f0.png)

Comments

[programming]

どのようなエラーが表示されるのか、追記してください。

[renren643]

追記しました。

Answer 1

モデルに原因があると思われます。
migrationファイルがあるのにデータベースにフィールドがないということは、migrateで何かしら問題があったのでしょう。
rake db:migrate:statusでmigrateされていないものがあるか確認してください。
そしてmigrateできていなかったら
rake db:migrate VERSION＝その時の日付
でmigrationしてください。

Comments

[renren643]

回答ありがとうございます。実行したところ、
database: /home/ubuntu/workspace/db/development.sqlite3

Status Migration ID Migration Name
--------------------------------------------------
up 20170825034244 Create posts
up 20170825074539 Create users
up 20170826055235 Add image name to users
とありました。migrateされていないものはなさそうです

[退会済みユーザー]

ちなみにremove_columnをしている形跡はありせんか？

[renren643]

ターミナルで実行したか？
と言うことでしょうか？

それならばしていません

[renren643]

ちなみに、このエラーはrubyやrailsのバージョンとは無関係ですか？？

Answer 2

ここは素直にimage_tagをつかう

<%= image_tag(@user.image_name) %>

でどうでしょう。

Comments

[renren643]

回答ありがとうございます。
<%= image_tag(@user.image_name}) %>にしてもエラーが出ました。
エラー画面は質問本文にアップしました

[NCC1701]

回答にtypoがありました。｝は不要です

[renren643]

typoって誤植の事言うんですね！勉強になります
やってみましたが、NoMethodError で、img srcのと変わらないエラーが出ました、

[NCC1701]

undefined method `image_name' for
は@userにそのようなメソッドが無いですとのエラーです
モデルの設計とあってますか？

[renren643]

質問本文内にあるcreateアクションで
@user = User.new(
name: params[:name],
email: params[:email],
image_name: "default_user.jpg"
)
がありますがこれじゃなくてですか？

[NCC1701]

create アクションが呼ばれたときはエラーが出なかったのですか？
（本来的にはコントローラでなくモデルの問題だと思いますので、Userモデルにimage_nameというフィールドがあるか、もしくはメソッドを作ったかを確認すべきです。）

[renren643]

create アクションが呼ばれたときはエラーが出なかったのですか？
>僕の知識不足でこの言葉の意味がわからないです。すいません、どう言うことでしょうか？

models/user.rb内のことでしょうか？
class User < ActiveRecord::Base
validates :name, {presence: true}
validates :email, {presence: true, uniqueness: true}
end
こうです

[NCC1701]

少なくともimage_nameメソッドは作っていないようです。
db/migrate/日付_create_users.rb か db/schema.rb
を見ればモデルの内容が見られます。そこにimage_nameがあるかどうかですが、たぶんないでしょう。

[renren643]

20170826055235_add_image_name_to_users.rbの中に
class AddImageNameToUsers < ActiveRecord::Migration
def change
add_column :users, :image_name, :string
end
end
を作っていました。

[renren643]

今僕が作っているものは、progateと言うサイトで練習で作ったやつで、「答え」と「手順」があるものなんです。それと全く同じ手順でつくったものなので、なぜダメなのかが全くわからないんです。。。
一番いいのは、データベースの中身に image_name: "default_user.jpg"で定義したdefault_user.jpgの
写真が現段階で保存されているのか、データベースにimage_nameカラムが保存されているかわかれば少しは前に進めそうなのですが、それはどのようにして確認すればいいのでしょうか？

[NCC1701]

db/schema.rbの内容は？

[renren643]

db/schema.rbに書いてあるのって、データベースの中身ですか？
こう書いてありました。
ActiveRecord::Schema.define(version: 20170826055235) do

create_table "posts", force: :cascade do |t|
t.text "content"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end

create_table "users", force: :cascade do |t|
t.string "name"
t.string "email"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end

end

[NCC1701]

schema.rbはデータベースの最新の構造と一致するようになってます。しかしt.string "image_name"がないので、エラーになったのです。
どこかで「手順」の実行を忘れてしまったのでしょう。
rails db:migrateをしてください。その後、もう一度schema.rbを見れば、変化しているはずです。

[renren643]

rake db:migrateを実行しましたが何も起こりませんでした。（rake db:migrateとrails db;migrateは同じですよね？cloud9でrails db;migrateを実行するとエラーが出ます）
理屈としてはデータベースにimage_nameがないからデータベースに画像を保存することが
できない＝エラーがでると言う認識でいいですか？

[退会済みユーザー]

migrateでエラーが出るのは、migrationファイルを作成したものの、何かしらの原因でそのmigrateファイルに書かれていることがデータベースに反映できていないことを指しています！

Answer 3

全般的に書き方がrailsっぽくないのが気になります。
また、赤字になっているというのは、たまたま先頭だからかもしれません。
試しに、その部分をコメントアウトして、次の行も赤字ならば、そもそもそこに@userが使えないのではないでしょうか。
つまり、routesが当たっていないのではないでしょうか。

railsの大まかな流れを一度見直した方が良いと思います。

Comments

[renren643]

回答ありがとうございます。routes.rbではpost "users/create" => "users#create"、get 'users/:id' => "users#show"と記述しているのですがどうでしょう？
また、赤くなった行を消してサーバーを動かしたところきちんとなまえとeーmailの
画面が出て、エラーは出ませんでした。