ForEach loop in Swift to use content of 2D array

Question

I want to use a ForEach loop, but it doesn't work.

let navigation = [["Kunde", "person"], ["Wartung", "gear"]]


NavigationView {
    List {
        NavigationLink(destination: ListView(title: "Kunde")) {
            Label("Kunde", systemImage: "person")
        }
        NavigationLink(destination: ListView(title: "Wartung")) {
            Label("Wartung", systemImage: "gear")
        }
        // MANY MORE
    }
}

My approach prints the numbers from 0 to 7, but I want the contents of the array.

My approach:

let navigation = [["Kunde", "person"], ["Wartung", "gear"]]


NavigationView {
    List {
        ForEach(navigation.indices) { index in
            NavigationLink(destination: ListView(title: "\(index)")) {
                Label("\(self.index)", systemImage: "\(index)")
            }
        }
    }
}

George · Accepted Answer · 2021-11-30 01:50:02Z

2

You are only giving the index. You could do navigation[index][0] for example to get the title, but this is quite tedious.

A better way is to create a custom type, plus it will also be Identifiable so it's safer to use on a mutating list of items.

Code:

struct ContentView: View {
    private let navigation = [
        NavigatableView("Kunde", image: "person"),
        NavigatableView("Wartung", image: "gear")
    ]

    var body: some View {
        NavigationView {
            List {
                ForEach(navigation) { newView in
                    NavigationLink(destination: ListView(title: newView.title)) {
                        Label(newView.title, systemImage: newView.image)
                    }
                }
            }
        }
    }
}

struct NavigatableView: Identifiable {
    let id = UUID()
    let title: String
    let image: String

    init(_ title: String, image: String) {
        self.title = title
        self.image = image
    }
}

answered Nov 30, 2021 at 1:50

George

31k12 gold badges100 silver badges157 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Nikolass Over a year ago

I'm going to stick with your first version because it is shorter. THX!

George Over a year ago

@Nikolass Shorter code doesn't mean better code. You will have animation issues if you change navigation since you are identifying by index rather than a unique identifier. It's also not particular safe, because these arrays could be an unpredictable size.

Collectives™ on Stack Overflow

ForEach loop in Swift to use content of 2D array

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related