我无法处理控制器中的自定义操作

Question

在任务控制器中我有操作：

  def destroy_all
    current_user.tasks.destroy
    redirect_to root_path
  end

我的数据库架构如下所示：

  create_table "tasks", force: true do |t|
    t.string   "content"
    t.boolean  "done"
    t.datetime "created_at"
    t.datetime "updated_at"
    t.string   "users_id"
  end

我有关系：Tasks belongs_to User 和 User has_many Tasks。 通过actiondestory_all我想销毁所有当前用户的任务。

但是当我点击时：

<%= link_to "delete all", { controller: 'tasks', action: 'destroy_all'}, method: 'delete' %>

什么都没有发生。用户仍然有他的所有任务。

从服务器控制台登录：

Started DELETE "/tasks/destroy_all" for 127.0.0.1 at 2014-10-04 23:29:27 +0200
Processing by TasksController#destroy_all as HTML
  Parameters: {"authenticity_token"=>"fVPDi6bczNXlfhjCDI4pJhMUm3cjv6TN1Ny/ulUO4YQ="}
  User Load (0.2ms)  SELECT  "users".* FROM "users"  WHERE "users"."id" = 2  ORDER BY "users"."id" ASC LIMIT 1
Redirected to http://localhost:3000/
Completed 302 Found in 31ms (ActiveRecord: 0.2ms)


Started GET "/" for 127.0.0.1 at 2014-10-04 23:29:27 +0200
Processing by PagesController#home as HTML
  User Load (0.2ms)  SELECT  "users".* FROM "users"  WHERE "users"."id" = 2  ORDER BY "users"."id" ASC LIMIT 1
  Task Load (0.1ms)  SELECT "tasks".* FROM "tasks"  WHERE "tasks"."done" = 'f' AND "tasks"."user_id" = 2
  Rendered tasks/index.html.erb within layouts/application (2.4ms)
  Rendered layouts/_userbar.html.erb (0.2ms)
  Rendered layouts/_menu.html.erb (0.1ms)
Completed 200 OK in 46ms (Views: 44.3ms | ActiveRecord: 0.3ms)

还有routes.rb文件

  resources :tasks do
    collection do
      delete :destroy_all
    end
  end

编辑：我更新了 log 和 link_to 方法并添加了 routes.rb 文件。

Answer 1

首先，您的任务表需要字段“user_id”，而不是“users_id”。这可能会立即解决您的问题。

但是，您缺少 has_many 的力量！只需这样做：

current_user.tasks.destroy_all

调用“current_user.tasks”会为您获取用户任务的列表，然后您可以像我上面所做的那样对它们调用“destroy”。这更清晰，更易于阅读。

我希望这会有所帮助！

Answer 2

根据您的架构，users_id 是 String，可能是 "1" 或 "2"。但是current_user.id 是Integer，可能是1 或2。

在这种情况下，您将在 ActiveRecord 中找不到任何内容，因为它会查询 id 是字符串的项目，而您实际上发送的是一个整数。

因此尝试：

Task.destroy_all(users_id: current_user.id.to_s)

匹配类型。

================================

但是，您的结构很复杂，将来很难维护。我建议您执行以下操作：

1. 在Task 模型中，将users_id 更改为user_id，因为它只属于一个用户。
2. 将user_id的类型改为整数，因为它是id，逻辑上它只包含数值。
3. 正如 Jaime 所建议的，在 has_many 中使用 Rails 约定并编写类似 current_user.tasks.delete_all 的内容，以保持代码简洁且更具可读性。