Gorm Delete删除不存在记录时未返回ErrRecordNotFound的问题

核心结论 #

不需要为了确认记录存在而单独先执行SELECT查询。
GORM的Delete属于批量写操作，执行成功但匹配0行记录属于合法的业务场景，不属于系统错误，因此不会返回ErrRecordNotFound——后者仅会在First/Take/Last这类强制要求返回1条记录的查询方法找不到匹配数据时抛出。如果先查再删，不仅多了一次数据库IO开销，还会存在并发竞态问题：查询时记录存在，但执行删除前记录已经被其他请求删除，最终还是会出现删除0行的情况，前置查询无法彻底规避这个问题。

必做的校验项 #

• 前置参数合法性校验
  在进入数据库操作前，先校验传入的id是否合法，比如拦截nil值、对应类型的零值（数字类型id为0、字符串类型id为空串），避免无效查询条件触发全表扫描，甚至出现无WHERE条件的误删事故。
• 删除结果的影响行数校验
  直接读取Delete操作返回结果的RowsAffected字段，该字段代表本次操作实际删除的记录数：如果值为0，说明没有匹配到对应id的记录；值大于0说明删除成功。这个判断方式性能更高，也不存在并发竞态问题。

改造后的代码示例 #

import (
    "errors"
    "strings"
)

func (r *WorkspaceRepository) Delete(id any) (bool, error) {
    // 前置参数校验，可根据id实际使用的类型扩展判断逻辑
    switch v := id.(type) {
    case uint:
        if v == 0 {
            return false, errors.New("invalid workspace id")
        }
    case int64:
        if v == 0 {
            return false, errors.New("invalid workspace id")
        }
    case string:
        if strings.TrimSpace(v) == "" {
            return false, errors.New("invalid workspace id")
        }
    }

    res := r.db.Where("id = ?", id).Delete(&model.Workspace{})
    if res.Error != nil {
        return false, res.Error
    }

    // 无匹配记录，删除失败
    if res.RowsAffected == 0 {
        return false, nil
    }

    return true, nil
}

特殊场景说明 #

只有当你删除前需要做额外业务校验时，才需要提前执行SELECT查询，比如：

• 校验当前操作人是否拥有该工作区的删除权限
• 判断工作区是否处于不可删除状态（如已归档、存在运行中的关联任务）
• 你的模型使用了软删除（带gorm.DeletedAt字段），需要区分「记录从未存在」和「记录已被软删除」两种状态
  除此之外，单纯判断记录是否存在完全不需要前置查询。

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