如何修复奇怪的 Rails 路线：DELETE activity.id

Question

我正在使用 public_activity gem、rails 6.0.3.2 和 ruby​​ v2.71。

在列出跟踪活动的页面上，我想为记录列表中的每条记录放置一个销毁按钮。我已经用下面显示的代码做到了。但是，我收到路由错误。 rails 正在寻找的路线是

activity.id

我想要的路线是

activity/:id

这是 html (HAML) 视图的相关部分：

-@activities.each do |activity|
 ...<other code>...    
 .float-right
      = link_to 'Destroy', activity_path(activity), method: :delete, data: { confirm: 'Are you sure?' }, class: 'btn btn-sm btn-danger'

这是我的 config/routes.rb 文件中的相关路由：

delete 'activity/:id', to: 'home#destroy_activity'

此路由返回一个错误，指出“没有路由匹配 [DELETE]”/activity.239”（显然每个记录 id 都不同）。

如果我将路线更改为：

delete 'activity.:id', to: 'home#destroy_activity'

但这对我来说看起来很奇怪，我宁愿使用“正常”的 rails 路线。

我做错了什么？

Answer 1

尝试这样做：

activity_path(activity.id)

代替：

activity_path(activity)

您已经使用delete 'activity/:id', to: 'home#destroy_activity' 定义了一个自定义路由，因此这需要对象的id 而不是对象本身。最好总是在路由定义的末尾添加:as，这样您就可以这样做：

delete 'activity/:id', to: 'home#destroy_activity', as: :activity_destroy

然后你可以像这样引用这条路径：

activity_destroy_path(activity.id)

最好的方法是始终为这些东西设置一个单独的控制器。因此，当您在家庭控制器中创建了一个动作 destroy_activity 时，您可以创建一个单独的 activities_controller 并添加如下Kaom Te 在下面的answer 中提到的这样的路由：

resources activities

默认情况下，这将为活动生成 CRUD 操作，但您可以使用 only 或 except 对其进行限制。

Answer 2

Rails 鼓励使用宁静的资源。通常我会在我的 routes.rb 文件中做这样的事情。

# Add others if appropriate... :create, :show... etc.
resources activities, only: [:destroy]  

这将由您的活动控制器在活动控制器.rb 中处理。

那么你的链接就是

= link_to 'Destroy', activity, method: :delete, data: { confirm: 'Are you sure?' }, class: 'btn btn-sm btn-danger'

您的 ActivitiesController 看起来像这样

class ActivitiesController < ApplicationController
  def destroy
    @activity = PublicActivity::Activity.find(params[:id])
    @activity.destroy

    # Redirect or whatever here....
  end