在删除ForEach中的数组元素时,SwiftUI超出索引

时间:2020-04-25 19:01:50

标签: swift swiftui

我在这里浏览了不同的问题,但不幸的是我找不到答案。这是我的代码:

SceneDelegate.swift

...
let contentView = ContentView(elementHolder: ElementHolder(elements: ["abc", "cde", "efg"]))
...
window.rootViewController = UIHostingController(rootView: contentView)

ContentView.swift

class ElementHolder: ObservableObject {

    @Published var elements: [String]

    init(elements: [String]) {
        self.elements = elements
    }
}

struct ContentView: View {

    @ObservedObject var elementHolder: ElementHolder

    var body: some View {
        VStack {

            ForEach(self.elementHolder.elements.indices, id: \.self) { index in
                SecondView(elementHolder: self.elementHolder, index: index)
            }

        }
    }
}

struct SecondView: View {

    @ObservedObject var elementHolder: ElementHolder
    var index: Int

    var body: some View {
        HStack {
            TextField("...", text: self.$elementHolder.elements[self.index])
            Button(action: {
                self.elementHolder.elements.remove(at: self.index)
            }) {
                Text("delete")
            }
        }
    }

}

按下删除按钮时,应用崩溃,出现索引超出范围错误。

有两个奇怪的事情,应用程序何时运行

1)删除VStack并将ForEach放入body的{​​{1}}或

2)您将ContentView.swift的代码直接放入SecondView

只有一件事:我真的需要拥有ForEach,此代码只是另一个代码的简化。

更新

我更新了代码,并将ObservableObject更改为Text,因为我不能只传递字符串,所以需要双向连接。

2 个答案:

答案 0 :(得分:2)

问题是由单击删除按钮时执行更新的顺序引起的。

按下按钮,将发生以下情况:

  1. 元素持有人的elements属性已更改
  2. 这会通过objectWillChange协议的ElementHolder发布者发送通知,并由ObservableObject协议声明。
  3. 已订阅此发布者的视图将收到一条消息,并将更新其内容。
    1. SecondView会接收通知并通过执行body getter来更新其视图。
    2. ContentView通过执行body getter来接收通知并更新其视图。

要使代码不崩溃,必须在3.2之后执行3.1。 (据我所知)无法控制此顺序。

最优雅的解决方案是在SecondView中创建一个onDelete闭包,该闭包将作为参数传递。

这还将解决体系结构反模式,即元素视图可以访问所有元素,而不仅仅是显示的元素。

集成所有这些将导致以下代码:

struct ContentView: View {
    @ObservedObject var elementHolder: ElementHolder

    var body: some View {
        VStack {
            ForEach(self.elementHolder.elements.indices, id: \.self) { index in
                SecondView(
                    element: self.elementHolder.elements[index],
                    onDelete: {self.elementHolder.elements.remove(at: index)}
                )
            }
        }
    }
}

struct SecondView: View {
    var element: String
    var onDelete: () -> ()

    var body: some View {
        HStack {
            Text(element)
            Button(action: onDelete) {
                Text("delete")
            }
        }
    }
}

有了它,甚至可以删除ElementHolder并只拥有一个@State var elements: [String]变量。

答案 1 :(得分:1)

这里是可能的解决方案-使SecondView的正文与ObservableObject无关。

通过Xcode 11.4 / iOS 13.4进行了测试-没有崩溃

struct SecondView: View {

    @ObservedObject var elementHolder: ElementHolder
    var index: Int
    let value: String

    init(elementHolder: ElementHolder, index: Int) {
        self.elementHolder = elementHolder
        self.index = index
        self.value = elementHolder.elements[index]
    }

    var body: some View {
        HStack {
            Text(value)     // not refreshed on delete
            Button(action: {
                self.elementHolder.elements.remove(at: self.index)
            }) {
                Text("delete")
            }
        }
    }
}

另一种可能的解决方案是不要观察ElementHolder中的SecondView ...不需要显示和删除它-也不会崩溃

struct SecondView: View {

    var elementHolder: ElementHolder // just reference
    var index: Int

    var body: some View {
        HStack {
            Text(self.elementHolder.elements[self.index])
            Button(action: {
                self.elementHolder.elements.remove(at: self.index)
            }) {
                Text("delete")
            }
        }
    }
}

更新:文本字段SecondView的变体(仅文本字段本身已更改)

struct SecondViewA: View {

    var elementHolder: ElementHolder
    var index: Int

    var body: some View {
        HStack {
            TextField("", text: Binding(get: { self.elementHolder.elements[self.index] },
                set: { self.elementHolder.elements[self.index] = $0 } ))
            Button(action: {
                self.elementHolder.elements.remove(at: self.index)
            }) {
                Text("delete")
            }
        }
    }
}