iOS-NSUndoManager与怎样弄崩微信

检查项目bug的时候偶然发现,做过限制(比如说字数、表情)的textField、textView,触发限制条件后,会在使用undo功能时crash,之后发现微信也是一样的。
有朋友问在哪里崩了,不能复现,我举几个例子,其实有字数限制的输入框应该都有问题

我->个人信息->我的地址->新增地址
我->个人信息->名字
我->个人信息->个性签名

随便试了试qq、yy、简书、喜马拉雅的能输入汉字的输入框的字数限制,发现qq一般只提示不限制;yy禁用了undo;简书没做限制;做的最烂的是喜马拉雅,做了限制,但是可以轻松突破,输入任意长度的字符串。

DEMO

https://github.com/liulishuo/testUndo

思路

出现crash是因为,为了实现输入的过滤效果,会监听输入框的UIControlEventEditingChanged事件,截取字符串,手动给输入框的text属性赋值。正常情况下输入框执行setText:,默认不会注册到自己的undoManager上,并且会清空undoManger的undo、redo栈,这样并没有问题,问题是在于监听UIControlEventEditingChanged事件所执行的方法里是先对输入框的text做截取然后执行setText:。
看起来是截取的操作会入undo栈,之后的setText:方法并不会清空undo栈,导致做undo操作时,逆操作的是字符串截取的操作,操作的数据对不上,导致崩溃,这是我觉得比较合理的解释。

*** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[NSBigMutableString substringWithRange:]: Range {6, 4} out of bounds; string length 6'

以此为前提,我们有三个解决问题的方向:

  • 禁用undo功能,绕过去,yy是这样做的。

  • 使用setText:,每次在过滤操作时先将setText:注册到undoManager上,再进行setText:赋值操作。我试过不行。
    还是一样的错误-[NSBigMutableString substringWithRange:] range超限了,setText:的逆操作为啥也是这个,我不清楚。

  • 使用setText:,并确保和系统默认行为一致,也就是用setText:赋值,并清空undo栈。
    个人觉得这样能达到目的,最方便。

实现

先说微信,微信的输入框特点是:
1.汉字联想的时的字符数也一样有限制
2.文本长度满了,输入框就不能从任意位置插入任何字符(可能是为了规避系统九宫格键盘输入汉字的问题)
(不太好归纳,我的地址的收货人输入框貌似有两套逻辑,一个是最大长度16个字,另外一个是最大长度50个字,我每次crash回来都会切换。。。,但是两套逻辑都有各自的问题,有兴趣的同学自己试一下,我们这里只讨论会crash的情况,也就是最大长度16个字的限制条件下的问题)
所以我一开始以为,微信应该是这么实现的

- (BOOL)textField:(UITextField *)textField shouldChangeCharactersInRange:(NSRange)range replacementString:(NSString *)string
{
     //退格
    if([string isEqualToString:@""])
    {
        return YES;
    }
    
    //文本长度满不允许编辑 防止系统九宫格键盘在此时传入数字标号字符
    if(textField.text.length >= kMaxLength)
    {
        return NO;
    }
    
    //非联想状态
    if(!textField.markedTextRange)
    {
        NSString * tempString = [textField.text stringByReplacingCharactersInRange:range withString:string];
        NSLog(@"%@",tempString);
        
        if (tempString.length > kMaxLength)
        {
            textField.text = [tempString substringToIndex:kMaxLength];
            return NO;
        }
    }
    
    return YES;
}

但是这样写不会因为undo而crash,并且还有汉字联想无限输入的bug。
所以微信应该还用了这种方式

//微信

[_tf addTarget:self action:@selector(textFieldTextDidChanged:) forControlEvents:UIControlEventEditingChanged];

- (void)textFieldTextDidChanged:(UITextField *)sender
{
    NSString * tempString = sender.text;

    if (sender.markedTextRange == nil && tempString.length > kMaxLength)
    {
        sender.text = [tempString substringToIndex:kMaxLength];
    }
}

这种方式除了undo会crash,没有其他明显的漏洞。

修复这个bug,只需要加一行代码

- (void)textFieldTextDidChanged:(UITextField *)sender
{
    NSString * tempString = sender.text;
    
    if (sender.markedTextRange == nil && tempString.length > kMaxLength)
    {
        sender.text = [tempString substringToIndex:kMaxLength];
        [sender.undoManager removeAllActions];
    }
}

just for fun 我们来猜一下其他人的实现
yy的实现(机智)

//yy
- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {
    
     application.applicationSupportsShakeToEdit = NO;
    
    return YES;
}

喜马拉雅的实现(漏洞最多)

//喜马拉雅
- (BOOL)textField:(UITextField *)textField shouldChangeCharactersInRange:(NSRange)range replacementString:(NSString *)string
{
    if(range.location >= kMaxLength)
    {
        return NO;
    }
    else
    {
        return YES;
    }
}

其实我觉得在用户的输入阶段就屏蔽掉某些可能的输入,真是一件吃力不讨好的事情。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 157,012评论 4 359
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 66,589评论 1 290
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 106,819评论 0 237
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 43,652评论 0 202
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 51,954评论 3 285
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,381评论 1 210
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,687评论 2 310
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,404评论 0 194
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,082评论 1 238
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,355评论 2 241
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 31,880评论 1 255
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,249评论 2 250
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 32,864评论 3 232
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,007评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,760评论 0 192
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,394评论 2 269
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,281评论 2 259

推荐阅读更多精彩内容