未知动作，Ruby on Rails 4 初学者，销毁动作未知

Question

在我的删除页面上按“删除”后，我得到“找不到 ArticleController 的操作”。我认为它找不到破坏功能！这里缺少什么？

控制器：

class ArticleController < ApplicationController

def index
  @article= Article.all
end

def show
  @article= Article.find(params[:id])
end

def new
 @article= Article.new
end

def create
  @article= Article.new(user)

  if @article.save
      redirect_to(:action => 'index')
  else
      redirect_to(:action => 'index')
  end
end

def delete
  @article= Article.find(params[:id])
end

def destroy
  article.find(params[:id]).destroy
  redirect_to(:action => 'index')
end

private
  def user
  params.require(:article).permit(:title, :text)
  end
end

路线.rb

 Rails.application.routes.draw do

  resources :article 
   match ':controller(/:action(/:id))', :via => [:get, :post]
  #..

我的 delete.html.erb

  <h2>Delete</h2>

  <%= form_for :article, :url => url_for(:action => 'destroy', :id => @article) do |f| %>
    <%= @article.title %>
    <%= submit_tag("Delete It") %>
  <% end %>

Answer 1

我发现您发布的代码有一些问题。

首先，你的控制器应该是你的资源的复数名称，所以在这种情况下它应该是ArticlesController。

其次，关于您遇到的错误，在您的路由文件中使用resources :articles 时，销毁操作将映射到DELETE 请求。您的表单正在尝试 GET 销毁操作。

你想要解决的问题是

<%= form_for :article, :url => url_for(:action => 'destroy', :id => @article), :method => :delete do |f| %>

查看Rails Routing Guide，了解有关 Rails 如何使用 REST 的更多信息。

Answer 2

首先，您的destroy 方法无效且没有必要。将其从控制器中删除。

将您的 delete 方法更改为

def delete
  @article= Article.find(params[:id])
  if @article.destroy
  redirect_to(:action => 'index')
  else
  render 'delete'
end

最后，将您的form_for 更改为

<%= form_for :article, :url => url_for(:action => 'delete', :id => @article) do |f| %>
    <%= @article.title %>
    <%= submit_tag("Delete It") %>
  <% end %>

注意： 正如@brendon 所提到的，将您的控制器名称更改为 plural。目前它是 singular，这与Rails 的命名约定。

Answer 3

将 delete.html.erb 中的代码更改为

<h2>Delete</h2>

<%= @article.title %> <%= button_to 'Delete It', article, method: :delete %>