【SwiftUI】構造体配列を別関数で動的に入力したい。

Question

いつもお世話になっております。
SwiftUI初心者です。

Bool型の配列と、構造体の配列の２つがあり
Bool型の配列でTrueの場合、同じ添字の構造体要素（name）を抽出し
リストで表示するプログラムを組んでおりますが
エラーが出てしまい、処理を継続することができません。

どうすれば良いか分からず、ご教示いただけますと幸いです。

エラー内容

Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value

プログラム説明

ユーザーの方で設定するBool型配列をUserDefaultsで保存/読み込みを行います。
下記例ではBool型配列でTrueのとき同様の添字のFruitsListとなるマスタを抽出したいと思っております。

例）
【FruitsList】
りんご
バナナ
みかん
グレープフルーツ

【BoolList】
True
False
False
False

上記のとき、りんごをリストに表示

BoolListですべてFalseのとき、リストには何も表示しません。

以下、プログラムです。

プログラム

import SwiftUI

public var FruitsBool:[Bool] = UserDefaults.standard.object(forKey: "FruitsBool") as? [Bool] ?? [false,false,false,false]

struct ListArray: Identifiable {
    let id: Int
    let name: String
}

class FruitsList {
    private init() {}
    
    static let shared = FruitsList()
    
    let matrix: [ListArray] = [
        ListArray(id: 1 , name: "りんご"),
        ListArray(id: 2 , name: "バナナ"),
        ListArray(id: 3 , name: "みかん"),
        ListArray(id: 4 , name: "グレープフルーツ"),
    ]
}

struct ContentView: View {
    
    var matrix:[ListArray]? = nil
    
    var body: some View {
        
        NavigationView {
            
            List(ListArrayAdd()) { ListArray in
                
                Text(ListArray.name)
                
            }
        }
    }
}

func ListArrayAdd() -> [ListArray] {
    
    let FruitsList1 = FruitsList.shared
    var matrix:[ListArray]? = nil
    var i:Int = 0
    
    for n in 0...3 {
        
        if FruitsBool[n] == true {
            
            matrix![i] = ListArray(id: FruitsList1[n].id , name: FruitsList1[n].name)
            i += 1
        }
    }
    
    return matrix!
    
}


struct ContentView_Previews: PreviewProvider {
    static var previews: some View {
        ContentView()
    }
}

Comments

[hoshi-takanori]

nil を ! したら当然そうなるとしか…。ListArrayAdd で本当にしたいことは何ですか？

[lucia0320]

ご確認ありがとうございます。
別途端末にUserDefaultsで保存しているBool型の配列データがあり、
その配列の要素でTrueのものだけを抽出し、リスト化することが目的となります。

[hoshi-takanori]

ごめんなさい、よく分からないんですが、Bool の配列ってことは、例えば [true, false, true, false, true] から、[true, true, true] を抽出したいのでしょうか？

[lucia0320]

大変申し訳ございません。
簡潔に書こうと思ったのですが、かえって分かりづらい質問となってしまいました。
質問内容を修正しましたのでご確認いただけますと幸いです。

[hoshi-takanori]

修正ありがとうございます。

とりあえず、ContentView の var matrix:[ListArray]? = nil は使ってないようなので、不要ですね。
次に、ListArrayAdd の var matrix:[ListArray]? = nil ですが、初期値を nil にする必要がないと思います。
nullable である必要もないので、var matrix:[ListArray] = [] としましょう。
それから、配列に要素を追加するには append メソッドを使います。変数 i も不要です。

(FruitsBool がグローバル変数なのは…、動いてるみたいだからとりあえずいいことにしますか…。)

[lucia0320]

ありがとうございます。

matrix!.append(ListArray(id: FruitsList1[n].id , name: FruitsList1[n].name))

上記にこちらに書き換えてみたのですが以下のエラーが出てしまいます。
Value of type 'FruitsList' has no subscripts

let FruitsList1 = FruitsList.shared　を
let FruitsList1:[ListArray] = FruitsList.shared　と
書き換えてもダメのようです・・・。

構造体配列に要素追加する際の記述法をご存知でしたらご教示頂けますでしょうか。
こちら調べてみたのですが、該当する記事が見当たりませんでした。

[hoshi-takanori]

matrix!.append じゃなくて、matrix を [ListArray]? から [ListArray] に変更した上で matrix.append しましょう。
また、FruitsList.shared　は FruitsList 型で、そこから [ListArray] を取り出すには matrix プロパティにアクセスする必要があります。(そもそも FruitsList 構造体の存在意義がよく分かりません…。)

[lucia0320]

ありがとうございます。
解決いたしましたのでCloseとさせていただきます。

Answer 1

とりあえずこういうことでしょうか。

func ListArrayAdd() -> [ListArray] {

    let FruitsList1 = FruitsList.shared
    var matrix: [ListArray] = []

    for n in 0...3 {
        if FruitsBool[n] == true {
            matrix.append(FruitsList1.matrix[n])
        }
    }

    return matrix
}

またはこうも書けます。(書かなくていいです。)

func ListArrayAdd() -> [ListArray] {
    return FruitsList.shared.matrix.enumerated().filter { (n, _) in FruitsBool[n] }.map { (_, x) in x }

    // または
    // return zip(FruitsBool, FruitsList.shared.matrix).compactMap { $0 ? $1 : nil }
}

Comments

[lucia0320]

ありがとうございます。
無事、動くことができました！
他の記述方法につきましてもありがとうございます。