索引超出范围的 SwiftUI 问题

Question

我知道这是一个非常简单的问题，但我只是坚持使用 atm，所以任何建议都将不胜感激，因为我是 SwiftUI 的新手。

我正在尝试从 firebase 下载文本并将其渲染到视图中，但我不断收到超出范围的错误：

致命错误：索引超出范围：文件 Swift/ContiguousArrayBuffer.swift，第 444 行

代码如下：

var body: some View{
    
    ZStack {
        
        if fetch.loading == false {
            LoadingView()
        }
        
        else{
            Text(names[0])
                .bold()
        }
    }
    .onAppear {
        self.fetch.longTask()
    }
    
}

这是获取内容页面：

@Published var loading = false

func longTask() {

    DispatchQueue.main.asyncAfter(deadline: .now()) {
        
        let db = Firestore.firestore()
        db.collection("Flipside").getDocuments { (snapshot, err) in
            if let err = err {
                print("Error getting documents: \(err)")
                
                return
            } else {
                for document in snapshot!.documents {
                    let name = document.get("Name") as! String
                    let description = document.get("Description") as! String
                    //name = items[doc]
                    print("Names: ", name)
                    print("Descriptions: ", description)
                    names.append(name)
                    descriptions.append(description)
                }
            }
        }
        self.loading = true
    }
}

所以基本上当视图出现时，从 Firebase 获取数据，当数据下载后显示 menuPage() 直到然后显示 Loading Data 文本。

欢迎任何帮助！

Answer 1

正如 Rob Napier 提到的，问题是您在填充数组之前访问数组索引。

我建议对您的代码进行一些改进。此外，您可以创建一个结构来将所有属性保存在一个位置，而不是维护单独的数组（names、descriptions、...）。这将允许您只为您的项目使用一个数组。

struct Item {
    let name: String
    let description: String
}

class Fetch: ObservableObject {
    @Published var items: [Item] = [] // a single array to hold your items, empty at the beginning
    @Published var loading = false // indicates whether loading is in progress

    func longTask() {
        loading = true // start fetching, set to true
        let db = Firestore.firestore()
        db.collection("Flipside").getDocuments { snapshot, err in
            if let err = err {
                print("Error getting documents: \(err)")
                DispatchQueue.main.async {
                    self.loading = false // loading finished
                }
            } else {
                let items = snapshot!.documents.map { document in // use `map` to replace `snapshot!.documents` with an array of `Item` objects
                    let name = document.get("Name") as! String
                    let description = document.get("Description") as! String
                    print("Names: ", name)
                    print("Descriptions: ", description)
                    return Item(name: name, description: description)
                }
                DispatchQueue.main.async { // perform assignments on the main thread
                    self.items = items
                    self.loading = false // loading finished
                }
            }
        }
    }
}

struct ContentView: View {
    @StateObject private var fetch = Fetch() // use `@StateObject` in iOS 14+

    var body: some View {
        ZStack {
            if fetch.loading { // when items are being loaded, display `LoadingView`
                LoadingView()
            } else if fetch.items.isEmpty { // if items are loaded empty or there was an error
                Text("No items")
            } else { // items are loaded and there's at least one item
                Text(fetch.items[0].name)
                    .bold()
            }
        }
        .onAppear {
            self.fetch.longTask()
        }
    }
}

请注意，可能不需要通过下标访问数组。如果只有一项并且您尝试访问 items[1]，您的代码仍然会失败。

相反，您可能可以使用first 来访问第一个元素：

ZStack {
    if fetch.loading {
        LoadingView()
    } else if let item = fetch.items.first {
        Text(item.name)
            .bold()
    } else {
        Text("Items are empty")
    }
}

或使用ForEach 显示所有项目：

ZStack {
    if fetch.loading {
        LoadingView()
    } else if fetch.items.isEmpty {
        Text("Items are empty")
    } else {
        VStack {
            ForEach(fetch.items, id: \.name) { item in
                Text(item.name)
                    .bold()
            }
        }
    }
}

此外，如果可能，请避免强制展开选项。如果snapshot == nil，代码snapshot!.documents 将终止您的应用程序。此答案中提供了许多有用的解决方案：

• What does “Fatal error: Unexpectedly found nil while unwrapping an Optional value” mean?

Answer 2

基本问题是您在names 数组被填充之前评估names[0]。如果数组为空，那么您会看到此崩溃。您可能想要的是：

Item(title: names.first ?? "", ...)

您过早评估names[0] 的原因是您在获取实际完成之前调用了completed。您正在与初始方法调用同步调用它。

也就是说，您必须始终考虑存在连接错误或数据为空或数据已损坏的情况。一般来说，你应该避免下标数组（更喜欢.first之类的东西），当你下标数组时，你必须首先确保你知道有多少元素。