Relamでのエラー

Question

class TopViewViewController: UIViewController,UITableViewDataSource,UITableViewDelegate {
    

    @IBOutlet weak var TopViewTabelView: UITableView!
    //var item = [String]()
    let ToDo = TopTodo()
    var todoItem: Results<TopTodo>!
    
    
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // 永続化されているデータを取りだす
        do{
            let realm = try Realm()
            self.todoItem = realm.objects(TopTodo.self)
            self.TopViewTabelView.reloadData()
        }catch{
            
        }
        
        navigationController?.navigationBar.prefersLargeTitles = true //これがやりたかった設定やメモ帳のナビゲーションの可変のやつ
        navigationItem.title = "データベース"
        TopViewTabelView.tableFooterView = UIView(frame: .zero)
        self.view.backgroundColor = UIColor.rgb(r: 250, g: 204, b: 125, alpha: 1)
        TopViewTabelView.frame = CGRect(x: 0, y: 64, width: 375, height: 667)
        
    }
    
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        TopViewTabelView.reloadData()
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    
    @IBAction func addBtr(_ sender: Any) {
        let alert = UIAlertController(title: "タイトル", message: "メッセージ", preferredStyle: .alert)
        
        // OKボタンの設定
        let okAction = UIAlertAction(title: "OK", style: .default, handler: {
            (action:UIAlertAction!) -> Void in
            
            // OKを押した時入力されていたテキストを表示
            if let textFields = alert.textFields {
                
                // アラートに含まれるすべてのテキストフィールドを調べる
                for textField in textFields {
                    // 変数で持つ必要はないのではないでしょうか？
                    let newTodo = TopTodo()
                    newTodo.item = textField.text!
                    do{
                        let realm = try Realm()
                        
                        // トランザクション開始
                        try realm.write({ () -> Void in
                            realm.add(newTodo)
                            print("ToDo Saved")
                        })
                        // ここでトランザクションが終了
                        
                        // ここでtodoItemの更新
                        self.todoItem = realm.objects(TopTodo.self)
                        self.TopViewTabelView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
                        print(textField.text!)
                        
                    }catch{
                        print("Save is Faild")
                    }
                }
                self.TopViewTabelView.reloadData()
            }
        })
        alert.addAction(okAction)
        
        // キャンセルボタンの設定
        let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil)
        alert.addAction(cancelAction)
        
        // テキストフィールドを追加
        alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in
            textField.placeholder = "テキスト"
        })
        alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生
        
        self.present(alert, animated: true, completion: nil)
    }

    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return todoItem.count
        
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let Cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
        /*let now = NSDate() // 現在日時の取得
        
        let dateFormatter = DateFormatter()
        dateFormatter.locale = NSLocale(localeIdentifier: "en_US") as Locale? // ロケールの設定
        dateFormatter.dateFormat = "yyyy年MM月dd日 HH:mm"//:ss" // 日付フォーマットの設定

        let deterstring = dateFormatter.string(from: now as Date)
        print(deterstring) // -> 2014/06/25 02:13:18*/
       
        let object = todoItem[indexPath.row]
        Cell.textLabel?.text = object.item //+ deterstring
        Cell.detailTextLabel?.text = object.date.description
        //Cell.detailTextLabel?.text = deterstring
        
        return Cell
    }
    
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCellEditingStyle, forRowAt indexPath: IndexPath) {
        
        
        if(editingStyle == UITableViewCellEditingStyle.delete) {
            do{
                let realm = try Realm()
                try realm.write {
                    realm.delete(self.todoItem[indexPath.row])
                }
                TopViewTabelView.deleteRows(at: [indexPath], with: UITableViewRowAnimation.fade)
            }catch{
            }
            TopViewTabelView.reloadData()
        }
    }

}

import RealmSwift

class TopTodo: Object{
    // 管理用 ID。プライマリーキー
    @objc dynamic var id = 0
    @objc dynamic  var item = ""
    
    /// 最終更新日時
    @objc dynamic var date = NSDate()
    
     //id をプライマリーキーとして設定
    override static func primaryKey() -> String? {
        return "id"
    }

}

困っていること

セルのサブタイトルの更新時間を取得したく、オブジェクトクラスに@objc dynamic var date = NSDate()を追加したところ、numberOfRowsInSectionのreturn todoItem.counでエラーが起きてしまった。
以下がエラーの内容です。
Fatal error: Unexpectedly found nil while unwrapping an Optional value
実行すると落ちてしまいます。

Answer 1

その行で Unexpectedly found nil while unwrapping an Optional valueで落ちたと言うことは、todoItemがnilだったということです。

todoItemを設定している処理は

        do{
            let realm = try Realm()
            self.todoItem = realm.objects(TopTodo.self)
            self.TopViewTabelView.reloadData()
        }catch{

        }

です。
おそらくtodoItemがnilになった原因は、try Realm()の処理がエラーになり、self.todoItem = 〜の処理が実行されずにcatch節に実行が移ったのだと思います。
最悪なのは、do-catchの構文を使っているのに、catch節の中に何も書いていないことです。このため、ここでエラーが発生しているのにエラー処理を何もせず、エラーがなかったことにされ、todoItemがnilのまま続きが動作して質問の箇所でエラーが発生したのだと思います。
せめて、次のようにcatch節でエラーをprintしておけば、try Realm()がエラーになった原因がコンソールメッセージで出力されるはずです。

        }catch{
            print(error)
        }

そもそも try Realm()がエラーになったら、Realmが一切使えないということなので続行しても仕方がなく、この段階でエラーになった原因をしっかり調べる必要があります。
そのため、通常ここではdo-catchを使わず、次のように try!を使って、
エラー発生時はエラーメッセージを表示してすぐにアプリを停止させるのが一般的だと思います。

    let realm = try! Realm()
    self.todoItem = realm.objects(TopTodo.self)

---

try Realm()がエラーになった原因については、そのエラーメッセージを確認しないと正確なことはわかりませんが、上記の修正をして表示されたエラーメッセージにMigration is requiredといったようなメッセージが出ていたら、

オブジェクトクラスに@objc dynamic var date = NSDate()を追加した

ということなので、前回実行した時とデータ構造が変わり、前回のアプリ実行で保存していたデータが使用できなくてエラーになったということだと思います。

このアプリがまだユーザーにリリースされていないものであれば、今自分がテストに使っているシミュレータや実機からこのアプリを削除して再実行するだけで以前に保存していたデータは削除されるので、エラーは解消されると思います。

もし、既にこのアプリがユーザーにリリースされていて、今回はそのアプリのバージョンアップでデータ項目を追加しようとしているのなら、ユーザーが保存していたデータを引き継いで新しいデータ構造で処理できるよう「マイグレーション」という処理を実施する必要があります。

Comments

[退会済みユーザー]

懇切にご教授していただき有難うございます。
解決できました。
今回はまだ開発中の段階でしたのでマイグレーションを行う必要はありませんでした。
しかし、マイグレーションという処理は今後必要になっていくと思いますので勉強したいと思います。