假ArrayList导致的线上事故......

线上事故回顾

晚饭时,当我正沉迷于排骨煲肉质鲜嫩,汤汁浓郁时,产研沟通群内发出一条消息,显示用户存在可用劵,但进去劵列表却什么也没有,并附含了一个视频。于是我一边吃了排骨,一边查看消息点开了视频,en~,视频跟描述一样。但没有系统告警,用户界面也没有明显的报错提示,怀疑是小部分特殊情况导致的,查看消息后几秒,我直接被@来处理问题,擦,只好把外卖盒重新盖好,先去处理问题。


处理经过

通过群内产品发的用户邮箱查到了用户id,再根据接口的相关日志结合uid在日志平台进行关联查询,查到日志后,再拿到traceId进行链路查询,果不其然,发现了异常日志,如下部分日志所示

java.lang.UnsupportedOperationException: null
    at java.util.AbstractList.add(AbstractList.java:148) ~[na:1.8.0_151]
    at java.util.AbstractList.add(AbstractList.java:108) ~[na:1.8.0_151]
复制代码

乍一看,这不是空指针嘛,so easy啊

image.png

仔细一瞧,这
UnsupportedOperationException是个什么玩意

于是,根据日志找到代码中报错的那一行,下面给大家简单模拟下

@Slf4j
@SpringBootTest
public class Demo {

    public void test(Context context) { 
        context.getList().add("Code皮皮虾");
    }

}

@Data
class Context {

    private List<String> list;

}
复制代码

基本操作就是拿到上下文中的List,然后再add一个元素

image.png

讲道理,add操作是不会有问题的,有问题的还得是List,追根溯源,让我康康这个List是怎么来的

于是我一顿狂点,来到了set这个list的位置

@Slf4j
@SpringBootTest
public class Demo {

    public void test(Context context) {
        context.setList(Arrays.asList("Code皮皮虾"));
    }

}

@Data
class Context {

    private List<String> list;

}
复制代码

context.setList(Arrays.asList("Code皮皮虾")); 这行看起来好像没问题啊

Arrays.asList(T... a)我们平时也会用,传入一个数组,返回出一个List没啥问题呀

image.png

那我再试试add方法

image.png

擦,问题复现了,还真是Arrays.asList(T... a)生成的List的add方法报错

由于线上存在问题,则先修改为以下代码上线,也就是修改为我们平时正常的写法

image.png

上线后,观察了下日志,群里回复已解决问题,也让用户重试,发现没问题,自此问题解决。

接下来,咱们来看看为啥Arrays.asList(T... a)的add方法会报错


追根溯源

进入asList方法,发现底层new了一个ArrayList,并将数组传入作为List的元素

@SafeVarargs
@SuppressWarnings("varargs")
public static <T> List<T> asList(T... a) {
    return new ArrayList<>(a);
}
复制代码

emm,看起来很简单啊,没问题啊,咋会报错呢

别着急,咱们在点开这个ArrayList瞅瞅

private static class ArrayList<E> extends AbstractList<E>
    implements RandomAccess, java.io.Serializable
{
    private static final long serialVersionUID = -2764017481108945198L;
    private final E[] a;

    ArrayList(E[] array) {
        a = Objects.requireNonNull(array);
    }
    // ... 省略
}
复制代码

擦,这ArrayList是Arrays类的一个静态内部类,不是我们经常用的java.util.ArrayList

image.png

真是离谱他妈给离谱开门,离谱大家了,还是我源码看得太少了,呜呜呜~

继续看,这个静态内部类ArrayList继承了AbstractList,而且默认是没有实现add方法的

image.png

也就是说调用add方法会直接调用父类,也就是AbstractList的add方法,源码点开一看,真相大白了

AbstractList的add方法直接抛出
UnsupportedOperationException异常,跟线上报错一模一样!!!

public boolean add(E e) {
    add(size(), e);
    return true;
}

public void add(int index, E element) {
    throw new UnsupportedOperationException();
}
复制代码

至此,排查结束,继续吃饭去排骨去咯~~~

image.png

小彩蛋

在使用Arrays.asList(T... a)方法时,如果只是单个元素的话,Idea会提示我们更建议Collections.singletonList

image.png

别用!!!真的别用!!!

因为Collections.singletonList底层跟Arrays.asList(T... a)差不多

SingletonList也是继承了AbstractList的一个内部类,调用add一样会报
UnsupportedOperationException异常

public static <T> List<T> singletonList(T o) {
    return new SingletonList<>(o);
}

private static class SingletonList<E>
    extends AbstractList<E>
    implements RandomAccess, Serializable {

    private static final long serialVersionUID = 3093736618740652951L;

    private final E element;

    SingletonList(E obj) {
        element = obj;
    }
}
复制代码

结尾

当然咯,也不是禁止使用Collections.singletonList和Arrays.asList(T... a),只是我们在使用的时候一定要区分一下场景,如果创建的是一个不会再添加元素的List,那么则可以使用

但我们平时不想写那么麻烦,想要在创建的时候就把元素塞到List中,那咋办呢?

我们其实能使用google的工具类

<dependency>
    <groupId>com.google.guava</groupId>
    <artifactId>guava</artifactId>
    <version>23.0</version>
</dependency>
复制代码

如下写法

@Slf4j
@SpringBootTest
public class Demo {

    public static void main(String[] args) {
        List<String> strings = Lists.newArrayList("Code皮皮虾", "哈哈哈");
        strings.add("憨憨熊");
        System.out.println(strings);
    }
}
复制代码

其内部已经为我们封装好了,拿来即用即可,哈哈

@SafeVarargs
@CanIgnoreReturnValue // TODO(kak): Remove this
@GwtCompatible(serializable = true)
public static <E> ArrayList<E> newArrayList(E... elements) {
  checkNotNull(elements); // for GWT
  // Avoid integer overflow when a large array is passed in
  int capacity = computeArrayListCapacity(elements.length);
  ArrayList<E> list = new ArrayList<>(capacity);
  Collections.addAll(list, elements);
  return list;
}

作者:Code皮皮虾
链接:
https://juejin.cn/post/7159839509868183583

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

推荐阅读更多精彩内容