Rails关系型RDBMS系统中destroy操作执行报错问题咨询

这段destroyBook方法逻辑存在致命错误，无法正常运行，不管是Web端触发请求，还是把逻辑搬到rails console中执行，都会抛出异常，你看到的页面报错就是代码逻辑问题直接导致的。

• 执行顺序完全颠倒：代码第一部分就调用@book.destroy把对应ID的Book主表记录物理删除了，后续两行又调用Book.find_by(id:params[:id])查询已经被删掉的记录，这个查询的返回结果是nil，再对nil调用.book_authors、.book_categories方法会直接抛出NoMethodError，也就是undefined method 'xxx' for nil:NilClass类错误，和页面展示的报错完全匹配：
  
• 存在冗余数据库查询：就算调整执行顺序，也完全没必要在已经查到@book实例的情况下，反复用ID查两次Book记录，平白增加不必要的数据库开销。
• 外键约束触发风险：如果数据库层面给book_authors、book_categories表配置了指向books表的外键约束，在关联表数据没清理的情况下直接删Book主表记录，会直接触发关系型数据库的外键校验错误，连第一行destroy操作都执行失败。

临时修复（仅调整控制器代码） #

把删除顺序调换，直接复用已查询到的@book实例操作，不要重复查库：

def destroyBook
    @book = Book.find(params[:id])
    # 先清理关联表数据，再删除主表记录
    @book.book_authors.destroy_all
    @book.book_categories.destroy_all
    @book.destroy
    redirect_to "/books/views"
end

最佳实践（Rails标准写法） #

Rails的Active Record关联本身提供了自动处理关联数据删除的配置，不需要在控制器里手动写关联删除逻辑，只需要在Book模型中配置关联依赖即可：

1. 先修改app/models/book.rb的关联配置：

class Book < ApplicationRecord
  # 配置dependent参数，删除Book记录时自动删除关联的关联表数据
  has_many :book_authors, dependent: :destroy
  has_many :book_categories, dependent: :destroy
end

2. 控制器代码可以简化到只剩核心逻辑：

def destroyBook
    @book = Book.find(params[:id])
    @book.destroy
    redirect_to "/books/views"
end

提示：如果关联表数据量很大，不需要触发关联模型的回调，可以把dependent: :destroy换成dependent: :delete_all，直接生成SQL批量删除，执行效率更高。

• 原错误代码逐行搬到console执行时，执行到删除主表后再查记录调关联方法的步骤，一定会报nil的方法缺失错误，不可能顺利执行。
• 调整删除顺序，或者配置好模型的dependent关联后，在console中执行对应删除逻辑可以正常运行。

内容的提问来源于stack exchange，提问作者Afaq Nasir