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

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

ただいまの
回答率

87.49%

投稿の編集ができません

解決済

回答 1

投稿 編集

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

score 18

rails初学者で初めてオリジナルアプリを作っているのですが、投稿したもの(データベースに保存したもの)の編集が上手く行えない状態に陥ってしまっています

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

NoMethodError in PostsController#update
undefined method `id' for nil:NilClass

Extracted source (around line #48):


    if @post.save
      redirect_to("/users/#{@user.id}")

    else
      render("posts/edit")


#48はredirect_to("/users/#{@user.id}")の部分です

上記のようなエラーが発生しているのですが、NilClassとなっているidはカラム名なのにNilclassとなっていて、なにが原因でこうなっているのか(なにをどう修正すればいいのか)が分からない状態です

該当のソースコード

**contorollers - posts_contoroller**

class PostsController < ApplicationController

  before_action :authenticate_user

  before_action :ensure_correct_user,{only:[:edit,:update,:destroy]}

  def index
    @post = Post.all.order(created_at: :desc)
  end

  def show
    @post = Post.find_by(id: params[:id])
  end


  def new
    @post = Post.new
  end




  def create
    @post = Post.new(
      content: params[:content],
      type: params[:type],
      time: params[:time],
      user_id: @current_user.id
    )
    if @post.save
      redirect_to("/users/#{@user.id}")
    else
      render("posts/new")
    end
  end



  def edit
    @post = Post.find_by(id: params[:id])
  end

  def update
    @post = Post.find_by(id: params[:id])
    @post.content = params[:content]

    if @post.save
      redirect_to("/users/#{@user_id}")

    else
      render("posts/edit")
    end
  end

  def destroy
    @post = Post.find_by(id: params[:id])
    @post.destroy
    redirect_to("/users/#{@user.id}")
  end



  def ensure_correct_user
    @post = Post.find_by(id: params[:id])
    if @post.user_id != @current_user.id
      flash[:notice]= "権限がありません"
      redirect_to("/posts/index")
    end
  end

end
**posts - edit.html.erb**

<div class="edit-wrapper">
      <div class="cont">
        <div class="edit-wrapper-main">
          <div class="edit-title">
           <h1>今日の積み上げ</h1>
          </div>
          <div class="edit-contents">

            <div class="edit-content edit-1">
              <p>今日の積み上げ時間</p>
              <%= form_tag("/posts/#{@post.id}/update") do %>
              <p><textarea class="edit-time" type="text" name="time" value="<%= @post.time %>">   </textarea></P>
            </div>

            <div class="edit-content edit-2">
              <p>今日やったこと</p>
              <textarea class="edit-thing" type="text" name="type" value="<%= @post.type %>">   </textarea>
            </div>

            <div class="edit-content">
              <p>具体的な内容</p>
              <textarea name="content" rows="8" cols="80" value="<%= @post.content %>" > </textarea>
            </div>

            <input class="edit-submit" type="submit" name="" value="投稿">
             <% end  %>

          </div>

        </div>
      </div>
    </div>
routes.rb

Rails.application.routes.draw do
  post "login" => "users#login"
  #OK
  get "login" => "users#login_form"
  #OK
  post "logout" => "users#logout"
  post "users/:id/update" => "users#update"
  get "users/:id/edit" => "users#edit"

  post "users/create" => "users#create"

  #OK
  get "signup" => "users#new"

  #OK
  get "users/:id" => "users#show"

  get "posts/index" => "posts#index"

  get "posts/new" => "posts#new"
  get "posts/:id" => "posts#show"
  post "posts/create" => "posts#create"

  get "posts/:id/edit" => "posts#edit"
  post "posts/:id/update" => "posts#update"
  post "posts/:id/destroy" =>"posts#destroy"
  get "/" => "home#top"
  #OK
  get "about" => "home#about"
  #OK
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end
**migrate - add_time_to_post.rb**
class AddTimeToPosts < ActiveRecord::Migration[5.2]
  def change
    add_column :posts, :time, :integer
  end
end
**create_posts.rb**
class CreatePosts < ActiveRecord::Migration[5.2]
  def change
    create_table :posts do |t|
      t.text :content

      t.timestamps
    end
  end
end
**migrate - add_type_to_post.rb**

class AddTypeToPosts < ActiveRecord::Migration[5.2]
  def change
    add_column :posts, :type, :string
  end
end

試したこと

コントローラーの

def update
    @post = Post.find_by(id: params[:id])
    @post.content = params[:content]

    if @post.save
      redirect_to("/users/#{@post.user_id}")

    else
      render("posts/edit")
    end
  end


の部分のredirect_to("/users/#{@post.user_id}")にしてみたのですが、今度は編集画面で投稿ボタンを押しても編集されない(エラーメッセージなどはなくただ投稿が保存されずページが変遷しないという状況)になってしまいました

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

@userはそれを唱えればいつでもどこでもその時に必要なuserが召喚される魔法の呪文  ではありません。唱える前に必要な値をセットしておく必要があります
@user に値が入っていない(nil)からです。
どのUserの画面に行きたいのでしょう?
そのPostの持ち主? でしたら redirect_to("/users/#{@post.user_id}")
login user でしたら redirect_to("/users/#{@current_user.id}")

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/01/22 14:45

    ご回答ありがとうございます

    教えていただいたものを試した結果エラー自体は解消されました

    しかし、質問文の「試したこと」にある編集画面で投稿ボタンを押しても編集されない(エラーメッセージなどはなくただ投稿が保存されずページが変遷しないという状況)はredirect_to("/users/#{@current_user.id}")をを入れても解消できませんでした

    キャンセル

  • 2020/01/22 17:02

    その時のlogがあるとはっきりするのですが、、、、
    @post.content = params.permit(:content)[:content]
    としてみてください

    キャンセル

  • 2020/01/22 18:30

    教えていただいた通り@post.content = params.permit(:content)[:content]とした結果、ページが変遷してcontentの部分のデータがしっかり編集・保存され、time,typeも同様にしたところ問題解決できました

    ありがとうございました

    キャンセル

  • 2020/01/22 18:46

    paramsでDBを扱うときはpermitしない項目は無視されます。
    strong paramater で調べてください

    キャンセル

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

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

関連した質問

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