質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.37%

railsで作成中の画像にリンクを貼り付けたい

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 2,384

score 12

railsで作成中の画像にリンクを貼り付けたい

現在、railsで画像にリンクを張り付けshow(一覧ページ)に飛ばすリンクを作成中なのですが、以下のエラーが出ています。

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

NoMethodError in Books#show
Showing /vagrant/bookers-lv2/app/views/books/show.html.erb where line #44 raised:

undefined method `user_id' for nil:NilClass

該当のソースコード

<table class="table table-book" style="width: 70%;">
<thead>
<h2>Book detail</h2>
</thead>
<tbody>

<tr>
<% @books.each do |book| %>
<td><%= link_to user_path(book.user.user_id) do %>
<%= attachment_image_tag @user, :profile_image %>
<%= book.user.name %>
<% end %></td>
<td><%= book.title %></td>
<td><%= book.body %></td>
<% end %>
</tr>
</tbody>
</table>

以下はbookのコントローラーです。

class BooksController < ApplicationController
def index
@user = User.find(current_user.id)
@users = User.all
@books = Book.all
@book = Book.new
end

def show
@user = User.find(current_user.id)
@users = User.all
@books = Book.all
@newbook = Book.new
end

def create
book = Book.new(book_params)
book.save
flash[:success]='You have created book successfully.'
redirect_to users_detail_path(book.id)
end

def edit
@book = Book.find(params[:id])
end

def update
book = Book.new(book_params)
book.save
flash[:success]='You have updated book successfully.'
redirect_to books_path(book.id)
end

def destroy
book= Book.find(params[:id])
book.destroy
redirect_to books_path
end

private
def book_params
params.require(:book).permit(:title, :body)
end

end

試したこと

このファイルはuserとbookの2つのモデルがあり、1対多の関係です。link_toの指定先はshowページでbookのモデルに結びついたuserのidを指定したのですが、エラーが発生しました。

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

ここにより詳細な情報を記載してください。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • bamboo-nova

    2019/07/20 17:03

    すみません、Markdown形式でプログラムを載せていただけないでしょうか?
    ```と```の間に文章を挿入していただけるだけでも大分読みやすくなります。

    キャンセル

  • bamboo-nova

    2019/07/20 17:45

    すみません、元の質問をMarkdown形式で修正してください。あと、ビューとコントローラで別々に分けてください

    キャンセル

回答 1

0

<% @books.each do |book| %>
<td><%= link_to user_path(book.user.user_id) do %>
<%= attachment_image_tag @user, :profile_image %>


ここの部分ですが、userにはuser_idではなくidではないでしょうか。(質問者さんがどのような挙動であってほしいか書かれていなかったので推測ですが...)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/20 17:25 編集

    まだプログラミングを初めて2週間なので、説明が曖昧ですみません。マークダウン記法ももう少し勉強したいと思います。bamboo-nova様のご指摘どうり実装しましたが今度は、以下のエラーが出ました。

    ```
    NoMethodError in Books#show
    Showing /vagrant/bookers-lv2/app/views/books/show.html.erb where line #44 raised:

    undefined method `id' for nil:NilClass

    ```

    ここのページは各ユーザーのプロフィール画像が表示され、画像をクリックすると各ユーザーの詳細ページに移動する機能を実装したいです。


    ```
    ActiveRecord::Schema.define(version: 2019_07_18_045038) do

    create_table "books", force: :cascade do |t|
    t.string "title"
    t.text "body"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "user_id"
    end

    create_table "post_images", force: :cascade do |t|
    t.integer "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    end

    create_table "users", force: :cascade do |t|
    t.string "email", default: "", null: false
    t.string "encrypted_password", default: "", null: false
    t.string "reset_password_token"
    t.datetime "reset_password_sent_at"
    t.datetime "remember_created_at"
    t.integer "sign_in_count", default: 0, null: false
    t.datetime "current_sign_in_at"
    t.datetime "last_sign_in_at"
    t.string "current_sign_in_ip"
    t.string "last_sign_in_ip"
    t.string "name"
    t.text "introduction"
    t.integer "profile_image_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["email"], name: "index_users_on_email", unique: true
    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
    end

    end
    ```

    以上のuserテーブルのuser_idを引っ張ってきたいのですがエラーが出ます。

    キャンセル

  • 2019/07/20 18:06

    すみません、<%= link_to user_path(book.user) do %>としてみて頂けませんか?

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.37%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る