ForEach - Deleting last index causes Index out of bounds error

Question

tmsm1999 OP

Created Oct ’21

Replies 3

Boosts 0

Views 1.1k

Participants 3

Hello everyone,

I trying to draw a custom view inside a for each (list style), that is inside a Scroll View, that is inside a Navigation view. Like this.

Navigation View {
    ScrollView {
         ForEach(array of objects ...) {
               CustomView()
         }
    }
}

The custom view calls up a sheet that has a button that is able to delete elements inside the collection used in the foreach.

Unless I use this asyncAfter after dismissing the sheet I always get index out of bounds when I try to remove the last element of the array of objects in the for each:

DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) {
            workouts.removeAll(where: { $0.id == workoutToRemoveID })
        }

I have been trying to solve this bug, but so far no luck. Could you give me a hand?

Thank you for your help!

Boost

Answer 1

Claude31 OP

Oct ’21

Please show the complete code for

         ForEach(array of objects ...) {

Do you define the id ?

ForEach(arrayOfObjects, id: \.self )

0

Answer 2

OOPer OP

Oct ’21

Can you show a complete code? (It can be simplified, but ready to run.)

0

Answer 3

tmsm1999 OP

Oct ’21

Here is my code:

Here is my code:

@Binding var workouts: [Workout]
@State private var showWorkoutStudioEditor = false
@State private var sortedBy: SortBy = .nameAscending

    
@Environment(\.scenePhase) private var scenePhase

let saveAction: () -> Void

var body: some View {

        NavigationView {

            ScrollView(showsIndicators: true) {

                ForEach($workouts, id: \.id) { workout in

                    if !(sortedBy == .favorites) || workout.isFavorite.wrappedValue {

                        WorkoutCard(

                            workout: workout,

                            workouts: $workouts,

                            removeWorkout: removeWorkout

                        )
                            .padding([.leading, .trailing])
                    }
                }
         }
}

0