realmでTODOアプリを作った時のcheckmarkの表示について

Question

【現在作成中のアプリの概要】

TODOリストのアプリを作成しています。
データの永続化には『realm』を使用しています。

realmに記録するデータは下記のToDoModel.swiftに記載しています。

【現在詰まっている箇所】

セルを選択した時にチェックマークを表示できるようにさせたいのですが、うまくいきません。
また、チェックマークの有無もrealmで記録させたいのですが方法がわかりません。

自身で調べたところ、チェックマークの表示にbool型が絡んでくるのはわかりましたが解決には至りませんでした。
上記の実装を可能にする方法を知っている方がいらっしゃいましたら、ご教授の程よろしくお願い致します。

import UIKit
import RealmSwift



class ViewController: UIViewController,UITextFieldDelegate,UITableViewDelegate,UITableViewDataSource {

    
    
    @IBOutlet weak var todoTableView: UITableView!
    
    
    
  
    var itemList: Results<TodoModel>!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        let realmInstance = try! Realm()
        self.itemList = realmInstance.objects(TodoModel.self).sorted(byKeyPath: "order")
        
        //更新
        self.todoTableView.reloadData()
        
        
        // UITableViewDataSource を self に設定
        self.todoTableView.dataSource = self
        // UITableViewDelegate を self に設定
        self.todoTableView.delegate = self
        
        
        
        //常時編集状態にする(isEditing,allowsSelectionDuringEditing)
//        todoTableView.isEditing = true
        todoTableView.isEditing = false
        todoTableView.allowsSelectionDuringEditing = true
        
        // trueで複数選択、falseで単一選択
        todoTableView.allowsMultipleSelection = true
        
        
        
        
    }
    // Add ボタンをクリックした際に実行する処理
    //セルに文字を入力する
    @IBAction func tapAddButton(_ sender: Any) {
        
        var textField = UITextField()
        // ① UIAlertControllerクラスのインスタンスを生成　参照サイト（https://qiita.com/funacchi/items/b76e62eb82fc8d788da5）
        let alert = UIAlertController(title: "タスクを追加", message: "", preferredStyle: .alert)
        // ② Actionの設定
        let action = UIAlertAction(title: "追加する", style: .default) { (action) in
        let instancedTodoModel:TodoModel = TodoModel()
        instancedTodoModel.todo = textField.text
                    
        print(instancedTodoModel)
        let realmInstance = try! Realm()
        try! realmInstance.write{
        realmInstance.add(instancedTodoModel)
        }
        self.todoTableView.reloadData()
        }
                
        alert.addTextField { (alertTextField) in
        alertTextField.placeholder = "タスクを入力"
        textField = alertTextField
        }
        // キャンセルボタンの追加
        let cancelAction: UIAlertAction = UIAlertAction(title: "キャンセル", style: UIAlertAction.Style.cancel, handler:{
        // ボタンが押された時の処理を書く（クロージャ実装）
        (action: UIAlertAction!) -> Void in
        print("Cancel")
        })
        // ③　UIAlertControllerにActionを追加
        alert.addAction(cancelAction)
        alert.addAction(action)
        present(alert, animated: true, completion: nil)
 
    }
    // Remove All ボタンをクリックした際に実行する処理
    //セルに表示されている文字をすべて削除する
    @IBAction func tapRemoveAllButton(_ sender: Any) {
        let realmInstance = try! Realm()
        try! realmInstance.write{
            //↓以下のコードを有効にするとrealmに記録しているすべてのデータが消えてしまう
//            realmInstance1.deleteAll()
            //↑上記のコードだとrealmに記録しているすべてのデータが消えてしまうため下記のコードに書き換えた↓
            realmInstance.delete(itemList)
        }
        self.todoTableView.reloadData()
    }
    
    //セルの編集モードをONにする
    @IBAction func edit(_ sender: Any) {
        todoTableView.isEditing = true
    }
    
    // セルが選択された時に呼び出される
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        print("セル(indexPath)が選択されました")
        //テーブルビューのセルをクリックしたら、アラートコントローラを表示
//        showAlertController(indexPath)
        let item: TodoModel = self.itemList[(indexPath as NSIndexPath).row]
        
        item.done = !item.done
