全てのTextFieldが空の時を判別したい

Question

現在tableviewに複数のtextfieldがあり、全てのtextfieldが空の時は画面右上のbarbuttonitemをタップ不可にし、どれかに文字が1文字でも入るとbarbuttonitemをタップ可能にしたいのですが、以下のようにしてもうまくいきません。2文字目を入力した段階でbarbuttonitemがタップ可能になりますし、文字を全て消してもbarbuttonitemはタップ可能で、次に1文字入力するとbarbuttonitemがタップ不可能になるというように1アクション遅れます。

func textField(_ textField: UITextField, shouldChangeCharactersIn range: NSRange, replacementString string: String) -> Bool {
        let filledTextFieldCells = self.tableView.visibleCells.filter {
            if let cell = $0 as? Checkable {
                return cell.isFilledTextField()
            } else {
                return false
            }
        }
        if filledTextFieldCells.count == 0 {
            self.navigationItem.rightBarButtonItem?.isEnabled = false
        } else {
            self.navigationItem.rightBarButtonItem?.isEnabled = true
        }
        
        return true
    }

class NameEditTableViewCell: UITableViewCell, Checkable {

    @IBOutlet weak var nameTextField: UITextField!
    
    func isFilledTextField() -> Bool {
        return !self.nameTextField.text!.isEmpty ? true : false
    }
    
}

どうすれば入力と同時にtextfieldが空なのかどうかを判別することができるでしょうか？
どなたかわかる方がいれば教えていただきたいです。よろしくお願いします。

///回答を受けての追記///

func textField(_ textField: UITextField, shouldChangeCharactersIn range: NSRange, replacementString string: String) -> Bool {
        let newText = (textField.text as! NSString).replacingCharacters(in: range, with: string)
        
        let filledTextFieldCells = self.tableView.visibleCells.filter {
            if let cell = $0 as? Checkable {
                return cell.isFilledTextField(text: newText)
            } else {
                return false
            }
        }
        if filledTextFieldCells.count == 0 {
            self.navigationItem.rightBarButtonItem?.isEnabled = false
        } else {
            self.navigationItem.rightBarButtonItem?.isEnabled = true
        }
        
        return true
    }

extension Checkable {
    func isFilledTextField(text: String) -> Bool {
        return !text.isEmpty ? true : false
    }
}

///notificationを使ったやり方に変更///

func didChangeNotification(notification: Notification) {
        var noEmptyTextField = true
        
        if let textField = notification.object as? UITextField {
            switch textField.tag {
            case CellTag.name.rawValue:
                if let text = textField.text, !text.isEmpty {
                    noEmptyTextField = true
                } else {
                    noEmptyTextField = false
                }
            case CellTag.category.rawValue:
                if let text = textField.text, !text.isEmpty {
                    noEmptyTextField = true
                } else {
                    noEmptyTextField = false
                }
            case CellTag.price.rawValue:
                if let text = textField.text, !text.isEmpty {
                    noEmptyTextField = true
                } else {
                    noEmptyTextField = false
                }
            default:
                break
            }
        }
        
        if noEmptyTextField {
            self.navigationItem.rightBarButtonItem?.isEnabled = true
        } else {
            self.navigationItem.rightBarButtonItem?.isEnabled = false
        }
    }

Comments

[fuzzball]

(deleted)

Answer 1

以下のように記述することで判定できると思います、参考にしてみてください。

import UIKit

class ViewController: UIViewController {

    @IBOutlet weak var barButtonItem: UIBarButtonItem!
    @IBOutlet weak var textField: UITextField!
    @IBOutlet weak var textField2: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        barButtonItem.isEnabled = false
        
