感谢我们 iOS 团队的赵恒、刘家飞发现此 bug。
码农的桌面.jpg
一. 问题描述
首先,创建一个 textField,使用 addTarget 函数来监听 textField 的 editingChanged 事件。
lazy var textField: UITextField = { let tf = UITextField() tf.keyboardType = .numberPad tf.textAlignment = .center tf.addTarget(self, action: #selector(valueChange(textField:)), for: .editingChanged) return tf }() @objc func valueChange(textField:UITextField) { print("------value change--- \(textFiled.text) ----") if (textFiled.text?.count == 6) { //do something 和服务器通讯验证用户输入的验证码是否正确 } }
如果系统升级到最新的 iOS 12.0,并结合系统中文键盘使用 AutoFill (自动识别短信验证码)的功能,就会发现 valueChange 函数中 "do something" 的这段代码被调用了三次。
在 iOS 12 设备上使用系统的中文(简体)
键盘或者中文(繁体)
键盘会打印出以下 log
------value change--- Optional("123456") ---- ------value change--- Optional("123456") ---- ------value change--- Optional("") ---- ------value change--- Optional("1") ---- ------value change--- Optional("12") ---- ------value change--- Optional("123") ---- ------value change--- Optional("1234") ---- ------value change--- Optional("12345") ---- ------value change--- Optional("123456") ----
再测试一下系统其他语言的键盘,例如 英文(美国)
键盘或者日文
键盘,打印出来的 log
------value change--- Optional("") ---- ------value change--- Optional("1") ---- ------value change--- Optional("12") ---- ------value change--- Optional("123") ---- ------value change--- Optional("1234") ---- ------value change--- Optional("12345") ---- ------value change--- Optional("123456") ----
由此,得出以下结论:
在系统中文键盘上 editingChanged 事件会莫名地多执行
两次
。
使用 RxSwift 会得到同样的结论,下面是使用 RxSwift 的代码片段:
textField.rx.text.orEmpty .subscribe(onNext: { text in let count = text.count if count == 6 { //do something 和服务器通讯验证用户输入的验证码是否正确 } }) .disposed(by: bag)
二. bug report
提交issue.PNG
提交完成.PNG
三. 解决方案
使用 UITextFieldDelegate 来代替原先 editingChanged / RxSwift 的监听事件,具体解决的代码片段如下:
func textField(_ textField: UITextField, shouldChangeCharactersIn range: NSRange, replacementString string: String) -> Bool { if range.location == 5 { var text = "" if let tfText = textField.text { text = tfText + string } if text.count == 6 { //do something 和服务器通讯验证用户输入的验证码是否正确 } } return true }