railsでdestroyメソッドが反映されない件について

Question

destroyメソッドが反映されなくて困っています。

class PostsController < ApplicationController
  def index

    @posts = Post.all
  end

  def new
    @post = Post.new
  end

  def create
    Post.create(post_params)
  end

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

  def edit
  end

  def show
  end

  def update
  end

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

Rails.application.routes.draw do
  devise_for :users

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

<div class="posts">
  <% @posts.each do |post| %>
     <li>
        <%= link_to '削除', "/posts/#{post.id}", method: :delete %>
     </li>
      <%= simple_format(post.text) %>
  <% end %>
</div>

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

rails sした際に

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

削除
vvvvvvvvv

この用に表示されます

vvvvvvvvvはコメントとして送信して表示された結果です。
この際に消去を押すと下の用になりback to indexを押してメイン画面に戻っても

ログインアプリ
ログアウト
マイページ
投稿する
dddd
back to index

↓

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

削除
vvvvvvvvv

コメントのvvvvvvvvvが消えません。

destroyメソッドを反映させコメントを消すにはどうしたらいいでしょうか？
お力を貸していただけると助かります。
またこちらの質問の仕方が分かりにくかったりしたら補足させていただきますのでよろしくお願いします。

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

↓

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

に改善させていただきました。がやはりまだ消去ボタンがつかえません。
お力を貸していただけると助かります。　よろしくお願いします。

Answer 1

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

の中にタイポがあります。

post = Postt.find(params[:id])

を

post = Post.find(params[:id])

にしたら解決するのでは…？

Comments

[dokodoko]

回答ありがとうございます。協力してくださり本当にありがたいです。
指摘して頂いた点を改善させていただきましたがやはり消去ボタンが反映されませんでした。
せっかく回答して頂いたのに活かせずに申し訳ありません
他に何か原因があればご指摘頂けると助かります。
お手数ですがよろしくお願いします。

[k-810]

def destroy
post = Postt.find(params[:id])
if post.user_id == current_user.id
post.destroy
end
end
のところが
def destroy
ではなくて
def delete
ではないでしょうか？

[k-810]

あ，すみません，やっぱりdestroyで合ってます????

[k-810]

def destroy
post = Postt.find(params[:id])
if post.user_id == current_user.id
post.destroy
end
end
がprivateより下にあるのが原因だと思います

[dokodoko]

解決しました‼️
ありがとうございます。
未熟な私に最後まで付き合ってくださりありがとうございました。（ ; ; ）

本当にありがとうございました。

[k-810]

解決してよかったです！！

[dokodoko]

privateの位置なんて全く原因の可能性に考えてなかったので質問してなかったら私では解決出来なかったと思います。
ありがとうございました。

Answer 2

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

のメソッドのところで　redirect_toを入れてないからなのかなあ
省略もできたような気もするけど、一度入れて試してみてはいかがでしょう

Comments

[dokodoko]

回答ありがとうございます。協力してくださり本当にありがたいです。

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

このような記述で当っているでしょうか？
試したところやはり反映されませんでした。

質問に質問で返してしまいすいません。

[story_aniki]

その書き方で問題ありませんね
そうでしたか、無理でしたか、もう一つ考えられるとすれば、k-810さんの指摘したタイプミスは間違いなくバグでしたから、それを修正しても症状が変わらないという点に着目して、修正が反映されてない可能性はないですか？

[dokodoko]

返信ありがとうございます。
すいません。私が未熟なためにちゃんとstory_anikiさんのアドバイスの意味を理解してないのかも知れないですが

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

end

postt →　post　　

の修正が反映されてないというのは、postt の時点でエラーになってないのはおかしいということでしょうか？

せっかく返信してもらったのにうまく理解出来ずにすいません。

[dokodoko]

解決しました。
未熟な私に対してstory_anikiさんが早い返信で私の質問に最後まで付き合ってくださってくれて感謝の気持ちしかありません。
ありがとうございました。