Сравните два массива примитивов в Java?

Question

Я знаю об Arrays.deepEquals (Object [], Object []), но это не работает для примитивных типов (из-за ограничений массивов и автобокса, см. этот связанный пост ).

Имея это в виду, это самый эффективный подход?

boolean byteArrayEquals(byte[] a, byte[] b) {
    if (a == null && b == null)
        return true;

    if (a == null || b == null)
        return false;

    if (a.length != b.length)
        return false;

    for (int i = 0; i < a.length; i++) {
        if (a[i] != b[i])
            return false;
    }
    return true;
}

Answer 1

Измените первое сравнение на:

if (a == b)
    return true;

Это не только ловит " оба null " случаи, но также "сравни массив с самим собой" случай.

Однако, для более простой альтернативы - используйте Arrays.equals , который имеет перегрузки для каждого примитивного типа. (Реализация очень похожа на вашу, за исключением того, что она выводит длину массива из цикла. В .NET это может быть антиоптимизацией, но я предполагаю, что разработчики библиотеки JRE, вероятно, знают лучше для JVM:)

Answer 2

Я думаю, что наиболее эффективным должно быть использование вспомогательных методов в Массивы class, потому что они могут быть реализованы более умно. Так что в этом случае используйте

Arrays.equals(a, b);

Answer 3

Я не знаю, поможет ли это кому-нибудь, но, похоже, это работает:

        if(type == type_BooleanArray) {
            boolean eq = Arrays.equals((boolean[]) thisObj, (boolean[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_ByteArray) {
            boolean eq = Arrays.equals((byte[]) thisObj, (byte[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_ShortArray) {
            boolean eq = Arrays.equals((short[]) thisObj, (short[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_CharArray) {
            boolean eq = Arrays.equals((char[]) thisObj, (char[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_IntArray) {
            boolean eq = Arrays.equals((int[]) thisObj, (int[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_LongArray) {
            boolean eq = Arrays.equals((long[]) thisObj, (long[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_FloatArray) {
            boolean eq = Arrays.equals((float[]) thisObj, (float[]) thatObj);
            if(!eq) {
                return false;
            }
        } else if(type == type_DoubleArray) {
            boolean eq = Arrays.equals((double[]) thisObj, (double[]) thatObj);
            if(!eq) {
                return false;
            }
        } else {
            if(!thisObj.equals(thatObj)) {
                return false;
            }
        }

Очевидно, array.equals (otherArray) выполняет array == otherArray , а не то, что вы ожидаете.