        // 入力値が変更された時に呼ばれる通知を登録
        NotificationCenter.default.addObserver(
            self,
            selector: #selector(self.didChangeNotification(notification:)),
            name: NSNotification.Name.UITextFieldTextDidChange,
            object: nil
        )
    }
    
    var isInputTextField1 = false
    var isInputTextField2 = false
    var isInputTextField3 = false
    
    // 入力値が変更された時に呼ばれるメソッド
    func didChangeNotification(notification: Notification) {
        
        if let textField = notification.object as? UITextField {
            if let text = textField.text {
                switch textField.tag {
                case CellTag.name.rawValue    : isInputTextField1 = text.isEmpty ? false : true
                case CellTag.category.rawValue: isInputTextField2 = text.isEmpty ? false : true
                case CellTag.price.rawValue   : isInputTextField3 = text.isEmpty ? false : true
                default: break
                }
            }
        }
        
        barButtonItem.isEnabled = isInputTextField1 || isInputTextField2 || isInputTextField3 ? true : false
    }

    // 通知の解除
    deinit {
        NotificationCenter.default.removeObserver(self)
    }
    
    override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
        view.endEditing(true)
    }
}

Comments

[tarofess]

ご回答ありがとうございます。なるほど、NotificationCenterでもtextfieldの変化をハンドルできるんですね。しかしtextfieldが画面に複数あって、今の所うまくいってないです...。

[_Kentarou]

回答を変更しました、通知のオブジェクトにUITextFieldのインスタンスが付いてくるのでどのテキストフィールドかは判断できます。

[tarofess]

どのtextfieldのインスタンスなのか判別することはできるようになりました。しかし一つのtextfieldが空になるとbarButtonItemのenableがfalseになってしまいます。。。すみません、お手数をおかけしますが質問に現在詰まっている箇所のコードを追記しましたので、ご教授いただきたいです。

[_Kentarou]

仕様が分からないのですが、複数あるTextFieldのうち一つでも入力があればボタンを押せるようにしたいと言うことですか？

[tarofess]

はい、 複数textfieldがあり、一つでも入力があればボタンを押せるようにしたいです。今、一つでも入力があればボタンを押せるようになっているのですが、例えば二つのtextfieldに文字が入力されていて、その状態で一つのtextfieldの文字を全て消すとボタンが押せなくなってしまっています。

[_Kentarou]

そのViewControllerクラスに変数として宣言してあるUITextFieldですか？

[tarofess]

ViewControllerにUITextFieldの変数は宣言しておらず、UITableViewCellのサブクラス内でtextfieldは宣言しています。

[_Kentarou]

強引ではありますがこのパターンに当てはめた実装で回答を修正しました、試してみてください。

[tarofess]

ご提示のコードで動かすと期待していた動作になりました！最後まで親切に回答してくださり、誠にありがとうございます。助かりましたmm

Answer 2

1アクション遅れるのは、textField(_:shouldChangeCharactersIn:replacementString:)の仕様を理解していないからです。この関数が呼ばれた時点ではテキストフィールドは更新されていません。

変更箇所(range)と変更内容(string)が渡されますので、元のテキストに対して、

let newText = (textField.text as NSString).replacingCharacters(in: range, with: string)
//動作未確認(nilチェックなど全くしていません)

とすると、変更後の文字列を生成できます。

この辺りを考慮して作り直してみてはどうでしょうか？

Comments

[tarofess]

ご回答ありがとうございます。shouldChangeCharactersInが呼ばれた時点でtextfieldは更新されないのですね。なのでご提示のコードを参考に修正してみたのですが、textfieldのどれか一つが空になるとbarbuttonitemがfalseになってしまいます。コードを追記したのですが、どこがおかしいでしょうか？

[fuzzball]

回答しておいてアレですが、textField(_:shouldChangeCharactersIn:replacementString:)を使うのはあまり良くないようなので、_Kentarouさんの回答を参考にした方が良いと思います。

参考URL
http://blog.fenrir-inc.com/jp/2011/01/textfield-validation.html
http://blog.niw.at/post/92806309874

[tarofess]

リンクの記事を見ると確かにshouldChangeCharactersInは使わない方が良さそうですね......。
notificationを使うことにします。ありがとうございます。