これはJava SynchronizedCollectionクラスのバグですか？

Question

内部クラスSynchronizedCollectionがあります。コンストラクタが2つあるjava.util.Collections の内部にあります。最初はコレクションを取り、もう1つはコレクションとミューテックスをとります。 元のコンストラクタはnullでないという引数をチェックします。しかし、後でしないでください！ここには が実装されています。この実装で

SynchronizedCollection(Collection<E> c) { 
     if (c==null) 
      throw new NullPointerException(); 
     this.c = c; 
     mutex = this; 
    } 
SynchronizedCollection(Collection<E> c, Object mutex) { 
     this.c = c; 
     this.mutex = mutex; 
} 

私は2番目の コンストラクタにnullを送信することにより、不変クラスを破ることができます。私は自分自身を納得させることはできませんしかし、ジョシュ・ブロッホとニールGafter氏がこれを見ることができなかった

SynchronizedCollection(Collection<E> c) { 
     this(c,this) 
    } 
SynchronizedCollection(Collection<E> c, Object mutex) { 
     if (c==null) 
      throw new NullPointerException(); 
     this.c = c; 
     this.mutex = mutex; 
} 

：

は、私はこのようなものであるべきと考えています。あなたは本当に私がここで逃したものを教えてくれますか？編集

---

：攻撃の可能性それらのコンストラクタの

Map<String, String> m = new Map<String, String>(){ 

     @Override 
     public int size() { 
      // TODO Auto-generated method stub 
      return 0; 
     } 

        . 
        . 
        . 

     @Override 
     public Collection<String> values() { 
      return null; 
     } 


    }; 

    Map<String, String> synchronizedMap = Collections.synchronizedMap(m); 
    Collection<String> values = synchronizedMap.values(); 

Answer 1

これはバグです。両方のコンストラクタは一貫している必要があります。両方とも例外をスローするか、スローする必要がありません。

これは今、両方のコンストラクタが例外をスローするJava 8で修正されています：

SynchronizedCollection(Collection<E> c) { 
    this.c = Objects.requireNonNull(c); 
    mutex = this; 
} 

SynchronizedCollection(Collection<E> c, Object mutex) { 
    this.c = Objects.requireNonNull(c); 
    this.mutex = Objects.requireNonNull(mutex); 
} 

Answer 2

両方がパッケージ保護及びおそらくはCollectionsのpublicsynchronizedList()とsynchronizedSet()方法によりnull引数と共に使用することができる唯一の最初のものです。

他のコンストラクタは内部で（Collectionsクラス内で）使用され、最初の引数はさまざまな実装（呼び出しコード）では決してnullになることはありません。

java.utilパッケージで何かを作成しようとすることもできますが、おそらくSecurityExceptionが表示されます。