Question

Je connais Arrays.deepEquals (Object [], Object []), mais cela ne fonctionne pas pour les types primitifs (en raison des limitations relatives aux tableaux et à l'autoboxing, voir cet article associé ).

Dans cet esprit, est-ce l'approche la plus efficace?

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;
}
Était-ce utile?

La solution

Modifiez votre première comparaison comme suit:

if (a == b)
    return true;

Ceci n’attrape pas seulement le " les deux null & cas, mais aussi "comparer un tableau à lui-même" cas.

Toutefois, pour une solution de remplacement plus simple, utilisez Arrays.equals qui a des surcharges pour chaque type de primitive. (L’implémentation est très similaire à la vôtre, à la différence qu’elle élève la longueur du tableau de la boucle. Sur .NET, cela peut être une anti-optimisation, mais je suppose que les développeurs de la bibliothèque JRE en savent probablement mieux pour la JVM:)

Autres conseils

Je pense que le plus efficace devrait être d'utiliser les méthodes d'assistance dans Arrays , car ils pourraient être implémentés plus intelligemment. Donc, dans ce cas, utilisez

Arrays.equals(a, b);

Je ne sais pas si cela aidera quelqu'un, mais cela semble fonctionner:

        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;
            }
        }

Apparemment, array.equals (otherArray) effectue un array == otherArray et ne correspond pas à ce que vous attendez.

Licencié sous: CC-BY-SA avec attribution
Non affilié à StackOverflow
scroll top