java - Collections.unmodifyingCollection 和 Collections.unmodifyingSet

Question

假设我有以下 Set

Set<String> fruits = new HashSet<String>();
fruits.add("Apple")
fruits.add("Grapes")
fruits.add("Orange")

如果我想创建一个防御性副本，以便如果原始列表被修改，副本不会反射(reflect)它，我可以这样做:

Set<String> unmodifiableFruits = Collections.unmodifiableSet(new HashSet(fruits))

所以如果我这样做:

fruits.add("Pineapple")     
println(unmodifiableFruits)

unmodifiableFruits不会有菠萝。

或者我可以这样:

Set<String> unmodifiableFruits = Collections.unmodifiableCollection(fruits)

结果是一样的，unmodifiableFruits不会有菠萝。

问题:

1. 假设我通过了 fruits作为类的参数，首选方法是 Collections.unmodifiableCollection() ？

原因是，我已经 read声明new如果我要使用 Collections.unmodifiableSet() 在构造函数中是一个不好的做法，我需要声明 new HashSet<String>(fruits) .

为什么我不能这样做？

Collections.unmodifiableSet(fruits)

并让它返回一个不可修改的集合。

相反，我必须这样做:

Collections.unmodifiableSet(new HashSet<String>(fruits))

是不是因为Set是一个接口(interface)，它不知道返回哪个实现？

Answer 1

Groovy 增强了集合方法，这意味着它向标准集合类添加了方法。

其中一个方法是 toSet() :

Convert a Collection to a Set. Always returns a new Set even if the Collection is already a Set.

Example usage:

def result = [1, 2, 2, 2, 3].toSet()
assert result instanceof Set
assert result == [1, 2, 3] as Set

当你写下这个:

Set<String> unmodifiableFruits = Collections.unmodifiableCollection(fruits)

它意味着调用 .toSet() 来将 unmodifyingCollection 返回的 Collection 强制转换为 Set， 隐式复制数据。

当你写下这个:

Set<String> unmodifiableFruits = Collections.unmodifiableSet(fruits)

返回的值已经是一个Set，因此不会调用toSet()，这意味着unmodifyingFruits和fruits 共享数据。

这就是为什么在使用 unmodifyingSet 时必须通过添加 new HashSet(...) 来显式复制数据。

Is using Collections.unmodifiableCollection() the proper way when passing a set into the constructor?

绝对不是。使用 unmodifyingCollection() 并分配给 Set，隐式调用 toSet 复制数据，隐藏了执行复制的事实。

为了确保代码的可读性，即任何阅读代码的人(包括 3 年后的你自己)都会理解它的作用，请使用复制构造函数编写代码来显式复制数据。

嗯，当然，除非这是代码混淆的练习，在这种情况下，这是一个很好的误导性技巧。