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

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

ただいまの
回答率

90.43%

  • Ruby

    8452questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    7889questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

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

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 97

-maeji-

score 18

初学者です。
シンプルなアプリを作成する課題にとり組んでいます。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: "ご確認ください" } %>
に編集した後のエラー

![イメージ説明](ca57f3411511d5a52e3b9fbf903e17fb.png)

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

ターミナルで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: "ご確認ください" } %>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/12/29 13:10

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

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

    キャンセル

  • 2018/12/29 13:23

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

    キャンセル

  • 2018/12/31 13:29

    >ルートの一覧表を確認

    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にするとかですかね?

    キャンセル

  • 2018/12/31 14:07

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

    キャンセル

  • 2019/01/11 17:03

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

    キャンセル

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

  • Ruby

    8452questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    7889questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。