Railsでundefined method `[]' for nil:NilClass

Question

前提・実現したいこと

Rails6でツイートアプリを作っている者です。

ユーザー同士のフォロー機能を導入している時にハマってしまいました。

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

NoMethodError in RelationshipsController#create
undefined method `[]' for nil:NilClass

該当のソースコード

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

エラーメッセージで検索してもそれらしいページはヒットせず、対応の仕方がわからず困っています。
ご教授お願いいたします。足りない情報などございましたらお手数ですがコメントでおしらせください。

Answer 1

そのリンク先の記事を少し読みましたが、該当と思われるフォームが

<%= 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 %>

こんな感じで書かれているのでparams[:relationship]はnilです。
ただコピペするんじゃなくて、コードの意味や実行結果を確認しながら写経したほうがいいですよ。

修正するポイントはparams[:relationship][:follow_id]の部分か、
もしくはform_for ... endのフォーム部分どちらか一方でいいです。
フォームから送信される値を受け取れるようなparamas[:key]の書き方にするか
params[:relationship][:follow_id]に値を送信できるよなフォームにするかってことです。

Comments

[rhiroe]

とりあえず記事の方には修正リクエストを投げておいた

[bok_sakai]

ありがとうございます。
落とし穴に気づけませんでした。肝に命じます。

form_forをparams[:relationship][:follow_id]に値を送信できるよなフォームにするにはどのようなコードにするのが一番正しいでしょうか。@relationshipをつけたりしてみたのですがどれも上手くいきませんでした、、。

[rhiroe]

英語にはなりますが参考になるリンクを貼っておきます。
https://apidock.com/rails/ActionView/Helpers/FormHelper/form_for
英語が読めなくてもGoogle翻訳を通せばある程度理解できると思います。

盲目的にコピペをされている印象を受けたのでできれば自力で解決して欲しいところです。

どうしてもわからないのであれば、記事内の同じフォームを書いているhamlのコードが
params[:relationship][:follow_id]に値を送信できるような正しいフォームになっています。
もう一度言いますが、できればURLを貼ったドキュメントを参考に自力で解決してください。

[rhiroe]

一応日本語の公式ドキュメントも貼っておきます
https://railsguides.jp/form_helpers.html#%E3%83%A2%E3%83%87%E3%83%AB%E3%82%AA%E3%83%96%E3%82%B8%E3%82%A7%E3%82%AF%E3%83%88%E3%82%92%E6%89%B1%E3%81%86

個人的にはAPIdockの方が簡潔でわかりやすいのでおすすめです。

[bok_sakai]

できる限り複数のサイトを見比べながら、自分の正しいと思うコードを参考に導入していたのですが、力不足でした。

ご丁寧にありがとうございます。お気にかけていただいた分、きちんとリンク先で学ぼうと思います。もし理解できなければ再度質問させていただくかもしれません、その時はよろしくお願いいたします。

[bok_sakai]

リンク先の内容、難しいところもあり全てではありませんが、大枠は理解できました！
valueを指定する必要があったのですね、勉強になりましたm(_ _)m

Answer 2

params[:relationship][:follow_id]

実際にどういうデータが入っているか不明ですが、params[:relationship]がデータが無ければNilとなり、Nilオブジェクトに対して[:follow_id]でアクセスしようとしているのでご提示のエラーとなると思います。

Comments

[bok_sakai]

ご回答ありがとうございます。
基本的に以下のサイトの通りに導入を進めていたのですが、Nillになり得るポイントが分かりません。

https://qiita.com/MitsuguSueyoshi/items/e41e2ff37f143db81897#ルーティングを書く終了

よろしければ教えていただけないでしょうか。

[kazto]

当該の記事の正当性が分かりませんが、少なくともコードだけから判断して、Nilになりえる箇所は上記のとおりです。

[bok_sakai]

上記とは、どの部分のことでしょうか。
理解力不足なもので、申し訳ありません。

[kazto]

params[:relationship][:follow_id] の部分です。

Answer 3

そのエラーの出ている行、あるいはその前の行の変数をチェックしてみてください。
nilになってませんか。

Comments

[bok_sakai]

ありがとうございます。

https://qiita.com/MitsuguSueyoshi/items/e41e2ff37f143db81897#ルーティングを書く終了

上記のサイトを参考に導入していたのですが、nillになり得る箇所を見つけられません。教えていただけないでしょうか。

[y_waiwai]

なり得る箇所を探すんではなく、エラーの出てる行の変数をプリントアウトするなりして内容をチェックしましょう

あるいはRubyをデバッグできる環境を揃えましょう。
そうすれば、当てずっぽでコードを書かなくても済みますよ
「ruby デバッグ」でぐぐるとやり方出てきますね

[bok_sakai]

デバッグですか、ありがとうございます。
早速調べてみます。