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

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

ただいまの
回答率

90.99%

  • Ruby on Rails

    6223questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2312questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

急にでてくるno method error

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 168

renren643

score 171

今までは問題なかったのに、急にno method errorがでてきました。
イメージ説明

class CommentsController < ApplicationController
  before_action :set_comment, only: [:show, :edit, :update, :destroy, :upvote]
  before_filter :authenticate_user!

  respond_to :html

  def index
    if params[:user_id]
      @user = User.find(params[:user_id])
      @comments = @user.comments
    else
      @comments = Comment.all 
    end
  end

  def show
    @comment = Comment.find(params[:id])
  end




  def create
    @song = Song.find(params[:song_id])
    @comment = @song.comments.new(comment_params)
     @comment.user = current_user
   respond_to do |format|
      if @comment.save
        format.html { redirect_to @song, notice: '投稿に成功しました' }
        format.json { render json: @comment, status: :created, location: @comment }
      else
        format.html { render action: "new" }
        format.json { render json: @comment.errors, status: :unprocessable_entity }
      end
    end
  end



  def destroy
    @comment.destroy
    respond_to do |format|
      format.html { redirect_to :back, notice: 'コメントを削除しました' }
      format.json { head :no_content }
    end
  end

  def upvote
    @comment.upvote_by current_user
    redirect_to :back
  end



  private
    def set_comment
      @comment = Comment.find(params[:id])
    end

    def comment_params
      params.require(:comment).permit(:song_id, :body, :user_id, :image)
    end

end
class SongsController < ApplicationController
  before_action :set_song, only: [:show, :edit, :update, :destroy]
  before_action :authenticate_user!, except: [:index, :show]
  # GET /songs
  # GET /songs.json
  def index
     if params[:user_id]
      @user = User.find(params[:user_id])
      @songs = @user.songs # ここでアソシエーションが生きる
     else
      @songs = Song.all
     end
  end

  # GET /songs/1
  # GET /songs/1.json
  def show
    @song = Song.find(params[:id])
  end

  # GET /songs/new
  def new
    @song = current_user.songs.build
  end

  # GET /songs/1/edit
  def edit
  end

  # POST /songs
  # POST /songs.json
  def create
    @song = current_user.songs.build(song_params)

    respond_to do |format|
      if @song.save
        format.html { redirect_to @song, notice: 'Song was successfully created.' }
        format.json { render :show, status: :created, location: @song }
      else
        format.html { render :new }
        format.json { render json: @song.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /songs/1
  # PATCH/PUT /songs/1.json
  def update
    respond_to do |format|
      if @song.update(song_params)
        format.html { redirect_to @song, notice: 'Song was successfully updated.' }
        format.json { render :show, status: :ok, location: @song }
      else
        format.html { render :edit }
        format.json { render json: @song.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /songs/1
  # DELETE /songs/1.json
  def destroy
    @song.destroy
    respond_to do |format|
      format.html { redirect_to songs_url, notice: 'Song was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  def upvote
    @song = Song.find(params[:id])
    @song.upvote_by current_user
    redirect_to :back
  end


  private
    # Use callbacks to share common setup or constraints between actions.
    def set_song
      @song = Song.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def song_params
      params.require(:song).permit(:title, :body)
    end
end

もしかしたら知らないうちに何か余計なことをかもしれませんが、ついさっきまで普通に動いていたのに不思議です。
rails4を使っていて、たまにこういったことがあるのですが、これはバグか何かなのでしょうか?

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+3

一番考えられるのはSongを削除したのにそれに紐付いたCommentを削除していないという状況です。このようなCommentから関連付けを通じてSongを参照してもnilになります。nilに対してtitleメソッドを呼ぼうとしてエラーになっています。

こういうことが起きないように、Songで、自身が削除されたら紐付いたCommentも削除するようにするようにしてください

has_many :comments, :dependent => :destroy

さらに、DBに外部キー制約を追加してください。

add_foreign_key :comments, :songs, on_delete: :cascade

comment.songnilでないかチェックして処理する、というようなアドホックなコードを書いてはいけません。comment.songがnilになるような状況を作り出さないようにすることを考えないと、どこもかしこもif !なんとか.nil?見たいなコードだらけになっていまいます。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/14 19:27 編集

    add_foreign_key :comments, :books, on_delete: :cascadeの:books,とは、なんでしょうか?この場合でいうと、songsですか?
    また、外部キーということも初めて聞いて、調べて見たのですが、外部キーの追加の仕方は、ALTER TABLE commnet add_foreign_key :comments, :books, on_delete: :cascadeをターミナルで実行したらいのでしょうか?

    キャンセル

  • 2017/10/16 09:14

    booksはsongsの書き間違いです。add_foreign_keyはマイグレーションのファイルに書いてください。rails g migrationで生成されるやつです。

    キャンセル

  • 2017/10/16 14:13 編集

    rails g migration add_foreign_key :comments, :songs, on_delete: :cascadeをターミナルで実行するということでしょうか?

    また、「has_many :comments, :dependent => :destroy」だけ記述してもうまく機能しているようなのですが、外部キー制約はなんの役目を果たしているのでしょうか?

    キャンセル

0

エラー内容の通り、該当箇所でcomment.songがnilであったためにcomment.song.titleが参照できずエラーが発生しています。

index.html.erbと、どの画面操作を行った際に発生したものかも確認する必要がありますが、commentに対応するsongがないなどの状況がどのように発生するのかを確認されるとよいものと思います。

そのような状況が通常操作で発生する場合があるのであれば、comment.songが参照可能かをindex.html.erbにてnilチェック等行い、確認したうえで参照するような処理とされるとよいかと思います。


追記:

コードを読むと、songに対するcommentなのですね。
であれば、suzukisさんのご指摘はごもっともだと思います。

私の回答はcommentとsongは「0または1」の関係と考え、ない場合もあるのであればnilチェック等による場合分けで対処できるのではないか、との回答でした。
songの削除に伴いcomment自体も削除されることに問題がないのであれば、そのような対応(commentの削除)もできると思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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

  • Ruby on Rails

    6223questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2312questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。