Scaffold機能でユーザーと入力内容を紐付ける方法

Question

以下のページを参考に小説サイトを作成しているのですがうまくいません。
Ruby on RailsでユーザIDと入力データの関連付けをする

具体的に、「undefined method `Short' for #User:0x007fde39eb2e30」と言った感じのエラーがでています。

###エラー内容

NoMethodError in ShortsController#index
undefined method `shorts' for #<User:0x007f004de722d8>

Extracted source (around line #8):
  # GET /shorts.json
  def index
    @shorts = current_user.shorts.all
  end

  # GET /shorts/1

###問題のcontrollerのコードです。

class ShortsController < ApplicationController
  before_action :set_short, only: [:show, :edit, :update, :destroy]
  before_action :authenticate_user!

  # GET /shorts
  # GET /shorts.json
  def index
    @shorts = current_user.shorts.all
  end

  # GET /shorts/1
  # GET /shorts/1.json
  def show
  end

  # GET /shorts/new
  def new
    @short = Short.new
  end

  # GET /shorts/1/edit
  def edit
  end

  # POST /shorts
  # POST /shorts.json
  def create
    @short = Short.new(short_params)

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

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

  # DELETE /shorts/1
  # DELETE /shorts/1.json
  def destroy
    @short.destroy
    respond_to do |format|
      format.html { redirect_to shorts_url, notice: 'Short was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_short
      @short = current_user.shorts.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def short_params
      params.require(:short).permit(:title, :description, :noval, :user_id)
    end
end

###追加補足
Userモデルはdevise+Omniauthで実装済みです。

###開発環境

• Ruby on Rails5.1.2

###主に使用しているGem

• devise
• omniauth

Comments

[suzukis]

エラーが出た箇所がエラーメッセージに記載されていると思います。これではどこでエラーになっているのか判断できません。

Answer 1

UserモデルからShortモデルへの関連がないのが原因です。

current_user.shortsを呼び出すためには
UserからShortへの1対多の関連が必要で
model/userには以下のような記述があるべきです。

has_many :shorts

参考にされたページを見るとuserにuserを関連付けていてここが間違っています。
確認してみてください。

1. usersとpostsは１対多になる。

ファイル：app/models/user.rb
以下の記述を追加
belongs_to :user

Comments

[firefox636]

ありがとうございます。解決しました

Answer 2

これ見る限りだとshortモデルのみが登録されて
current_userには関連づけられてないように見えますが把握してますか？

追記
元の記事のインデクスにユーザー作成があったのでこちらを参考にしてみてそれから本問題へ取り組んでみてください

Comments

[firefox636]

関連づけされていないところまで気づきませんでした。
質問にも記載したURLをただコピーしてみただけなので、どのような処理をしているのかよくわかっておりません。

[VerfolgungEin]

これ引用元が悪いですね・・・
userモデルを関連付けされているにかかわらず、userモデルとコントローラを記述してないのが今回の原因です
そのため、ユーザーモデルを新規に作るところから始めてみましょう

[firefox636]

devise+OmniauthでUserモデルは作成済みです。

Answer 3

def set_short
  @short = current_user.shor.find(params[:id])
end

shorになってますがこのせいではないですよね？

Comments

[firefox636]

すみません、shorではなくshortsの間違いです。