一个集合应该只包含不可变元素吗？

Question

我有一个SetA类型的元素。

如果我修改集合中包含的元素（考虑到这种修改可能会改变任何a的身份），我的代码是否有异味？

Answer 1

大多数时候一组应该只包含immutables。

从the JavaDoc of the Set interface：

注意：如果使用可变对象作为一组元素大，一定要小心。没有指定集合的行为如果对象的值以影响等于比较的方式更改，而对象是集合中的元素，则该对象的值会发生更改。这种禁令的一个特例是，一个集合不允许自己作为一个元素。

强调我的。

如果您能够在不影响equals()的情况下修改元素，我会强烈重新考虑您的等值实现是否正确。

Answer 2

由于您可以通过修改里面的可变元素来打破Set，所以不可变的肯定是首选。

Answer 3

一个问题，举例来说，就是如果你添加一个A a一组，然后在改变其哈希码，有一个机会，set.contains(a)将返回错误的方式发生变异a。

简单的例子：

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。

相反，修改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; 
    } 
    } 
}