SwiftUI使用NavigationLink跳转后调用API出现Index out of range致命错误

你遇到的数组越界报错由多个常见的SwiftUI开发误区共同导致：

• iOS 15及更早版本中，NavigationLink(isActive:destination:label:)的destination参数会在父视图body渲染时提前求值、提前创建子视图实例，而非点击跳转后才创建。即使你没有触发跳转逻辑，子视图也会提前初始化、触发内部的task异步请求。
• 子视图中使用@ObservedObject直接初始化ViewModel不符合SwiftUI的内存管理规则：@ObservedObject不持有传入的对象实例，视图每次重绘都会重新生成ViewModel，导致状态混乱、数据异常。
• ViewModel的getMessage(id:)方法中存在未做边界校验的数组下标访问操作，异步请求未完成/数据异常时直接访问空数组/下标超出范围的数组元素就会触发崩溃。

1. 修复子视图ViewModel声明 #

将子视图中@ObservedObject替换为@StateObject，确保ViewModel由当前子视图持有、只会初始化一次：

// 子视图代码修改
@StateObject private var viewModel = SomeThing() // iOS14+支持，若需兼容更低版本请将ViewModel从父视图传入
let id: Int

var body: some View {
    // 先判断数据是否存在，避免越界访问
    if !viewModel.message.isEmpty {
        Text("\(viewModel.message)")
    } else {
        ProgressView() // 加载态占位
    }
    .task {
        // 增加id合法性校验再发起请求
        guard id > 0 else { return }
        await viewModel.getMessage(id: id)
    }
}

2. 避免NavigationLink提前加载子视图 #

如果你的App兼容iOS15及更早版本，使用LazyView包装destination，确保子视图只有在跳转触发后才会初始化：

// 先定义通用LazyView
struct LazyView<Content: View>: View {
    let build: () -> Content
    init(_ build: @autoclosure @escaping () -> Content) {
        self.build = build
    }
    var body: Content {
        build()
    }
}

// 父视图NavigationLink修改为
NavigationLink(destination: LazyView(childView(id: 123)), isActive: $isSelection) { EmptyView() }

如果你的App最低支持iOS16，建议直接改用NavigationStack + navigationDestination，原生不会提前加载目标视图：

// iOS16+ 父视图实现示例
@State private var navigateId: Int? = nil

var body: some View {
    NavigationStack {
        Text("button")
            .onTapGesture {
                navigateId = 123
            }
            .navigationDestination(item: $navigateId) { id in
                childView(id: id)
            }
    }
}

3. 增加数组访问安全校验 #

在ViewModel的getMessage方法中，所有数组下标访问前都要做边界判断：

// 数组安全访问扩展（全局通用）
extension Collection {
    subscript(safe index: Index) -> Element? {
        return indices.contains(index) ? self[index] : nil
    }
}

// ViewModel中数组访问示例
func getMessage(id: Int) async {
    // 原写法，可能越界：self.message = dataArray[id]
    // 替换为安全访问
    if let safeMsg = dataArray[safe: id] as? String {
        self.message = safeMsg
    } else {
        self.message = "数据异常"
    }
}

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