Rails 7中更新记录时如何正确清理关联的ActiveStorage附件？

你遇到的问题核心是Strong Parameters没有放行remove_documents参数，导致你的删除逻辑根本没执行，另外还有几个小细节可以优化，我来一步步帮你解决：

1. 先修正Strong Parameters，放行删除参数 #

你的post_params方法里只允许了:title和documents: []，但表单提交的remove_documents数组被强参数过滤掉了，所以params[:post][:remove_documents]一直是nil，删除循环根本跑不起来。修改成下面这样：

private
def post_params
  params.require(:post).permit(:title, documents: [], remove_documents: [])
end

2. 优化删除逻辑的健壮性 #

你当前的删除逻辑没问题，但可以调整得更稳妥——比如用find_by代替find避免报错，处理完删除后移除参数让update更干净：

def update
  # 处理附件删除
  if params[:post].dig(:remove_documents).present?
    params[:post][:remove_documents].each do |attachment_id|
      # 只操作当前帖子关联的附件，避免误删其他记录的内容
      attachment = @post.documents.find_by(id: attachment_id)
      if attachment
        attachment.purge # 彻底删除存储文件+数据库记录
        Rails.logger.info "已删除附件：#{attachment.filename}"
      end
    end
    # 移除已处理的参数，避免后续update时干扰
    params[:post].delete(:remove_documents)
  end

  if @post.update(post_params)
    redirect_to @post, notice: '帖子更新成功'
  else
    render :edit
  end
end

这里purge方法会同时完成三个操作：删除存储服务里的文件、删除ActiveStorage::Attachment记录、删除ActiveStorage::Blob记录，完全符合你“彻底清理”的需求。如果用purge_later，会把删除任务放到后台队列，适合大文件，但需要确保你的后台队列（比如Sidekiq）是正常运行的。

3. 优化表单的用户体验（可选） #

你当前的标签和复选框没有关联，点击标签不会选中复选框，修改成下面这样能提升体验：

<% @post.documents.each do |doc| %>
  <div>
    <%= link_to doc.filename, url_for(doc) %>
    <%= check_box_tag "post[remove_documents][]", doc.id, false, id: "remove_doc_#{doc.id}" %>
    <%= label_tag "remove_doc_#{doc.id}", "Remove #{doc.filename}" %>
  </div>
<% end %>

4. 验证逻辑是否生效的小技巧 #

如果还是不确定删除逻辑有没有执行，可以在控制器里加日志输出，然后查看log/development.log，就能直观看到删除操作的执行情况。

这个方案是Rails 7中处理多附件删除的常规方式，不需要复杂的前端JS，完全符合你的需求。

内容来源于stack exchange