Setに不変要素のみが含まれるべきですか？

Question

Aの要素のSetがあります。

私のコードでは、セットに含まれる要素を変更すると臭いがあります（この変更により、aのIDが変更される可能性があることを考慮してください）。

Answer 1

。 the JavaDoc of the Set interfaceから

：

注：可変オブジェクトがセット要素として使用されている場合細心の注意を払わなければならない。オブジェクトがセット内の要素であるときに等価比較に影響するようにオブジェクトの値が変更された場合、セットの動作は指定されていません。この禁止の特別な場合は、セットがそれ自身を要素として含むことは許されないということです。

強調鉱山。

equals()に影響を与えずに要素を変更できる場合は、イコールの実装が正しいかどうかを強く再考します。

Answer 2

Setの中の変更可能な要素を変更することで、不変なものは間違いなくの優先です。

Answer 3

たとえば、A aをセットに追加した後、aをハッシュコードを変更する方法で変更すると、set.contains(a)がfalseを返す可能性があります。

簡単な例：セットのみimmutablesが含まれている必要があり、時間のほとんど

public static void main(String[] args) { 
    Set<A> set = new HashSet<>(); 
    A a = new A(1); 
    set.add(a); 
    System.out.println(set.contains(a)); //true 
    a.i = 2; 
    System.out.println(set.contains(a)); //false 
} 

static class A { 
    int i; 
    public A(int i) { this.i = i; } 
    @Override public int hashCode() { return i; } 
} 

Answer 4

フィールドが変更されてhashCodeの計算にも使用されると、Setが壊れてしまうため、ほとんどの場合臭いがあります。

Setの要素を変更すると、Setが壊れる可能性があります。

対照的に、Listの要素を変更しても、そのリストは改行されません。

簡単な例（assyliasの答えから取られ、延長）

public class Main { 

    public static void main(String[] args) { 
    Set<A> set = new HashSet<>(); 
    List<A> list = new ArrayList<>(); 
    A a = new A(1); 
    set.add(a); 
    list.add(a); 
    System.out.println(set.contains(a)); // true 
    System.out.println(list.contains(a)); // true 
    a.i = 2; 
    System.out.println(set.contains(a)); // false 
    System.out.println(list.contains(a)); // true 
    } 

    static class A { 
    int i; 

    public A(int i) { 
     this.i = i; 
    } 

    @Override 
    public int hashCode() { 
     return i; 
    } 

    @Override 
    public boolean equals(Object obj) { 
     if (obj == null) { 
     return false; 
     } 
     if (!(obj instanceof A)) { 
     return false; 
     } 
     A other = (A) obj; 
     if (i != other.i) { 
     return false; 
     } 
     return true; 
    } 
    } 
}