// Realm に保存したデータを UIAlertController に入力されたデータで更新
        let realmInstance = try! Realm()
        try! realmInstance.write{
            itemList[indexPath.row].done = item.done 
        }
        // リロードしてUIに反映
        self.todoTableView.reloadData()
    }
    
    //セルの選択が外れた時に呼び出される
    func tableView(_ tableView: UITableView, didDeselectRowAt indexPath: IndexPath) {

        }
    
    func numberOfSectionsInTableView(tableView: UITableView) -> Int {
            return 1
        }
    //セルの数を返す
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return self.itemList.count
    }
    
    //必須メソッド_埋め込むセルの値を設定
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        let testCell:UITableViewCell = tableView.dequeueReusableCell(withIdentifier: "testCell")!
        let item: TodoModel = self.itemList[(indexPath as NSIndexPath).row]
        testCell.textLabel?.text = item.todo
        //チェックマークを表示する処理ーitemのdoneがtrueだったら表示falseだったら非表示
        testCell.accessoryType = item.done ? .checkmark : .none
        return testCell
    }
    //並べ替え可能にするメソッド
    func tableView(_ tableView: UITableView, canMoveRowAt indexPath: IndexPath) -> Bool {
            return true
        }
    
    //realmでtableviewのセルを並び替えるメソッド（https://ja.stackoverflow.com/questions/35653/realmをデータソースにしてテーブルビューの並べ替えをしたい）
    //並べ替え結果を処理するメソッド
    func tableView(_ tableView: UITableView, moveRowAt sourceIndexPath: IndexPath, to destinationIndexPath: IndexPath) {
            // TODO: 入れ替え時の処理を実装する（データ制御など）
        print("並び替え開始")
        let realmInstance = try! Realm()
        try! realmInstance.write{
                let sourceObject = itemList[sourceIndexPath.row]
            print("最初の行",sourceObject.order)
                let destinationObject = itemList[destinationIndexPath.row]
           
                let destinationObjectOrder = destinationObject.order
       

                if sourceIndexPath.row < destinationIndexPath.row {
                    // 上から下に移動した場合、間の項目を上にシフト
                    for index in sourceIndexPath.row...destinationIndexPath.row {
                        let object = itemList[index]
                        object.order -= 1
                    }
                } else {
                    // 下から上に移動した場合、間の項目を下にシフト
                    for index in (destinationIndexPath.row..<sourceIndexPath.row).reversed() {
                        let object = itemList[index]
                        object.order += 1
                    }
                }
                // 移動したセルの並びを移動先に更新
            sourceObject.order = destinationObjectOrder
            print("最後の行",sourceObject.order)
//            self.todoTableView.reloadData()
            }
        
        }
    
    //以下のコードを追加すると削除モードが表示されなくなる（editingStyleForRowAt,shouldIndentWhileEditingRowAt）
    //編集状態の見た目を編集(左側の＋やーを表示)これを消すとマイナスボタンが表示される
//    func tableView(_ tableView: UITableView, editingStyleForRowAt indexPath: IndexPath) -> UITableViewCell.EditingStyle {
//            return .none //表示させない。
//        }
    
    
    //編集モード時に左にずれるか。
    func tableView(_ tableView: UITableView, shouldIndentWhileEditingRowAt indexPath: IndexPath) -> Bool {
//            return false//ずれない。
            return true//ずれる
        }

//     テーブルビューの編集を許可
    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }

    // テーブルビューのセルとデータを削除
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == UITableViewCell.EditingStyle.delete {
            // データを削除
            let realmInstance = try! Realm()
            try! realmInstance.write {
                realmInstance.delete(itemList[indexPath.row])
            }
            // セルを削除
            tableView.deleteRows(at: [indexPath as IndexPath], with: UITableView.RowAnimation.automatic)
        }
    }
    
    
     //テーブルビューのセルをクリックしたら、アラートコントローラを表示する処理
    func showAlertController(_ indexPath: IndexPath){
        let alertController: UIAlertController = UIAlertController(title: "ToDo を編集", message: "", preferredStyle: .alert)
        // アラートコントローラにテキストフィールドを表示 テキストフィールドには入力された情報を表示させておく処理
        alertController.addTextField(configurationHandler: {(textField: UITextField!) in
            textField.text = self.itemList[indexPath.row].todo})
        
        // アラートコントローラに"OK"ボタンを表示 "OK"ボタンをクリックした際に、テキストフィールドに入力した文字で更新する処理を実装
        alertController.addAction(UIAlertAction(title: "OK", style: .default, handler: {
            (action) -> Void in self.updateAlertControllerText(alertController,indexPath)
        }))
        // アラートコントローラに"Cancel"ボタンを表示
        alertController.addAction(UIAlertAction(title: "Cancel", style: .cancel, handler: nil))
        self.present(alertController, animated: true, completion: nil)
        
        //removeTask
    }

    // "OK"ボタンをクリックした際に、テキストフィールドに入力した文字で更新
    func updateAlertControllerText(_ alertcontroller:UIAlertController, _ indexPath: IndexPath) {
        // guard を利用して、nil チェック
        guard let textFields = alertcontroller.textFields else {return}
        guard let text = textFields[0].text else {return}

        // UIAlertController に入力された文字をコンソールに出力
        print(text)

        // Realm に保存したデータを UIAlertController に入力されたデータで更新
        let realmInstance = try! Realm()
        try! realmInstance.write{
            itemList[indexPath.row].todo = text
        }
        self.todoTableView.reloadData()
    }
}

