非平凡（看似）StringIndexOutOfBoundsException答案

【问题标题】：Non trivial (seemingly) StringIndexOutOfBoundsException非平凡（看似）StringIndexOutOfBoundsException
【发布时间】：2019-04-22 04:59:31
【问题描述】：

我们的崩溃日志系统显示崩溃，我不明白它是如何发生的。用户输入是未知的。我已经添加了日志记录，但只有在下一个版本（大约 2 周）之后才能看到结果。

下面的代码怎么会抛出下一个异常：

Crashes with java.lang.StringIndexOutOfBoundsException length=0; index=1

不是因为并发问题？换句话说，它崩溃的情况可能是什么以及如何避免它（不将其包装到try-catch）？如果这是一个并发问题（我没有看到，但也许我错了）会是什么情况？

private String contactName; // has getter and setter

public String getFirstSymbol() {
  String firstSymbol = "";
  String trimmed = contactName.trim();
  if (!TextUtils.isEmpty(trimmed)) { // TextUtils.isEmpty is Android method: return str == null || str.length() == 0;
    String[] parts = trimmed.toUpperCase().split("\\s+");
    firstSymbol = parts[0].substring(0, 1); // crashes here:
  }
  return firstSymbol;
 }

上周在 7 台设备上发生了约 100 次崩溃。

这是堆栈跟踪

  Fatal Exception: java.lang.StringIndexOutOfBoundsException: length=0; index=1
  at java.lang.String.substring(String.java:1971)
  at my.app.model.Contact.getFirstSymbol(Contact.java:256)
  at my.app.ui.AvatarView.bind(AvatarView.java:73)
  at my.app.viewholder.CallLogViewHolder.bind(CallLogViewHolder.java:50)
  at my.app.viewholder.CallLogViewHolder.bind(CallLogViewHolder.java:23)
  at my.app.ContactListAdapter.onBindViewHolder(ContactListAdapter.java:68)
  at my.app.ContactListAdapter.onBindViewHolder(ContactListAdapter.java:25)
  at androidx.recyclerview.widget.RecyclerView$Adapter.onCreateViewHolder(RecyclerView.java:6781)
  at androidx.recyclerview.widget.RecyclerView$Adapter.onBindViewHolder(RecyclerView.java:6781)
  at androidx.recyclerview.widget.RecyclerView$Adapter.getItemCount(RecyclerView.java:6781)
  at androidx.recyclerview.widget.RecyclerView$Adapter.bindViewHolder(RecyclerView.java:6823)
  at androidx.recyclerview.widget.RecyclerView$Recycler.tryBindViewHolderByDeadline(RecyclerView.java:5752)
  at androidx.recyclerview.widget.RecyclerView$Recycler.tryGetViewHolderForPositionByDeadline(RecyclerView.java:6019)
  at androidx.recyclerview.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:5858)
  at androidx.recyclerview.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:5854)
  at androidx.recyclerview.widget.LinearLayoutManager$LayoutState.next(LinearLayoutManager.java:2230)
  at androidx.recyclerview.widget.LinearLayoutManager.layoutChunk(LinearLayoutManager.java:1557)
  at androidx.recyclerview.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1517)
  at androidx.recyclerview.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:612)
  at androidx.recyclerview.widget.RecyclerView.dispatchLayoutStep2(RecyclerView.java:3924)
  at androidx.recyclerview.widget.RecyclerView.dispatchLayout(RecyclerView.java:3641)
  at androidx.recyclerview.widget.RecyclerView.onLayout(RecyclerView.java:4194)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:323)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:261)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at androidx.viewpager.widget.ViewPager.onLayout(ViewPager.java:1775)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at com.google.android.material.appbar.HeaderScrollingViewBehavior.layoutChild(HeaderScrollingViewBehavior.java:142)
  at com.google.android.material.appbar.HeaderScrollingViewBehavior.findFirstDependency(HeaderScrollingViewBehavior.java:142)
  at com.google.android.material.appbar.ViewOffsetBehavior.onLayoutChild(ViewOffsetBehavior.java:41)
  at com.google.android.material.appbar.AppBarLayout$ScrollingViewBehavior.onMeasureChild(AppBarLayout.java:1556)
  at androidx.coordinatorlayout.widget.CoordinatorLayout.onLayout(CoordinatorLayout.java:888)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:323)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:261)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at com.google.android.material.appbar.HeaderScrollingViewBehavior.layoutChild(HeaderScrollingViewBehavior.java:142)
  at com.google.android.material.appbar.HeaderScrollingViewBehavior.findFirstDependency(HeaderScrollingViewBehavior.java:142)
  at com.google.android.material.appbar.ViewOffsetBehavior.onLayoutChild(ViewOffsetBehavior.java:41)
  at com.google.android.material.appbar.AppBarLayout$ScrollingViewBehavior.onMeasureChild(AppBarLayout.java:1556)
  at androidx.coordinatorlayout.widget.CoordinatorLayout.onLayout(CoordinatorLayout.java:888)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:323)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:261)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1791)
  at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1635)
  at android.widget.LinearLayout.onLayout(LinearLayout.java:1544)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:323)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:261)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1791)
  at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1635)
  at android.widget.LinearLayout.onLayout(LinearLayout.java:1544)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:323)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:261)
  at com.android.internal.policy.DecorView.onLayout(DecorView.java:955)
  at android.view.View.layout(View.java:20964)
  at android.view.ViewGroup.layout(ViewGroup.java:6440)
  at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:3092)
  at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2779)
  at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1863)
  at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:8072)
  at android.view.Choreographer$CallbackRecord.run(Choreographer.java:911)
  at android.view.Choreographer.doCallbacks(Choreographer.java:723)
  at android.view.Choreographer.doFrame(Choreographer.java:658)
  at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:897)
  at android.os.Handler.handleCallback(Handler.java:790)
  at android.os.Handler.dispatchMessage(Handler.java:99)
  at android.os.Looper.loop(Looper.java:164)
  at android.app.ActivityThread.main(ActivityThread.java:7000)
  at java.lang.reflect.Method.invoke(Method.java)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:441)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1408)

