SwiftUI中在async块内修改@State变量是否安全？

你当前的实现既不符合SwiftUI的视图生命周期规范，也存在线程安全问题，不是合理的实践。

存在的核心问题 #

• 首先，body 属性是SwiftUI要求的纯计算属性，仅用于构建视图树，不允许在内部直接执行带有副作用的逻辑：你每次触发body重绘都会创建一个全新的异步下载任务，会出现重复下载、旧请求结果覆盖新请求、任务无自动取消逻辑导致内存泄漏等问题，在列表单元格复用的场景下这个问题会非常明显。
• 其次是你关心的线程安全问题：@State 包装的属性仅允许在**主线程（主Actor上下文）**中进行赋值修改。你使用的 URLSession.data(from:) 方法的回调执行上下文是后台线程，await 之后的代码默认会在后台线程运行，此时直接给 image 赋值属于跨线程操作UI状态，属于未定义行为，轻则出现UI不更新、渲染错乱，重则直接触发崩溃。

改进方案 #

你应该使用SwiftUI原生提供的 .task 修饰符来管理异步任务，这个修饰符会自动绑定视图生命周期：视图出现时启动任务，视图销毁时自动取消任务，还支持绑定id参数，当id变化时自动重启任务，完美适配你的需求。同时 .task 修饰符默认在主Actor上下文执行内部代码，对@State的赋值天然线程安全。

改进后的代码示例：

struct CachedAsyncImage<I: View, P: View>: View {
    @State private var image: UIImage?
    
    let content: (Image) -> I
    let placeholder: () -> P
    let url: URL?
    
    public init(
        url: URL?,
        @ViewBuilder content: @escaping (Image) -> I,
        @ViewBuilder placeholder: @escaping () -> P
    ) {
        self.url = url
        self.content = content
        self.placeholder = placeholder
    }
    
    var body: some View {
        Group {
            if let image = image {
                content(Image(uiImage: image))
            } else {
                placeholder()
            }
        }
        // 绑定url作为任务id，url变化时自动取消旧任务启动新任务
        .task(id: url) {
            guard let url = url else { return }
            do {
                let (data, _) = try await URLSession.shared.data(from: url)
                guard let uiImage = UIImage(data: data) else { return }
                // task块默认在主Actor运行，此处赋值线程安全
                image = uiImage
            } catch {
                // 可按需添加自定义错误处理逻辑
                return
            }
        }
    }
}

如果你需要自定义异步逻辑的执行上下文，也可以在对@State赋值前加上 await MainActor.run { image = uiImage } 主动将赋值操作切换到主线程，确保线程安全。

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