ｓrailsにおいて編集が反映されない件について

Question

railsで編集機能が反映されません。

ログインアプリ
ログアウト
マイページ
投稿する
削除
編集
dddddd

---

いま現在アプリはこのようになっています。

編集ボタンを押すと

http://localhost:3000/posts/4/edit

---

ログインアプリ
ログアウト
マイページ
投稿する
dddddd　投稿

---

このようになるのでddddddの部分をupdatedに変えて

---

ログインアプリ
ログアウト
マイページ
投稿する
updated 投稿

---

投稿ボタンを押して
http://localhost:3000/posts/5

ログインアプリ
ログアウト
マイページ
投稿する
you posted
go to home

go to homeを押してホームに戻っても

http://localhost:3000/posts

---

ログインアプリ
ログアウト
マイページ
投稿する
削除
編集
dddddd

---

このようにddddddのままで困っています。

Rails.application.routes.draw do
  devise_for :users

    root to: 'posts#index'
    resources :posts
    resources :users
  end

class PostsController < ApplicationController
  def index

    @posts = Post.all
  end

  def new
    @post = Post.new
  end

  def create
    Post.create(post_params)
  end


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

  def update
    @post = Post.find(params[:id])
    if post.user_id == current_user.id
      post.update(post_params)
    end
  end

  def show
  end

  def destroy
   post = Post.find(params[:id])
    if post.user_id == current_user.id
     post.destroy
    end
  end


    private
  def post_params
    params.require(:post).permit(:text).merge(user_id: current_user.id)
  end
end

<div class="posts">
  <% @posts.each do |post| %>
     <li>
        <%= link_to '削除', "/posts/#{post.id}", method: :delete %>
     </li>
     <li>
        <%= link_to '編集', "/posts/#{post.id}/edit", method: :get %>
    </li>

      <%= simple_format(post.text) %>
  <% end %>
</div>

<div class = "posts">
  <%= form_for(@post) do |f| %>
  <%= f.text_field :text %>
  <%= f.submit '投稿' %>
  <% end %>
</div>

<div class = "posts">
  <%= form_for(@post) do |f| %>
  <%= f.text_field :text %>
  <%= f.submit '投稿' %>
  <% end %>
</div>

<div class = "posts">
  <h3> you posted </h3>

<a class = " btn " href= "/posts"> go to home  </a>


</div>

どうかお力を貸していただけると助かります。よろしくお願いします。

postcontroller

def update
@post = Post.find(params[:id])
if @post.user_id == current_user.id
post.update(post_params)
end
end

postから＠postに変更させていただきました。

結果このようにエラーが出ました。

NameError in PostsController#update
undefined local variable or method `post' for #<> Did you mean? @post

@post = Post.find(params[:id])
if @post.user_id == current_user.id
post.update(post_params)　　　　　　　　　　赤のライン
end
end

Comments

[退会済みユーザー]

post を create したのがべつユーザって可能性はないんですか？　
あと update でデータベース関連のエラーってでてないですか？

[dokodoko]

まずご協力ありがとうございます。
一応もう一度確認して来たのですが同一ユーザーでした。
エラーは出ないままホーム画面まで行きます。

[退会済みユーザー]

if post.user_id == current_user.id
post.update
が @post じゃないのかな…

view でつかってないんだったら
def update
@post = Post.find(params[:id])
こっちを @ とるのでもいいかも

あと def update が２個あるのもおかしい気がする

[退会済みユーザー]

post.update(post_params)
のほうも @ がいりますね

Rails 関係ない ruby の言語レベルの話なのでこれでエラーが出る理由がわからないなら初心者マークつけたほうが優しく解説してもらえていいかもです

[dokodoko]

すいません。アドバイスありがとうございました。
chicaさんの
@post = Post.find(params[:id])
こっちを @ とるのでもいいかも

の一文を参考にさせていただきました。

[退会済みユーザー]

いままで def update が２つあったせいで後に書いてある方をとおっていて何の処理もされなかった。
下を消したために上の update を通るようになって @post に値を入れてるのに post から値を取り出そうとして nil 参照エラーが出ているという状態だと思います
def update 内で使う変数を @post か @ なし post で統一する必要があるんだと思います

解決したら自己回答して解決済みにしてください

[dokodoko]

初心者マークがあるのですね。
わたしはつけるべきだと思います。ご親切にありがとうございます。

[dokodoko]

最後まで私にアドバイスありがとうございます。
協力してもらった立場としてはchicaさんに対してベストアンサーするのが礼儀かなと思うのですが自己解決で構わないのですか？

[退会済みユーザー]

解決内容は質問文に追記じゃなくじぶんで解答欄に記述したほうがいいかもです
未解決、未回答のままだと回答者さんが見に来て無駄な時間を使ってしまうかもしれないので

[dokodoko]

確かに@post　にPost.find(params[:id])を収納してるのに
if post.user_id == current_user.id
ここでpostを記入してもPost.find(params[:id])をchicaさんの言う通り統一しないと呼び出せないと思いました。
わかりやすくありがとうございます。

def update
@post = Post.find(params[:id])
if post.user_id == current_user.id
post.update(post_params)
end
end

[dokodoko]

ご指摘ありがとうございます。
わかりました。解決方法の記入場所を修正させていただきます。

[dokodoko]

chicaさんに対してベストアンサーさせていただきたかったのですが申し訳ありません。
ですがchicaさんのおかげ解決できました。ありがとうございます。！

解決したのに未回答のままだと解決するために来てくれた親切な方への配慮が足りませんでした。
以後気をつけます。
ご指摘ありがとうございました。

Answer 1

解決方法

class PostsController < ApplicationController
  def index

    @posts = Post.all
  end

  def new
    @post = Post.new
  end

  def create
    Post.create(post_params)
  end


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



  def show
  end
　
　def update
    post = Post.find(params[:id])
    if post.user_id == current_user.id
      post.update(post_params)
    end
  end


  def destroy
   post = Post.find(params[:id])
    if post.user_id == current_user.id
     post.destroy
    end
  end


    private
  def post_params
    params.require(:post).permit(:text).merge(user_id: current_user.id)
  end
end

この部分を
def update
@post = Post.find(params[:id])
if post.user_id == current_user.id
post.update(post_params)
end
end

こう変えました
def update
post = Post.find(params[:id])
if post.user_id == current_user.id
post.update(post_params)
end
end