質問

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;
}
役に立ちましたか?

解決

最初の比較を次のように変更します。

if (a == b)
    return true;

これは、「両方のヌル」をキャッチするだけではありません。ケースだけでなく、「配列を自分自身と比較する」こともできます。ケース。

ただし、より簡単な代替手段として- Arrays.equals 。各プリミティブタイプのオーバーロードがあります。 (実装は、ループ長から配列の長さを引き上げることを除いて、あなたのものに非常に似ています。.NETでは、最適化を無効にすることができますが、JREライブラリの実装者はおそらくJVMの方がよく知っていると思います:)

他のヒント

配列クラス。これらはより巧妙に実装される可能性があるためです。したがって、この場合、使用

Arrays.equals(a, b);

これが誰かに役立つかどうかはわかりませんが、これは機能しているようです:

        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 を実行しますが、あなたが期待するものではありません。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top