HealthKit数据写入Dictionary触发EXC_BAD_ACCESS(data race)求助

核心原因 #

Swift标准库的Dictionary本身不具备线程安全性，当多个线程同时对它执行读写操作时，会触发数据竞争（data race），进而导致随机崩溃、内存错误（比如double-free）——这和你遇到的“随机间歇性崩溃”、Thread Sanitizer检测结果完全匹配。你之前用的DispatchQueue/DispatchGroup可能只是管理了任务执行时机，但没有对consolidatedData的所有访问做同步控制。

具体修复方案 #

1. 用串行队列统一管理字典访问

给consolidatedData分配一个专属的串行队列，所有对该字典的读、写操作都通过这个队列执行，确保同一时间只有一个线程操作字典：

// 在HealthStore类内部声明串行队列
private let dataSyncQueue = DispatchQueue(label: "com.yourapp.health.data.sync")

// 更新字典时的代码修改
dataSyncQueue.sync {
    consolidatedData[quantityTypeIdentifier] = averageByDay
}

// 读取字典时也要走这个队列，比如：
var targetData: [Date: Double]?
dataSyncQueue.sync {
    targetData = consolidatedData[quantityTypeIdentifier]
}

2. 用NSLock加锁实现同步

如果不想用队列，也可以用锁包裹所有读写操作，确保同一时间只有一个线程能访问字典：

// 在HealthStore类内部声明锁
private let dataLock = NSLock()

// 更新字典的代码
dataLock.lock()
defer { dataLock.unlock() } // 确保锁一定会被释放
consolidatedData[quantityTypeIdentifier] = averageByDay

// 读取字典的代码
dataLock.lock()
defer { dataLock.unlock() }
let targetData = consolidatedData[quantityTypeIdentifier]

3. 用Actor封装线程安全的数据访问（Swift 5.9+）

如果项目支持Swift 5.9及以上版本，用Actor是更现代的方案——Actor天然保证内部数据的线程安全，所有对属性的访问都会自动串行化：

// 把原HealthStore中涉及consolidatedData的逻辑封装到Actor里
actor HealthDataManager {
    private var consolidatedData: [String: [Date: Double]] = [:]

    func updateConsolidatedData(for identifier: String, with averageByDay: [Date: Double]) {
        consolidatedData[identifier] = averageByDay
    }

    func getConsolidatedData(for identifier: String) -> [Date: Double]? {
        return consolidatedData[identifier]
    }
}

// 调用时需要用await（在异步上下文中）
let healthManager = HealthDataManager()
await healthManager.updateConsolidatedData(for: quantityTypeIdentifier, with: averageByDay)

关键排查点 #

• 务必检查所有访问consolidatedData的代码（包括读取操作），确保每一处都做了同步处理——只处理写入、忽略读取的话，依然会触发数据竞争。
• 之前的DispatchGroup只是用来等待异步任务完成，它本身不提供线程安全保护，所以即使任务都完成了，只要任务内部在多线程直接操作字典，问题依然存在。

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