2016-02-22 9 views
13

ArrayListは、その親クラスAbstractListからequalsの実装を継承していますが、あまり効果的ではありません。パフォーマンスを向上させるために、ArrayListがequals()をオーバーライドしないのはなぜですか?

まず、2つのArrayListのサイズを確認してから、これらのサイズが異なる場合はすぐにfalseを返すことができます。なぜArrayListはそれをしないのですか?

+0

は、私は、[このことを考えていない。このスレッドhttp://programmers.stackexchange.com/questions/239139/why-do-arrays-in-java-not-override-equals –

+1

を読んで試してみてください](http://stackoverflow.com/questions/18958113/jdk-implementation-of-abstractlistequals-does-not-check-for-list-size-equali)質問は重複しており、改めて投票しました。 – Maroun

答えて

7
this answer

に述べたようにいくつかの実装は、それらsize方法のO(n)の複雑性を有するので、これは実際分解ことができるので、これを行いません。

私はすべてのリストにequalsが一貫した作る実装はO(1)サイズの複雑さを持つコレクションに影響を与えることができることに同意し、多分Java開発者は、「それはあなたがドンときにそれを削除するよりも、あなたが必要なときにそれを挿入しはるかに簡単だと思いましたt(メソッド全体を再実装する必要があります)。たとえば、あなたは簡単のようなもので、この最適化を追加することができます。

public boolean equals(Object o) { 
    // here it is 
    if (o instanceof List && this.size() != ((List)o).size()) 
     return false; 

    // call the parent equals 
    return super.equals(o); 

しかし、それは、もともと(抽象クラ​​スで)サイズチェックで実施された場合は、全体のメソッドを再実装しなければならなかったとサイズを削除しますチェック:

public boolean equals(Object o) { 
    if (o == this) 
     return true; 
    if (!(o instanceof List)) 
     return false; 

    ListIterator<E> e1 = listIterator(); 
    ListIterator<?> e2 = ((List<?>) o).listIterator(); 
    while (e1.hasNext() && e2.hasNext()) { 
     E o1 = e1.next(); 
     Object o2 = e2.next(); 
     if (!(o1==null ? o2==null : o1.equals(o2))) 
      return false; 
    } 
    return !(e1.hasNext() || e2.hasNext()); 
} 
+1

ああ、私が思っていたときに忘れたのは、 'size()'のデフォルトの実装は 'O(1)'ですが、 'final'ではないので、オーバーライドされていないことを保証することはできません。 Obvs。 –

+0

私はまだ目を覚ましていませんが、手動で反復するのではなく、単にreturn super.equals(o);を呼び出すことができませんでしたか? – Marco13

関連する問題