便签SpannableString崩溃问题分析
1. 问题描述
自研的便签在加入多样式文本后出现偶现崩溃问题,代码提示如下,可以看到是在SpannableString对象初始化出现的越界错误,出错的代码很快就定位到,但是发现并不好分析,原因在于
(1)出错的代码在源码中,代码中只是对象初始化,按道理来说不会出现问题(后面证实该想法存在问题,初始化过程中会触发其他的代码)
(2)由于该问题是偶现问题,通过moneky测试发现的,没有办法找到必现路径
(3)在网上没有搜索出同样的问题,其实从这里就可以推断是自己代码中的问题
java.lang.IndexOutOfBoundsException: setSpan (0 ... -1) has end before start
at android.text.SpannableStringInternal.checkRange(SpannableStringInternal.java:428)
at android.text.SpannableStringInternal.setSpan(SpannableStringInternal.java:163)
at android.text.SpannableStringInternal.copySpans(SpannableStringInternal.java:68)
at android.text.SpannableStringInternal.(SpannableStringInternal.java:43)
at android.text.SpannableString.(SpannableString.java:30)
2.问题分析
(1)尝试复现问题
该功能是在加入富文本样式中后出现的,所以推断是跟样式存在联系,而且代码出错的触发时机是在打开页面或者页面重复滚动。测试时通过多次输入样式文本打开和滚动页面,最后发现当输入样式文本第一个字体为带样式文本时,上下滚动页面后会出现崩溃。
(2)源码分析
找到必现路径后,跟踪崩溃的源码,发现在SpannableStringInternal中会调用CopySpans方法,当调用到i=6时,发现st和en获取都为-1,设置setSpan的参数时就会出现开头所说的越界错误。
private void copySpans(Spanned src, int start, int end, boolean ignoreNoCopySpan) {
Object[] spans = src.getSpans(start, end, Object.class);
for(int i = 0; i < spans.length; ++i) {
if (!ignoreNoCopySpan || !(spans[i] instanceof NoCopySpan)) {
int st = src.getSpanStart(spans[i]);
int en = src.getSpanEnd(spans[i]);
int fl = src.getSpanFlags(spans[i]);
if (st < start) {
st = start;
}
if (en > end) {
en = end;
}
this.setSpan(spans[i], st - start, en - start, fl, false);
}
}
}
查看getSpanStart方法代码(SpannableStringBuilder类中),getSpanStart值与mIndexOfSpan值有关,mIndexOfSpan是IdentityHashMap类型对象,IdentityHashMap是以对象作为key值保存,具体使用可参考官方文档,这里我们可以当hashMap进行理解。
查看以下代码,我们发现getSpanStart返回-1的情况只有两种:
- mIndexOfSpan为空,由于i=6时才会触发该条件,说明mIndexOfSpan不可能为空
- mIndextOfSpan在获取i为spans[6]时对应的object为null,所以这里推断在SpannableString初始化时,在代码的其他地方设置了该object的值为null
private IdentityHashMap
在SpannableStringBuilder类中查看mIndexOfSpan的引用,发现在removeSpan方法中会移处指定的标记对象。
public void removeSpan(Object what, int flags) {
if (mIndexOfSpan == null) return;
Integer i = mIndexOfSpan.remove(what);
if (i != null) {
removeSpan(i.intValue(), flags);
}
}
在removeSpan方法中设置断点,调试跟踪代码,调用栈如下。在SpannableString初始化时会调用onSelectionChange方法中,在onSelectionChanged方法中会判断selStart和selEnd参数进行span的设置,当满足设定条件时,会调用removeSpan方法移除设置的富文本样式(该方法为自己调用代码,不属于源码)。
问题的原因就很简单,在SpannableString调用了removeSpan方法,移除相应的对象引用,导致在copySpans中调用时,遍历mIndexOfSpan时获取该值为空,导致越界错误。
但这里存在两个疑惑,
在SpannableString初始化中为什么调用onSelectionChange
在调用CopySpans方法时,首先会遍历源对象的span对象,怎么确保出错的span不在onSelectionChange方法前调用,也就是onSelectionChange调用的时机
这里我们先分析第一个疑惑:从以上调用栈,可以获取函数调用为
SpannableStringInterna.copy --> SpanSpannableStringInternal.setSpan --> SpannableStringInternal.sendSpanAdded --> ChangeWatcher.onSpanChanged --> TextView.spanChange
在设置span时,会触发onSpanAdd方法,最后会调用到TextView的onSelectionChange方法,这里的关键在于Span数组中包含了TextView的ChangeWatcher对象。
SpannableStringInternal类的SetSpan方法
private void setSpan(Object what, int start, int end, int flags, boolean enforceParagraph) {
this.checkRange("setSpan", start, end);
...
if (this instanceof Spannable) {
this.sendSpanAdded(what, start, end);
}
}
SpannableStringInternal类的sendSpanAdded方法
private void sendSpanAdded(Object what, int start, int end) {
SpanWatcher[] recip = getSpans(start, end, SpanWatcher.class);
int n = recip.length;
for (int i = 0; i < n; i++) {
recip[i].onSpanAdded((Spannable) this, what, start, end);
}
}
ChangeWatcher的onSpanChanged
public void onSpanChanged(Spannable buf, Object what, int s, int e, int st, int en) {
TextView.this.spanChange(buf, what, s, st, e, en);
}
TextView的spanChange方法
void spanChange(Spanned buf, Object what, int oldStart, int newStart, int oldEnd, int newEnd) {
if (selChanged) {
...
if ((buf.getSpanFlags(what) & Spanned.SPAN_INTERMEDIATE) == 0) {
...
onSelectionChanged(newSelStart, newSelEnd);
}
}
}
那么这个changeWather是在何时增加到Span数组中呢,查看TextView的源码,在setText中可以看到,在设置文本时会自动添加TextWatcher。
出错的span为我们自己设置的StyleSpan(富文本样式),是在setText之后设置的span,所以也就确保出错的span是在onSelectionChange方法之后,为了验证这个问题,断点查看了span的顺序,如下所示,如猜想所示,这里也就解决了第二个疑惑。
private void setText(CharSequence text, BufferType type,
boolean notifyBefore, int oldlen) {
if (text instanceof Spannable && !mAllowTransformationLengthChange) {
if (mChangeWatcher == null) mChangeWatcher = new ChangeWatcher();
sp.setSpan(mChangeWatcher, 0, textLength, Spanned.SPAN_INCLUSIVE_INCLUSIVE
| (CHANGE_WATCHER_PRIORITY << Spanned.SPAN_PRIORITY_SHIFT));
}
}
3.解决方案
最后整理下总过程,在进行new SpannableString时会时会调用CopySpan方法,会遍历对象的span,当遍历到Span为ChangeWatcher时,会触发onSelectionChange函数,会主动调用removeSpan方法移除我们设定的样式span,同时移除mIndexOfSpan中对应的对象,导致在copySpan获取下次StyleSpan对象为空,出现崩溃问题。
可以看到,问题的关键在于onSelectionChange方法的调用,在问题验证过程中,发现将SpannableString修改为SpanStringBuilder时不会出现崩溃问题。
查看源码发现SpannableStringBuilder在setSpan时会设置标志位send为false,在SpannableStringBuilder的setSpan方法中会根据该值触发sendSpanAdded方法,所以也就不会触发onSelectionChange方法,最终的解决方案也很简单,将原来调用的修改为SpannableStringBuilder即可。
public SpannableStringBuilder(CharSequence text, int start, int end) {
if (text instanceof Spanned) {
Spanned sp = (Spanned) text;
Object[] spans = sp.getSpans(start, end, Object.class);
for (int i = 0; i < spans.length; i++) {
...
setSpan(false, spans[i], st, en, fl, false/*enforceParagraph*/);
...
}
}
}
private void setSpan(boolean send, Object what, int start, int end, int flags,
boolean enforceParagraph) {
checkRange("setSpan", start, end);
...
if (send) {
restoreInvariants();
sendSpanAdded(what, nstart, nend);
}
...
}
参考文档:
Spannable 源码分析