声明可选类型JournalEntry后仍报非可选值与nil比较警告问题

嗨，我来帮你捋清楚这个问题的根源～

问题出在哪？ #

你写的这段代码里，guard let entry = entryForEdit, entry != nil犯了一个逻辑冗余的错误：

• guard let entry = entryForEdit本身就是安全解包操作——只有当entryForEdit不为nil时，才会把它的值赋给非可选类型的entry，并进入后续代码块；如果entryForEdit是nil，直接执行else里的return。
• 这时候entry已经是确定有值的JournalEntry类型，不可能是nil，你再额外加entry != nil的判断完全多余，编译器自然会警告你“这个比较永远返回true”——因为entry根本没有为nil的可能。

修正后的代码 #

只需要把多余的entry != nil去掉就可以解决警告，同时逻辑完全不受影响：

class AddJournalEntryVC: UIViewController, UITextViewDelegate { 
    var willEdit:Bool? 
    var entryForEdit:JournalEntry? = nil 

    override func viewDidLoad() { 
        super.viewDidLoad() 
        if isEditing { 
            guard let entry = entryForEdit else { 
                // 当entryForEdit为nil时直接返回
                return 
            } 
            dateLabel.text = entry.dateString! 
            timeLabel.text = entry.timeString! 
            timestamp = entry.timestamp! 
        } 
    } 
}

额外小建议 #

你后面用!强制解包entry.dateString、entry.timeString这些属性，其实有崩溃风险——如果这些属性是nil，程序会直接闪退。建议换成更安全的写法，比如用空合并运算符设置默认值：

dateLabel.text = entry.dateString ?? "未设置日期"
timeLabel.text = entry.timeString ?? "未设置时间"

内容的提问来源于stack exchange，提问作者Cody Lucas