import Foundation
import RealmSwift

class TodoModel: Object{
    @objc dynamic var todo: String? = nil
    @objc dynamic var order: Int = 0 // 並べ替えのためのカラムが必要
    @objc dynamic var done: Bool = false
  
}

Comments

[hoshi-takanori]

うまくいかないとは、具体的に何がどううまくいかないのでしょうか？
Realm よく分かりませんが、item.done を反転させる処理も realmInstance.write する必要がありそう…。

[tyu]

現状は、セルをタップしてチェックマークを表示させようとすると、「Thread 1: "Attempting to modify object outside of a write transaction - call beginWriteTransaction on an RLMRealm instance first."」というエラーが表示されてしまいます。

item.done を反転後の realmInstance.write 処理も必須ですね。こちらはrealmのBool値の保存方法を調べて実装してみます。

[hoshi-takanori]

もしかして、write transaction というのがまさに realmInstance.write のことなのでは？

[tyu]

ありがとうございます！
realmInstance.writeの処理を追加したらエラーもなくなるかもしれませんね！
一応realmのbool値の保存について調べているのですが、まだ解決に至っていません。
「こうしたらいいんじゃないか」等がありましたら、教えていただけないでしょうか...

[hoshi-takanori]

Realm 知らないので一般論ですが、item.done = !item.done で、item.done の値を反転 (元が true なら false、false なら true にする) して item.done を更新する処理になっていて、たぶん Realm でもそれでいいのでは…。

[tyu]

ご教授いただきありがとうございます！
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) 内に item.done の更新をrealmに更新する処理を追記したのですが（上記のコードを修正しました）、「Thread 1: "Attempting to modify object outside of a write transaction - call beginWriteTransaction on an RLMRealm instance first."」のエラーが出てしまいます...
realmって難しいですね

[hoshi-takanori]

item.done = !item.done が try! realmInstance.write { 〜 } の外にありますが、これを中に入れれば良いのでは。
(そして、たぶん itemList[indexPath.row].done = item.done は不要かと…。)

[TsukubaDepot]

回答を書く前にコメントを確認してよかったです。
hoshi-takanoriさんご指摘の通り、item.done = !item.done を write block の中に入れる必要があります。
itemList（SwiftのResult型）はマネージドオブジェクトでライブアップデートが効きますが、インスタンスやそのコピーに書き込み操作を行う場合には write block 内で行う必要があります。
itemList[indexPath.row].done = item.done も不要ですね（直前のコードで参照先に直接書き込んでいるので）。

あと、現在のコードだと、並び替えても操作は有効になりません。
TodoModel に order というプロパティがありますが、新しい ToDo を書き込むごとに、この order に常に最新で最大の番号を与えるようにしなければ、現在のロジックだと並び替えの結果が思った通りにならないはずです。

[tyu]

hoshi-takanoriさん、TsukubaDepotさん、ご教授いただきありがとうございます！
お二人のお陰でチェックマークの永続化ができました。もう少しrealmについて勉強します...

TsukubaDepotさんがおっしゃっているように並び替えの結果が思うようにいきません。
別で質問をあげますので、お時間がありましたら、再度ご教授お願いしたいです。

[TsukubaDepot]

では、とりあえずこのご質問は自己解決ということで処理をお願いできますでしょうか。
できれば、簡単に修正点を挙げていただくと、別の方の参考になるかと思います。

[tyu]

承知いたしました。いつもお力添えありがとうございます。

Answer 1

下記コードに修正することで解決に至りました。
hoshi-takanoriさん、TsukubaDepotさん、お力添え誠にありがとうございます。

// セルが選択された時に呼び出される
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        print("セル(indexPath)が選択されました")
        //テーブルビューのセルをクリックしたら、アラートコントローラを表示
//        showAlertController(indexPath)
        let item: TodoModel = self.itemList[(indexPath as NSIndexPath).row]
        
        
        
        // Realm に保存したデータを UIAlertController に入力されたデータで更新
        let realmInstance = try! Realm()
        try! realmInstance.write{
            item.done = !item.done
//            itemList[indexPath.row].done = item.done
        }
        // リロードしてUIに反映
        self.todoTableView.reloadData()
    }