Realmのデータの追加について

Question

import UIKit
import RealmSwift

extension UIColor {
    class func rgb(r: Int, g: Int, b: Int, alpha: CGFloat) -> UIColor{
        return UIColor(red: CGFloat(r) / 255.0, green: CGFloat(g) / 255.0, blue: CGFloat(b) / 255.0, alpha: alpha)
    }
}

class TopViewViewController: UIViewController,UITableViewDataSource,UITableViewDelegate {
    

    @IBOutlet weak var TopViewTabelView: UITableView!
    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)
        
    }
    
   
    
    
    @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 {
                    //self.item.insert(textField.text!, at: 0)
                
                    let newTodo = self.ToDo
                    do{
                        let realm = try Realm()
                        try realm.write({ () -> Void in
                            realm.add(newTodo)
                            print("ToDo Saved")
                            self.ToDo.item.append(textField.text!)
                            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 dateString = dateFormatter.string(from: now as Date)
        print(dateString) // -> 2014/06/25 02:13:18*/
        
        let object = todoItem[indexPath.row]
        Cell.textLabel?.text = object.item
        Cell.textLabel?.textAlignment = NSTextAlignment.center
        /*Cell.detailTextLabel?.text = dateString
        Cell.detailTextLabel?.textAlignment = NSTextAlignment.right*/
        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()
        }
    }

    

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }

}

import RealmSwift

class TopTodo: Object{
    @objc dynamic  var item = ""
}

実現したいこと

tableViewに追加したデータを永続的に保存したい。

困っていること

現状ではtableViewにalertでデータを追加しているのですが一つしかデータを追加できなくて困っています。2つ目を追加しようとボタンをタップすると落ちていまう。（理由が分からない)
また、todoの一覧部分の永続保存の処理までは一応出来たが遷移先のtableViewのデータの保存の仕方が分からなくて困っている。(labelなど複雑な処理がある為)

gitに上げているので見て頂けると分かりやすいと思います。
https://github.com/haruka22/new-project

Answer 1

ご提示いただいたGithubのソースを試してみたところ
原因としてはinsertRowsをしたあとのtableViewの更新でDatasourceが持っているデータ数と表示するセルの数に不整合が生じているからだと思います。

// アラートに含まれるすべてのテキストフィールドを調べる
for textField in textFields {
    
    let newTodo = self.ToDo
    do{
        let realm = try Realm()
        try realm.write({ () -> Void in
            realm.add(newTodo)
            print("ToDo Saved")
            self.ToDo.item.append(textField.text!)
            self.TopViewTabelView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
            print(textField.text!)
        })
    }catch{
        print("Save is Faild")
    }
}
self.TopViewTabelView.reloadData()

try realm.write中でself.TopViewTabelView.insertRowsを呼んでいますが、
この時点だとRealmにはまだデータが追加されていない(=まだコミットされていないため)
一度トランザクションを抜けてコミットをする必要があります。

また、todoItemの更新が行われていないため、rowを追加してもTableViewの更新時点で
todoItemの数(numberOfRowsInSectionで使われている数)はTableViewでinsertしている分の数足りなくなっています。

下記のようにしたらどうなりますでしょうか？

// アラートに含まれるすべてのテキストフィールドを調べる
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()

あまり回答できる時間がなくて恐縮ですが、ご検討いただけましたら幸いです。
何卒よろしくお願い致します。

Comments

[退会済みユーザー]

ありがとうございます。見事に解決しました。処理を書く順番が問題だったんですね。
それと差し支えが無けれあば教えて頂きたいんですが、Objectのクラスに追加で@objc dynamic var date = NSDate()を追加したところnumberOfRowsInSectionでエラーが出てしまいました。
以下がエラーコードになります。
Fatal error: Unexpectedly found nil while unwrapping an Optional value
色々調べて見たのですがどこにもこのようなエラーで困っている人がいなく参考に出来る記事がなくて困っています。もしお時間ありましたらご教授して頂けると幸いです。

[退会済みユーザー]

どうしても分からない箇所がありご教授願いたいです。
Objectクラスにて、追加するセルの名前の他にサブタイトルで日時を追加したく、クラスの中に@objc dynamic var date = NSDate()を追加したところnumberOfRowsInSectionでアンラップエラーが起きてしまいその対処に苦慮しています。
エラーコードとしては、オプショル型が問題なのかなと思うのですがどこがいけないのかが分からないです。
差し支えなければご教授して頂けないでしょうか？
よろしくお願いします。
https://teratail.com/questions/132346

[退会済みユーザー]

お忙し中すいません。先日は懇切に教えて頂きありがとうございました。
また、分からないところがあり色々ネットなどでも調べて見たのですが解決方法が見つからず苦慮しています。もし宜しければお時間ある時にご教授願いたいです。
具体的に説明しますと、現在カスタムアラートを製作しており、その過程でアラート内に数字キーボードを用意しています。
そして、用意したキーボードからtextFieldsに値を入力していくのですがその際の条件分岐の仕方が分からなくて困っています。
やりたい事としては、セグメントコントロールで選択したテキストフィールドのみ入力を行えるようにしたいのですがそのやり方についてどうしても分からなくて困っています。
そのため、お時間のある時に御教授して頂けたら幸いです。何卒宜しくお願いいたします。
https://teratail.com/questions/135363

[退会済みユーザー]

お久しぶりです。お忙し中すいません。現在ios-chartsにて入力したデータを折れ線グラフにするグラフを製作しています。
大まかな部分は出来たのですが、入力した時の日付を横軸に表示させたいのですがどうも上手くいかなくて困っています。
そのため、もしお時間がありましたら見ていただけないでしょうか？