destroy、削除機能とそのボタンの作成の仕方がわからない。初心者

Question

初学者です。
シンプルなアプリを作成する課題にとり組んでいます。CRUDの機能を実装したいと思っているのですが、その一つであるdestroy（削除機能）の実装の仕方がわからず進めることができずにいます。

今回作成するのはこのようなアプリで、削除機能を実装したいのは、一覧画面（index)と詳細画面（show)です。
以下がお手本とされるものです。

---

そして以下が私の現在の進捗状況となります。(一覧画面indexのみ。詳細画面showアクションはまだ編集していません）

---

エディタでは以下のように編集してみました。

index.html.erb

<h1>タスク一覧</h1>
<ul>

  <% @tasks.each do |task| %>
  <li>
    title: <%= task.title %>
  </li>

  <li>
    content: <%= task.content %>
  </li>

  <%= link_to "delete", task, method: :delete,
                                    data: { confirm: "You sure?" } %>



  <% end %>



</ul>

task_controller.rb

class TasksController < ApplicationController
  def index
    @tasks = Task.all
  end

  def new
    @task = Task.new
  end

  def edit
    @task = Task.find(params[:id])
  end


  def create
    @task = Task.new(tasks_param)
    #@task = Task.new(title: params[:task][:title], content: params[:task][:content])

    if @task.save
    redirect_to tasks_url
    else
      render :new
    end
  end


  def update

    @task = Task.find(params[:id])
    if @task.update(tasks_param)
      redirect_to tasks_path
    else
      render :edit
    end
end


  def destroy
    
      @task = Task.find(params[:id])
    if @task.destroy(tasks_param)
      redirect_to tasks_path
    else
      render :index

    end
 end

  private
  def tasks_param
    params.require(:task).permit(:title,:content)
  end
end

ボタンタイプなのでbutton_toなど試してみたのですが、書き方が間違っているのかエラー画面が表示されてしまいます。

とりあえずlink_toを入れて置いてるような状態です。
コントローラーの書き方が間違っているのか。。。ご教授いただけますでしょうか。
削除機能、ボタンの追加について、です。よろしくお願いいたします。

追記
<%= link_to "delete", task, method: :delete,data: { confirm: "You sure?" } %>の部分を、

<%= link_to '削除', task_path(@task.id),method: :delete, data: { confirm: "削除してよろしいですか？", commit: "削除する", cancel: "やめる", title: "ご確認ください" } %>
に編集した後のエラー

Answer 1

ターミナルでrails routesを実行してルーティングを確認してみたらいかがでしょうか？

恐らくDELETE /tasks/:id(.:format) tasks#destroyという記述がでてくると思います。

<%= link_to "delete", task, method: :delete,data: { confirm: "You sure?" } %>の部分を
下記のように書き換えたらうまくいくと思います。

<%= link_to '削除', task_path(@task.id),method: :delete, data: { confirm: "削除してよろしいですか？", commit: "削除する", cancel: "やめる", title: "ご確認ください" } %>

Comments

[-maeji-]

回答ありがとうございます(^ ^)
まずroutesの確認を実行したところ、おっしゃった通りの記述でした。

次にlink文を編集してみたのですが、NoMethodError in Tasks#indexが表示されました。
この場合tasksコントローラーを編集してあげればいいんですかね？

[Malas]

No Method Errorは文字通りメソッドがないという意味です。
ルートの一覧表を確認してみましょう。
あとindex.html.erbのどこでエラーが発生していますか？

[-maeji-]

>ルートの一覧表を確認

route.rb

Rails.application.routes.draw do
# For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
resources :tasks
root to: 'tasks#destroy'

end
と編集してみました。

>index.html.erbのどこでエラー
13行目です。メソッドdeleteの部分をdestroyにするとかですかね？

[Malas]

なぜメソッドのdeleteをdestroyに変更するんですか？destroyというメソッドはありません。
root to: tasks#destroyというのは書く必要ありますか？
destroy.html.erbを作成しているんですか？

[-maeji-]

ごめんなさい。
rails routes、resources等の理解ができていませんでした。
resourcesで定義されているのでrootに書く必要はありませんでした。