tableViewで表示されている項目の並び替えの不具合について realm

Question

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

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

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

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

下記ページの内容の続きで、新規のToDoを書き込むごとにorder に常に最新で最大の番号を与えるようにしなければならないのですが、どのように実装したら良いかというところで詰まっています。

リンク内容

上記の実装を可能にする方法を知っている方がいらっしゃいましたら、ご教授の程よろしくお願い致します。

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]
        
        
        // Realm に保存したデータを UIAlertController に入力されたデータで更新
        let realmInstance = try! Realm()
        try! realmInstance.write{
            item.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]

itemList が order でソート済みなら、その末尾の項目の order が最大のはずですから、それに + 1 すればいいのでは。

Answer 1

コメント欄でhoshi-takanoriさんご指摘の通りです。

一方、具体的に実装するとなると、itemList が空配列の場合もあるため、次のような感じで実装することになると思います。

空配列の場合は self.itemList.last は nil となるため、続くブロックは実行されません。
一方、TodoModel()はデフォルト値として order = 0 となっているため、空配列の場合には順番は 0 から再開されます。

        let action = UIAlertAction(title: "追加する", style: .default) { (action) in
            let instancedTodoModel = TodoModel()
            instancedTodoModel.todo = textField.text
            
            // MARK: order をインクリメントする
            if let lastItem = self.itemList.last {
                instancedTodoModel.order = lastItem.order + 1
            }
            
            let realmInstance = try! Realm()
            
            try! realmInstance.write {
                realmInstance.add(instancedTodoModel)
            }

            self.todoTableView.reloadData()
        }

Comments

[tyu]

TsukubaDepotさん、hoshi-takanoriさん、ご教授いただきありがとうございます。
希望していた昨日の実装ができました。
お二人のように自力で解決できるように今回の内容を復習してみます。
本当にありがとうございました。