【问题讨论】：

value 是什么fieldString 你能提供一些示例输入吗？
我不确定您的问题是否可重现。因此，您应该向我们提供正在发生这种情况的输入。
您可以轻松找出导致问题的输入是什么。将您的代码修改为 1) 捕获异常，2) 打印出 trimmed 的值，3) 重新抛出异常。或者...设置断点。
@ArunKumarMN，你确定那是上面的确切代码吗？我有这样一个测试，它通过了。
@AkashShah，您缺少 if (!TextUtils.isEmpty(trimmed)) 条件。这基本上是 return str == null || str.length() == 0，加了就通过了。这是示例：onlinegdb.com/HkNlLCc5E

标签： java crash

【解决方案1】：

这里没有线程安全/并发问题。此方法可能与其他线程共享的唯一值是contactName 的值。那是一个String 参考。 String 是不可变的。此方法从（可能）共享的不可变 String 派生的其他值都是线程限制的。

除此之外，我很困惑。 trim 应该删除 \s+ 将匹配的任何前导或尾随字符。而!isEmpty 守卫消除了空字符串的可能性。所以parts 数组应该至少有一个元素，第一个元素应该至少有一个字符。

但例外情况似乎有所不同。

因此，这留下了人们通常认为荒谬的“替代”解释：

这可能发生在 JIT 编译器损坏的平台上。
这可能发生在String 或Pattern 中存在错误的平台上。
可能是硬件问题。
也许您的某个版本的应用程序的编译代码与您正在查看的源代码不匹配。

这些“替代”解释都没有给您任何前进的方向。所以，如果你找不到更好的解释，我建议你用try { ... } catch 包围这个有问题的代码，并尝试记录触发问题的contactName 字符串的长度和内容。将其包含在下一个版本中，如果/当您开始出现故障时，请准备好发布带有真正修复的次要版本。或者，只需将其视为用户输入了一个空的联系人姓名......或类似的东西。

【讨论】：

是的，我对代码正确性有同样的理由，只是想确定一下。谢谢，如果 smbd 有类似的问题，将等待一两天，但这看起来像答案。