背景

最近处理一个飞机下载视频速度慢的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;
        }