Swift : 配列の空かどうかの判定（isEmpty）がうまくいかない

Question

#　問題点
UserDefaultsの配列が、空なのに、下のプログラムで空ではない表示されてしまいます
そして、UserDefaultsが空だからだとは思いますが、dbCells = deathblowNameArrayUD.array(forKey: "DeathBlowArray") as! [String]のプログラムの部分で落ちてしまいます

override func viewDidAppear(_ animated: Bool) {
        super.viewDidAppear(animated)
        if deathblowNameArrayUD.array(forKey: "DeathBlowArray")?.isEmpty == true {
            debugPrint("空")
            noItemAlert()
        }else {
            debugPrint("空ではない")
            dbCells = deathblowNameArrayUD.array(forKey: "DeathBlowArray") as! [String]
            print(dbCells)
        }
    }

#　補足
このプログラムを実行する前にこのサイトを見ながら、UserDefaultsを全てリセットしています。
dbCellsはTableViewの作成に使う変数です。

ご教授いただければ幸いですm(_ _)m

Answer 1

        if deathblowNameArrayUD.array(forKey: "DeathBlowArray")?.isEmpty == true {

この行ですが、deathblowNameArrayUD.array(forKey: "DeathBlowArray") の型は Array? です。「UserDefaultsを全てリセットしている」とのことですが、"DeathBlowArray" に対応する値が存在しない（または配列でない）場合には nil が返ります。ので、nil、空の配列、空でない配列の 3 種類に対応するコードを書く必要があります。

で、Array? の値 array に対して array?.isEmpty とすると結果は nil, true, false の 3 種類となり、== true で空の配列の場合のみがひっかかり、空でない配列に加えて nil（"DeathBlowArray" に対応する値が存在しない）の場合も else 側で処理されることになって as! で落ちます。（そもそも as! なんて落ちる可能性があるものを使うのは、Swift の型安全性を自分から捨ててるとも言えます。）

自分だったら以下のように書くと思います。

    override func viewDidAppear(_ animated: Bool) {
        super.viewDidAppear(animated)

        let dbCells = (deathblowNameArrayUD.array(forKey: "DeathBlowArray") as? [String]) ?? []
        if dbCells.isEmpty == true {
            debugPrint("空")
            noItemAlert()
        }else {
            debugPrint("空ではない")
            print(dbCells)
        }
    }

Comments

[Amuy]

ご回答ありがとうございます！
落ちることは無くなりました。
この質問の問題は解決したのですが...cellが表示されません...このことは他の質問で質問します

Answer 2

if deathblowNameArrayUD.array(forKey: "DeathBlowArray")?.isEmpty == true {

は、たぶん

if deathblowNameArrayUD.array(forKey: "DeathBlowArray") != nil && deathblowNameArrayUD.array(forKey: "DeathBlowArray")!.isEmpty {

というように評価されると思いますので、配列そのものがnilだとelseの方に行ってしまうのだと思います。

Comments

[MasakiHori]

これはそうではなくてOptionalの
static func ==(lhs: Wrapped?, rhs: Wrapped?) -> Bool
と解釈されます。
https://developer.apple.com/documentation/swift/optional/2950146
trueが .some(true) として扱われます。

結果は下の評価式のようになりますが。

[takabosoft]

はい、コンパイラとしてはその関数に入ると思います。その中の処理を推測するとたぶん下のような感じになるのではないかと思います。もちろん厳密には違うと思いますが。

Answer 3

if let dbCells = deathblowNameArrayUD.array(forKey: "DeathBlowArray"), !dbCells.isEmpty {
   // 空ではない
}
else {
  // nilか空
}