Core Data中save方法调用时机疑问：上下文误用是否会引发问题？

你的代码存在几个明显的线程安全问题和逻辑错误，会导致崩溃、数据损坏或死锁风险，具体问题如下：

1. 主线程上下文（viewContext）的跨线程违规访问 #

viewContext通常配置为.mainQueueConcurrencyType，要求所有操作必须在主线程执行，或者通过viewContext.perform(_:)/viewContext.performAndWait(_:)在其绑定的队列中执行。但你的saveBook()是在非主线程调用的，直接用viewContext创建Book对象并修改属性，这违反了Core Data的并发规则，会触发线程冲突，大概率导致崩溃或数据不一致。

2. saveSync()方法的逻辑错误 #

• 子上下文未实际承载修改操作：你在viewContext中创建了Book对象，却新建了一个私有上下文作为viewContext的子上下文，然后尝试保存这个子上下文——但子上下文里根本没有任何待保存的修改，这个操作完全无效。
• 主线程阻塞与死锁风险：在私有队列的perform闭包中调用viewContext.performAndWait(_:)，会阻塞私有队列等待主线程完成保存。如果此时主线程正在等待私有队列的操作（比如你的扩展代码在后台线程调用saveBook()，主线程同时有其他Core Data操作），就会触发死锁。
• 父上下文保存时机错误：即使子上下文有修改，父上下文的保存也不需要在子上下文的perform闭包里同步执行，这种同步等待会破坏Core Data的异步队列设计，降低性能。

正确的实现方式 #

方案一：后台线程使用独立私有上下文（推荐） #

直接创建绑定到后台队列的私有上下文，在其队列内完成对象创建、修改和保存，避免跨线程访问主线程上下文：

func saveBook() {
    // 获取NSPersistentContainer实例（根据你的Core Data栈实现调整）
    let container = NSPersistentContainer(name: "YourModelName")
    container.loadPersistentStores(completionHandler: { (storeDescription, error) in
        if let error = error as NSError? {
            fatalError("Unresolved error \(error), \(error.userInfo)")
        }
    })
    
    // 创建后台上下文
    let backgroundContext = container.newBackgroundContext()
    backgroundContext.perform {
        let book = Book(context: backgroundContext)
        book.sessionId = UUID()
        book.appId = session.appId
        book.startedAt = session.startedAt
        book.createdBy = AppData.installId
        
        do {
            try backgroundContext.save()
            // 可选：通知主线程上下文刷新数据
            DispatchQueue.main.async {
                container.viewContext.refreshAllObjects()
            }
        } catch {
            let nsError = error as NSError
            print("Unresolved error \(nsError), \(nsError.userInfo)")
        }
    }
}

方案二：强制在主线程操作viewContext（适合简单场景） #

如果必须使用viewContext，确保所有操作都在主线程执行：

func saveBook() {
    DispatchQueue.main.async {
        let book = Book(context: self.viewContext)
        book.sessionId = UUID()
        book.appId = self.session.appId
        book.startedAt = self.session.startedAt
        book.createdBy = AppData.installId
        
        do {
            try self.viewContext.save()
        } catch {
            let nsError = error as NSError
            print("Unresolved error \(nsError), \(nsError.userInfo)")
        }
    }
}

关键规则回顾 #

• 每个NSManagedObjectContext绑定到特定队列，必须在其绑定队列内访问（或通过perform系列方法）。
• 后台操作优先使用独立的私有上下文，避免直接操作主线程的viewContext。
• 父-子上下文的使用场景是：子上下文在后台处理复杂修改，保存后将变更推送到父上下文，而非反向操作。

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