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

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

ただいまの
回答率

87.59%

【Ruby on Rails】フォロー機能 undefined method `id' for nil:NilClassというエラーメッセージが出る

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,733

score 17

前提・実現したいこと

フォロー機能をつけたい

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

イメージ説明

該当のソースコード

relationshipsテーブル

class CreateRelationships < ActiveRecord::Migration[5.2]
  def change
    create_table :relationships do |t|
      t.references :user, foreign_key: true
      t.references :follow, foreign_key: { to_table: :users }

      t.timestamps

      t.index [:user_id, :follow_id], unique: true
    end
  end
end


【Model】
relationship.rb

class Relationship < ApplicationRecord
  belongs_to :user
  belongs_to :follow, class_name: 'User'

  validates :user_id, presence: true
  validates :follow_id, presence: true
end


user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :post_images, dependent: :destroy
  has_many :favorites, dependent: :destroy
  has_many :fav_post_images, through: :favorites, source: :post_image
  has_many :cosplay_favorites, dependent: :destroy
  has_many :cosplay_fav_post_images, through: :favorites, source: :post_image
  has_many :post_comments, dependent: :destroy

  validates :name, presence: true, length: { in: 2..15 }

  attachment :profile_image



  has_many :relationships
  has_many :followings, through: :relationships, source: :follow
  has_many :reverse_of_relationships, class_name: 'Relationship', foreign_key: 'follow_id'
  has_many :followers, through: :reverse_of_relationships, source: :user

  def follow(other_user)
    unless self == other_user
      self.relationships.find_or_create_by(follow_id: other_user.id)
    end
  end

  def unfollow(other_user)
    relationship = self.relationships.find_by(follow_id: other_user.id)
    relationship.destroy if relationship
  end

  def following?(other_user)
    self.followings.include?(other_user)
  end
end


post_image.rb

class PostImage < ApplicationRecord
    belongs_to :user

    has_many :favorites, dependent: :destroy
    has_many :fav_users, through: :favorites, source: :user
    has_many :cosplay_favorites, dependent: :destroy
    has_many :cosplay_fav_users, through: :cosplay_favorites, source: :user
    has_many :post_comments, dependent: :destroy

    attachment :real_image
    attachment :cosplay_image

    default_scope -> { order(created_at: :asc) }
end


【Controller】
relationship_controller.rb

class RelationshipsController < ApplicationController
  before_action :set_user

  def create
    user = User.find(params[:relationship][:follow_id])
    following = current_user.follow(user)
    if following.save
      flash[:success] = 'ユーザーをフォローしました'
      redirect_to user
    else
      flash.now[:alert] = 'ユーザーのフォローに失敗しました'
      redirect_to user
    end
  end

  def destroy
    user = User.find(params[:relationship][:follow_id])
    following = current_user.unfollow(user)
    if following.destroy
      flash[:success] = 'ユーザーのフォローを解除しました'
      redirect_to user
    else
      flash.now[:alert] = 'ユーザーのフォロー解除に失敗しました'
      redirect_to user
    end
  end

  private

  def set_user
    user = User.find(params[:relationship][:follow_id])
  end
end


postimages_controller.rb

class PostImagesController < ApplicationController
    def new
        @post_image = PostImage.new
    end

    def create
        @post_image = PostImage.new(post_image_params)
        @post_image.user_id = current_user.id
        @post_image.save
        redirect_to post_images_path
    end

    def index
        @post_images = PostImage.page(params[:page]).reverse_order
        @post_image = PostImage.new
        @user = @post_image.user
    end

    def show
        @post_image = PostImage.find(params[:id])
        @post_comment = PostComment.new
    end

    def destroy
        @post_image = PostImage.find(params[:id])
        @post_image.destroy
        redirect_to post_images_path
    end

    private

    def post_image_params
        params.require(:post_image).permit(:real_image_name, :cosplay_image_name, :real_image, :cosplay_image, :caption, :favorites_count)
    end
end


users_controller.rb

class UsersController < ApplicationController
  def show
    @user = User.find(params[:id])
    @post_images = @user.post_images.page(params[:page]).reverse_order
  end

  def edit
    @user = User.find(params[:id])
  end

  def update
    @user = User.find(params[:id])
    @user.update(user_params)
    redirect_to user_path(@user.id)
  end

  private

  def user_params
    params.requrie(:user).permit(:name, :profile_image)
  end
end


【部分テンプレート】
relationships/_follow_button.html.erb

<% unless current_user == user %>
  <% if current_user.following?(user) %>
    <%= form_for(current_user.relationships.find_by(follow_id: user.id), html: { method: :delete }) do |f| %>
      <%= hidden_field_tag :follow_id, user.id %>
      <%= f.submit 'Unfollow', class: 'btn btn-danger btn-block' %>
    <% end %>
  <% else %>
    <%= form_for(current_user.relationships.build) do |f| %>
      <%= hidden_field_tag :follow_id, user.id %>
      <%= f.submit 'Follow', class: 'btn btn-primary btn-block' %>
    <% end %>
  <% end %>
<% end %>


【View】
index.html.erb

<div class="header">
  <nav class="navigation">
    <img src="/assets/logo.png">
    <ul>
      <li>
        <%= link_to "ログアウト", destroy_user_session_path, method: :delete %>
      </li>

      <li>
        <%= link_to '投稿する', new_post_image_path %>
      </li>

      <li>
        <%= link_to 'マイページ', user_path(current_user.id) %>
      </li>
    </ul>
  </nav>
</div>
<div class="post_images_index_wrapper">
  <% @post_images.each do |post_image| %>
    <div class="index_box">
      <div class="post_images_index_user">
        <ul>
          <li>
            <%= link_to user_path(post_image.id) do %>
              <%= attachment_image_tag @user, :profile_image, fallback: "no_image.jpg" %>
            <% end %>
          </li>
          <li>
            <p><%= link_to "#{post_image.user.name}", user_path(post_image.id) %></p>
          </li>



          <li>
            <%= render 'relationships/follow_button', user: @user %>
          </li>



        </ul>
      </div>
      <div class="post_images_index_title">
        <div class="image_title">
          <h2>Real</h2>
        </div>
        <div class="image_title">
          <h2>Cosplay</h2>
        </div>
      </div>
      <div class="post_images_box">
        <div class="post_image">
          <%= attachment_image_tag post_image, :real_image %>
        </div>
        <div class="post_image">
          <%= attachment_image_tag post_image, :cosplay_image %>
        </div>
      </div>
      <div class="image_name">
        <%= post_image.real_image_name %>
      </div>
      <div class="image_name">
        <%= post_image.cosplay_image_name %>
      </div>
      <div class="favorites_area">
        <div class="favorite_area">
          <%= render partial: 'post_images/post_images', locals: { post_image: post_image } %>
        </div>
        <div class="favorite_area">
          <%= render partial: 'post_images/cosplay_post_images', locals: { post_image: post_image } %>
        </div>
      </div>
      <div class="image_caption">
        <ul>
          <li>
            <%= link_to user_path(post_image.id) do %>
              <%= attachment_image_tag @user, :profile_image, fallback: "no_image.jpg" %>
            <% end %>
          </li>
          <li>
            <p><%= link_to "#{post_image.user.name}", user_path(post_image.id) %> </p>
          </li>
          <li>
            <p><%= link_to "#{post_image.post_comments.count}件のコメント", post_image_path(post_image.id) %></p>
          </li>
        </ul>
        <span class="caption"><%= post_image.caption %></span>
      </div>
    </div>
  <% end %>
  <%= paginate @post_images, class: "pagenate" %>
</div>


【ルーティング】
routes.rb

Rails.application.routes.draw do
  devise_for :users
  root 'post_images#index'
  resources :post_images, only: [:new, :create, :index, :show, :destroy] do
    resource :post_comments, only: [:create, :destroy]
  end
  resources :users, only: [:show, :edit]
  post '/favorite/:id' => 'favorites#favorite', as: 'favorite'
  post '/cosplay_favorite/:id' => 'cosplay_favorites#favorite', as: 'cosplay_favorite'


  resources :relationships, only: [:create, :destroy]


  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end

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

ruby 2.5.7p206
Rails 5.2.4.1

よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • winterboum

    2020/01/16 05:45

    index.htmlを呼び出しているcontrollerも載せてください

    キャンセル

  • is02

    2020/01/16 08:26

    申し訳ありません、追記致しました。

    キャンセル

回答 1

checkベストアンサー

+1

def index では @post_images しか定義していないのに
index.html で
<%= render 'relationships/follow_button', user: @user %>
@user を使っています。
このため partialで使われる user がnilとなっています。

def indexで @userを定義してください

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/01/16 19:56

    index.htmlをきちんと見ていなかったのですが今回見なおして問題がわかりました。
    が、その前に
    @post_image = PostImage.new ではuser情報が与えられていないので
    @user = @post_image.user はnilになります。

    問題は、indexで @userを使っていることです。
    「@userがnilになってる」という視点で今までコメントしてきちゃってましたが、@userそのものがいかんです。
    <% @post_images.each do |post_image| %>
    の中にuserは何回か出てきています
    <%= attachment_image_tag @user, :profile_image, fallback: "no_image.jpg" %>
    <%= link_to "#{post_image.user.name}", user_path(post_image.id) %> <%= render 'relationships/follow_button', user: @user %>
    これらの中の post_image.user の userが正解です。
    @userが出てきていますが、@userってある特定のuserということですから、indexの様な複数 を扱う中では出てくるのがおかしいのですが、なにか意味の有るuserなのかと思っていました。
    indexのなかの @user は意図としては
    @post_images.each do |post_image|
    のpost_imageのuserでは?
    ならば @userは post_image.user に修正してください

    キャンセル

  • 2020/01/16 19:58

    「@userってある特定のuserということですから」
    @userそのものにはそいういう意味はないです。そういう意味で大抵使われている かな

    キャンセル

  • 2020/01/16 22:13

    ありがとうございます!無事解決致しました!

    キャンセル

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

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

関連した質問

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