カスタムチェックボックス(UIButton)の挙動がおかしい

Question

前提・実現したいこと

Todoアプリを制作しているのですが、UIButtonを使ったチェックボックスの挙動がおかしいので質問させていただきました。
アプリはRealmに保存してあるデータを読み込んで表示させているのですが、
初期表示時に「未チェック(false)」となっているチェックボックスを1回クリックしても、「チェック済(true)」状態になりません。
もう一度クリックすると、やっと「チェック済(true)」になります。

発生している問題・エラーメッセージ

1. チェックボックスはカスタムクラス内のdidsetでプロパティ監視をして作成しているのですが、

didset内でデバッグコードを書くと、oldValueの値に変更前の値と逆の物が入っています。
(本来ならtrueの筈なのにfalseが入っている)

1. チェックボックスをクリックして最初に入るクラスのsenderのBoolを調べると、本来なら「true」が入るはずなのに「false」になってしまっています。

該当のソースコード

import UIKit

class CheckBox: UIButton {
    // Images
    let checkedImage = UIImage(named: "check_on")! as UIImage
    let uncheckedImage = UIImage(named: "check_off")! as UIImage

    // Bool property
    
    var isChecked: Bool = false {

        didSet{
            print("old",oldValue,"new",isChecked)
            if isChecked == true {
                self.setImage(checkedImage, for: UIControl.State.normal)
            } else {
                self.setImage(uncheckedImage, for: UIControl.State.normal)
            }
            print("old",oldValue,"new",isChecked)
        }
    }
    
    func nowChecked(check:Bool) {
        if check == true {
            self.setImage(checkedImage, for: UIControl.State.normal)
        } else {
            self.setImage(uncheckedImage, for: UIControl.State.normal)
        }
    }

    override func awakeFromNib() {
        self.addTarget(self, action:#selector(buttonClicked(sender:)), for: UIControl.Event.touchUpInside)
    }


    @objc func buttonClicked(sender: UIButton) {
        if sender == self {
            isChecked.toggle()
        }
    }
}

import UIKit

class TodoCell:UITableViewCell, UITextFieldDelegate{

    @IBOutlet weak var checkBtn: CheckBox!
    @IBOutlet weak var todoText: UITextField!
    @IBOutlet weak var infoBtn: UIButton!
    
    internal func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        textField.resignFirstResponder()
        return true
    }
    
    let checkedImage = UIImage(named: "check_on")! as UIImage
    let uncheckedImage = UIImage(named: "check_off")! as UIImage
    
    var todoIndex = 0
    var todoId = ""
    var tododate = Date()
    

    
    @IBAction func checkView(_ sender: CheckBox) {
        let check:Bool = sender.isChecked
        print(check,"に変更")
    
        InfoHelper().update(id:todoId,title:todoText.text!, date:tododate, check: check)

    }
}

import UIKit
import RealmSwift

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource, UIViewControllerTransitioningDelegate {
    
    var itemList: Results<TodoItem>!
    let realm = try! Realm()
    var token:NotificationToken!
    var index = 0
    var sendIndex = 0
    
    @IBOutlet weak var tableView: UITableView!
    
    override func viewDidLoad() {
        
        // realmのリストを取得
        itemList = realm.objects(TodoItem.self).sorted(byKeyPath: "date")
        token = realm.observe{ notification, realm in
            
            //変更があった場合にtableViewを更新
            self.tableView.reloadData()
            super.viewDidLoad()
        }
    }
    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }
    
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == .delete {
            InfoHelper().deleteItem(item: itemList[indexPath.row], token: token)
            tableView.deleteRows(at: [indexPath], with: .automatic)
        }
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return itemList.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "todoCell", for: indexPath) as! TodoCell
        let item = itemList[indexPath.row]
        cell.todoId = item.id
        cell.todoText.text = item.title
        cell.tododate = item.date
        cell.checkBtn.nowChecked(check: item.check)
        cell.infoBtn.tag = indexPath.row
        cell.todoText.tag = Int(item.id)!
        
        // テキスト入力が終わったタイミングでメソッド走るように指定
        cell.todoText.addTarget(self, action: #selector(self.textFieldEditingChanged(_:)), for: .editingDidEndOnExit)
        
        print(item.title,item.check)
        
        return cell
    }

試したこと

デバッグコードを入れて正しい値が入っているか確認しました。

補足情報（FW/ツールのバージョンなど）

Swift 5.4
Xcode 12.5.1

Comments

[hoshi-takanori]

ViewController で nowChecked を呼ぶんじゃなくて isChecked を設定すべきでは。

[koto]

以前はその表記（cell.checkBtn.isChecked = item.check）にしていたのですが、それだとチェックボックスを押しても反応しなくなってしまう(Booleanの値ごと変わらない)ため、nowCheckedのメソッドを新たに作り、それを呼び出していました。

[hoshi-takanori]

というか、ボタン押されたら Realm のデータを更新する必要があるのでは…。

[koto]

言葉足らずですみません、RealmのデータはTodoCell内の「@IBAction func checkView」で更新しています。
チェックボタンを押すと、InfoHelper()というクラスの更新メソッドに値が渡るようにしています。

[hoshi-takanori]

ボタンを押すと CheckBox クラスの buttonClicked と、TodoCell クラスの checkView の両方が呼ばれるってことですか？ その場合、どっちが先に呼ばれるかによって挙動が変わるのでは。buttonClicked が先に呼ばれれば、InfoHelper().update に渡る check の値は新しい値だけど、checkView が先なら古い値になるのでは。

[koto]

両方が呼ばれています。
確認したところ、先に呼ばれているのはcheckViewの方でした。
buttonClickedの方が先に呼ばれるようにしたいのですが、
IBActionを優先しない書き方がわからずにいます…。

[hoshi-takanori]

たぶん順番を変えるのは難しい (というか、順番は保証されない) と思うので、CheckBox クラスの buttonClicked を削除して、TodoCell クラスの checkView の先頭で sender.isChecked.toggle() してしまうのが早い気がします。

[koto]

その方法で出来ました！教えていただきありがとうございます！！

Answer 1

「質問への追記・修正の依頼」にて教えていただいた方法で解決しました。

1.「cell.checkBtn.nowChecked(check: item.check)」を
「cell.checkBtn.isChecked = item.check」に変更
2.CheckBoxクラスの「nowChecked」 「buttonClicked」を削除
3.TodoCellクラスのcheckView の先頭に「sender.isChecked.toggle()」を追加