急募【Swift】Todoアプリ

Question

簡単なTableViewを使用したTodoアプリについて質問です。

アプリの機能としてはDBにRealm、NavigationControllerを親にし、
一覧画面にTableViewControllerを配置。
セルを選択すると編集画面にコード遷移し、編集した値を一覧画面に返す流れです。

ここから質問ですが
アプリ内で登録されたTodo内容の表示をnumberOfRowsInSection、cellForRowAt、didSelectRowAt
で都度検索処理を入れているので、
例えばアプリ起動時に画面に７つセルが表示する場合、8回以上の検索が走ることになります。

データが少ないうちはいいですが、登録データが増えていくと負荷も比例して増えていくので、
検索処理を行うのを画面表示するときだけにしたいのですが、考えられる方法はありますでしょうか？

import UIKit
import RealmSwift

class TodoTableViewController: UITableViewController, EditViewControllerDelegate {

    // MARK: - Properties

    let realm = try! Realm()


    // MARK: - LifeCycle

    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.reloadData()
    }
    
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        settingView()
        
        
    }
    
    func settingView() {
        //最初から編集ボタンを表示させない
        tableView.isEditing = false
        //セルをタップできるようにする
        tableView.allowsSelectionDuringEditing = true
        //並び替え、削除ボタンを表示 タイトル名変更　色
        navigationItem.leftBarButtonItem = editButtonItem
        navigationItem.leftBarButtonItem?.tintColor = .blue
        //Realmのパス
        print(Realm.Configuration.defaultConfiguration.fileURL!)
    }
    
    
    // MARK: - Table view data source
    
    //セクションラベルの数
    override func numberOfSections(in tableView: UITableView) -> Int {
        return 1
    }
    
    //セルの行数
    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        //Realmをインスタンス化して使えるようにする
        let realms = realm.objects(Todo.self)
        
        return realms.count
    }
    
    // セルの中身、データを表示する　//withIdentifierを設定した名前に合わせる
    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)
        let realms = realm.objects(Todo.self)
        let todoArray = realms[indexPath.row]
        cell.textLabel?.text = todoArray.text
        cell.selectionStyle = .none
        
        return cell
    }
    
    //セルがタップされた時の処理
    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let realms = realm.objects(Todo.self)
        
        // var nextTextArray = realms[indexPath.row]
        //タップした時にその配列の番号を取り出して値を渡す
        let editVC = storyboard?.instantiateViewController(identifier: "EditView") as! EditViewController
        // editVC.todoString = nextTextArray.text
        editVC.editTodo = realms[indexPath.row]
        editVC.indexPath = [indexPath.row]
        editVC.editDelegate = self
        navigationController?.pushViewController(editVC, animated: true)
        
    }
    
    //セルの並び替え
    override func tableView(_ tableView: UITableView, moveRowAt fromIndexPath: IndexPath, to: IndexPath) {
    }
    
    //セルを編集できるようにするかどうか設定
    override func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }
    
    //データ削除設定
    override func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == .delete {
            let realm = try! Realm()
            let todos = realm.objects(Todo.self)
            let todo = todos[indexPath.row]
            
            try! realm.write {
                realm.delete(todo)
            }
            tableView.deleteRows(at: [indexPath], with: .fade)
        }
    }
    
    // MARK: - function
    func tapEditButton(indexPath: IndexPath) {
        let todos = realm.objects(Todo.self)
        tableView.reloadRows(at: [indexPath], with: .left)
        
        print("呼ばれた")
        
    }

    //    編集ボタン
    @IBAction func tapAddButton(_ sender: Any) {
        let realm = try! Realm()
        let alertController = UIAlertController(title: "Todoを追加しますか？", message: nil, preferredStyle: .alert)
        let action = UIAlertAction(title: "追加", style: .default){
            (void) in
            let textField = alertController.textFields![0] as UITextField
            
            if let text = textField.text {
                let todo = Todo()
                todo.text = text
                
                try! realm.write {
                    realm.add(todo)
                }
                self.tableView.reloadData()
            }
        }
        let cancel = UIAlertAction(title: "キャンセル", style: .cancel, handler: nil)
        alertController.addTextField{(textField) in
            textField.placeholder = "Todoの名前を入れてください。"
        }
        alertController.addAction(action)
        alertController.addAction(cancel)
        
        present(alertController, animated: true, completion: nil)
    }
   
}
 

import UIKit
import RealmSwift

protocol EditViewControllerDelegate {
    func tapEditButton(indexPath: IndexPath)

}


class EditViewController: UIViewController {

    // MARK: - Properties

    var editTodo = Todo()
    var indexPath = IndexPath(row: 0, section: 0)
    var editDelegate: EditViewControllerDelegate?
    
    @IBOutlet weak var todoTextField: UITextField!
  
    // MARK: - LifeCycle

    override func viewDidLoad() {
        super.viewDidLoad()
        
        todoTextField.text = editTodo.text
//        navigationItem.rightBarButtonItem = UIBarButtonItem(title: "更新", style: .plain, target: self, action: #selector(rowUpdata))

    }
    
    // MARK: - function
    
    @IBAction func tapEditButton(_ sender: Any) {
        //タップした時にその配列の番号を取り出して値を渡す
        let todoVC = storyboard?.instantiateViewController(identifier: "TodoTableView") as! TodoTableViewController
        let realm = try! Realm()
        let todos = realm.objects(Todo.self)


        try! realm.write {
            editTodo.text = todoTextField.text!
            
        }
        
        editDelegate?.tapEditButton(indexPath: indexPath )

        navigationItem.leftBarButtonItem?.isEnabled = false
        navigationController?.popViewController(animated: true)
    }
    
    
}

