删除多个图像活动存储阵列中的单个图像

Question

在我的编辑页面上，我显示了上传到活动存储数据库的产品的所有图像，并在其下方为该单个图像提供了一个专用的删除按钮。如果我手动插入 id，代码会删除图像，但我的代码发送的 id 给出了图像在数组中的位置。如何从数组位置编号中找到链接到图像的实际 id？

如果有更简单的方法可以做到这一点，也将不胜感激。

views/admin/products/_formedit.html.erb

<% (0...@admin_product.images.count).each do |image| %>
  <%= image_tag(@admin_product.images[image]) %>
  <%= link_to 'Remove', delete_image_attachment_admin_product_url(image), method: :delete, data: { confirm: 'Are you sure?' } %>
<% end %>

controllers/admin/products_controller.rb

def delete_image_attachment
  @image = ActiveStorage::Attachment.find(params[:id])
  @image.purge
  redirect_to contact_url
end

routes.rb

namespace :admin do
  resources :products do
    member do
      delete :delete_image_attachment
    end
  end
end

Answer 1

让我们从让路线不那么不稳定开始吧。

namespace :admin do
  resources :products do
    resources :images, only: :destroy
  end
end

这将创建一个 RESTful 嵌套路由 DELETE /admin/products/:product_id/images/:id，而不是一个奇怪的 RPC 样式 delete_image_attachment 路由。如果图像 id 是唯一的，您还可以使用 shallow: true 选项来取消嵌套路由。您的路线的问题在于它包含一个 ID，但它在错误的位置。当您查看时：

products/1/delete_image_attachment

这意味着它会删除products/1 的附件。而不是 id 为 1 的图像是实际发生的事情。

当您的迭代只是遍历集合而不是其索引时：

<% @admin_product.images.each do |image| %>
  <%= image_tag(image) %>
  <%= link_to 'Remove', 
              admin_product_image_path(@admin_product, image), 
              method: :delete, 
              data: { confirm: 'Are you sure?' } 
  %>
<% end %>

不要使用each |index|，然后在Ruby 中使用things[index] 获取项目。它是一种反模式，@admin_product.images.count 创建了一个完全可以避免的数据库查询。如果您需要该项目及其位置，请使用each_with_index。