性能优化 - 枚举的坑values()

Posted 2022-12-15 江西昊仔

背景
最近处理一个飞机下载视频速度慢的Bug时，根据同事的反馈，发现了一个之前一直被忽视的问题，枚举的values()这个方法巨坑！像公司项目中的代码，因为要解析飞机的推送为枚举，代码里有各种枚举，通常会这样写：
enum Number
/**
* 测试例子
*/
ONE(0),
TWO(1),
THREE(2),
FOUR(3),
FIVE(4),
SIX(5),
SEVEN(6);

    private int value;

    Number(int value) 
        this.value = value;
    

    public int value() 
        return this.value;
    

    public boolean equals(int b) 
        return value == b;
    
    // 实现了一个find方法，用来根据value来查找枚举
    public static Number find(int value) 
        Number result = ONE;
        for (int i = 0; i < values().length; i++) 
            if (values()[i].equals(value)) 
                result = values()[i];
                break;
            
        
        return result;
    

复制代码初看时候似乎没什么，但连上飞机后，各类数据大量高频推送时(比如说100HZ)，这个values()方法会被多次调用，实测时，会出现明显的内存抖动现象。
分析
枚举分析
从网上的研究 深入理解Java枚举类型(enum) 以及 Java枚举的values方法是拷贝操作，你知道吗？，加上自己反编译的结果来看：

枚举的values()方法是确实是一个拷贝方法，每次调用都会生成一个新数组，虽然数组成员不会多占内存，但一个新数组本身也是占用一定空间的（Java对象在内存的结构
）。
解决方案
既然如此，只能尽量减少values()的调用:
private static Number[] values;

    // 使用缓存
    public static Number newFind(int value) 
        Number result = ONE;
        if (values == null)
            values = values();
        
        for (int i = 0; i < values.length; i++) 
            if (values[i].equals(value)) 
                result = values[i];
                break;
            
        
        return result;
    

复制代码实测

实现两个函数，分别验证各自连点10下的内存使用情况：

10次操作过后，很明显的可以看到有内存抖动的现象。

10次操作后，基本无明显的变化，包括CPU也占用的更少。
总结
平常使用的values()没想到还有这个问题，这样的目的应该是为了不让外部使用者修改它，但显然也限制了一些能力，作为调用者，这种经常使用的功能，还是要深入源码理解才对。

作者：Hoker
链接：https://juejin.cn/post/6854573212588736519
来源：稀土掘金
著作权归作者所有。商业转载请联系作者获得授权，非商业转载请注明出处。