Comments

[gentaro]

ここでは【急募】とかいう自分勝手な要求をするとヘイトを集めがちです。
急いでるならお金出してお仕事として請け負ってくれる人に頼みましょう。

[h.h]

gantaroさんはteratailの運営の人、もしくはサイトの警備員でしょうか？
teratailとはプログラミングに特化したQ&Aサイトと認識しているのですが、
あなたがルールを作成している側ではないなら質問内容に対する意見は必要ないです。
ありがとうございます。

[fiveHundred]

gentaroさんのような考えを持っている方はここでは多いです。
ルール以前の問題として、自分勝手な人と思われて、回答を避けられることによる不利益のほうが大きいと思いますがどうでしょう。
（無論、回答が要らないというのであればその限りではありません）

[h.h]

なぜ多いとわかるのですか？
併せて、なぜ急募という言葉を使うと
自分勝手な人になるのでしょうか？
特に理由も抽象的で、
ただ批判してマウント取りたいだけにしか感じられません。

[gentaro]

私がどう思うかと言うより、過去に何度も同じ指摘を目にした事があるから親切心でアドバイスしてあげたつもりだし、まともな理解力があればガイドライン読めば指摘の趣旨ぐらいわかると思うけど、低評価とコメント見ても理解できない残念な人なら別にいいや。
これがマウントにしか見えてないんだよね？
じゃあ会話が成立しないから無視して今後も【急募】ってつけときゃ良いんじゃない？
なんでてめーの要望に急いで答えなきゃいけないんだよ、と思って回答できてもしたくない人は一定数居ると思うけど、まぁ頑張って。
回答を得るよりもマウント取った取られたの方が大事なんだよね。本末転倒だと思うけど。

Answer 1

少なくとも、現状のコードをあらかじめご提示していただいた方が良かったかと思います。

要点は、毎回 realm.object() でアクセスすることの問題点かと推測します。

Realm の扱いで重要なのは、object() で得られる値は managed object となり、アクセスするたびに常に最新のデータを得ることが可能だということです（オートアップデート、もしくはライブアップデート）。

• Working with Realm objects - Auto-updating objects

ただし、object() が返す値については中身の順番は必ずしも保証されていませんので、モデルにプライマリーキーを設定し、常にソートした状態で得られるような工夫は必要です。

加えて、プロパティの値が更新されたかどうかについては、オブザーバを設定する必要があり、たとえばその中で tableView.reloadData()を呼び出すなどの工夫は必要となります。

• Notifications - Realm notifications

これらを背景に簡単なサンプルを作ると、例えば次のような感じになるのではないでしょうか。

ただし、エラーハンドリングは全て try! で簡略化していますし、オブザーバの処理もクロージャに渡される値を無視していますので、そのあたりについては質問者さんの h.h さんで工夫される必要があるかと思います。

import UIKit
import RealmSwift

class Todo: Object {
    @objc dynamic var task = ""
    @objc dynamic var id = 0
    
    override static func primaryKey() -> String? {
        return "id"
    }
}

class ViewController: UIViewController {
    var todos: Results<Todo>!
    // 更新通知を受け取るための Token
    // see https://realm.io/docs/swift/latest/#object-notifications
    var token: NotificationToken!
    
    @IBOutlet weak var todoTableView: UITableView! {
        didSet {
            todoTableView.dataSource = self
            todoTableView.register(UITableViewCell.self, forCellReuseIdentifier: "cell")
        }
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        let realm = try! Realm()
        todos = realm.objects(Todo.self).sorted(byKeyPath: "id")
        
        // 更新されたら行う処理を記述する。
        // 詳しくは -> https://realm.io/docs/swift/latest/#object-notifications
        token = realm.observe { _, _ in
            self.todoTableView.reloadData()
            
        }
    }
    
    // TODO を加えたつもり
    @IBAction func addButton(_ sender: UIButton) {
        var id:Int
        
        // ひとつもデータが存在しないときには id を 1 にする。
        // すでに何らかのデータが存在したときには、一番新しい id に 1 加えた値を次の id にする
        if let firstTodo = todos.last {
            id = firstTodo.id + 1
        } else {
            id = 1
        }
        
        // 新しい TODO を加える
        let todo = Todo()
        todo.id = id
        todo.task = Int.random(in: 100...999).description
        
        let realm = try! Realm()
        
        try! realm.write {
            realm.add(todo)
        }
    }
}

extension ViewController: UITableViewDataSource {
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return todos.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
        
        cell.textLabel?.text = todos[indexPath.row].task
        
        return cell
    }
}

Comments

[h.h]

ご回答ありがとうございます。
申し訳ございません、ソースコード載せていたのですが削除してしまいました。
早速、試してみました所、ほとんど解決致しました。ありがとうございます！
もしお答えして頂けるなら、別画面で指定したセルを元画面でも更新させるreloadRowsを navigationControllerのpopViewControllerで行うことは出来ますでしょうか？

[TsukubaDepot]

これも、全体的な構造がわからないと的確なお答えは難しいのですが、viewWillAppear を override して解決するのであればその中で実行されるといいかと思いますし、あるいはデリゲートパターンなどを使って解決される方法もあるかもしれません。

[h.h]

お返事ありがとうございます。
TsukubaDepotさんのご提案の方法で解決致しました。