使用GRDB持久化数据时遇“Cannot use mutating member on immutable value”错误求助

首先明确报错核心原因：upsert是mutating方法（它会修改对象自身属性，比如插入数据库后给对象赋值自动生成的主键ID），而Swift函数参数默认是不可变的let常量，因此直接调用会触发报错。

你当前的"复制参数"方案是可行的，但可以通过以下几种方式优化，让代码更优雅：

1. 直接在函数内重新绑定可变变量 #

不用更换变量名，直接把参数重新声明为可变副本，比用w命名更具可读性：

static func saveWorkoutTemplate(wt: WorkoutTemplate) {
    guard let dbQueue = openDB() else { return }
    
    var wt = wt // 重新绑定为可变副本
    try! dbQueue.write { db in
        try! wt.upsert(db)
    }
}

本质和你当前的方案一致，但避免了无意义的变量命名，代码逻辑更连贯。

2. 改用inout参数（适合需要修改原对象的场景） #

如果业务逻辑需要让调用方传入的原对象也被更新（比如获取数据库生成的ID），可以把参数声明为inout：

static func saveWorkoutTemplate(wt: inout WorkoutTemplate) {
    guard let dbQueue = openDB() else { return }
    
    try! dbQueue.write { db in
        try! wt.upsert(db) // 直接修改传入的可变对象
    }
}

调用时需要传入可变变量：

var myTemplate = WorkoutTemplate(...)
saveWorkoutTemplate(wt: &myTemplate)
// 此时myTemplate已包含数据库生成的ID等更新后的数据

3. 封装非mutating的持久化方法 #

如果不想修改原对象，可以封装一个返回新实例的非mutating方法：

extension WorkoutTemplate {
    func saved(in db: Database) throws -> WorkoutTemplate {
        var copy = self
        try copy.upsert(db)
        return copy
    }
}

然后在保存函数中调用：

static func saveWorkoutTemplate(wt: WorkoutTemplate) {
    guard let dbQueue = openDB() else { return }
    
    try! dbQueue.write { db in
        let savedTemplate = try wt.saved(in: db)
        // 若需要使用保存后的对象（比如带ID），可在此处处理
    }
}

这种方式不会修改原对象，适合需要保留原对象初始状态的场景。

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