这种去重方法真的有用么

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

This topic created in 1412 days ago, the information mentioned may be changed or developed.

为什么 copy 下来运行就报错，空指针，不清楚在网上的例子是怎么跑的

studentList = studentList.stream().collect(
  Collectors.collectingAndThen(
     Collectors.toCollection(() -> new TreeSet<>(Comparator.comparing(Student::getName))), ArrayList::new));

不知道为啥 string 类型字段就报错空指针，int 没错。

就是为了要去重重复对象，重复对象的判断条件是某个字段的值相同。找了好多都是这么写的，但是一跑就报错。很神奇。

ecti

指针

new

字段

22 replies • 2022-07-08 21:29:58 +08:00

AoEiuV020CN

Jul 6, 2022

最好上个完整一点点的代码，
然后是哪个对象在哪里空了，期望是什么，
这里 stream 可读性很糟糕，不如 for 循环，或者 addAll ，

luxinfl

Jul 6, 2022

@AoEiuV020CN 我搞出来了，tm 是别人传的数据有问题。。一个列表里面有个对象搞错字段了。。。

nothingistrue

Jul 6, 2022

原理是，利用 Set 不允许添加重复元素的特性（添加重复元素时忽略，不是报错）来去重。具体操作是，将当前集合的所有元素尝试全部放入一个 Set 中（用哪个 Set 的实现类取决于如何判重），然后在将完成的 Set 转换回原来的集合，在将元素放入到 Set 中的时候，重复元素会被排除掉。但是你抄这段代码可读性是真特么差。

nothingistrue

Jul 6, 2022

尝试换一种写法：

studentList = new TreeSet<>(Comparator.comparing(Student::getName)).addAll(studentList).stream().collect(Collectors.toList())

zhuangzhuang1988

Jul 6, 2022

还是写简单的 foreach 吧
这样看真难看懂
《 effective java 》里也有 stream 的讨论，不要无脑 stream

Duolingo

Jul 6, 2022

数据量不大的话不如直接 removeIf 。

Set<String> tmp = new HashSet<>();
studentList.removeIf(student -> !tmp.add(student.getName()));

Leviathann

Jul 6, 2022

换个库把，stream 除了 parallel 功能，其他完全没 eclipse collection 好用，性能也不如，java 官方的视频都说 eclipse collection 的抽象程度更高且性能也更好

nothingistrue

Jul 6, 2022

好吧，我写错了，addAll 不能链式操作，要想一句代码就完成的话，只能使用原先那段很难看的代码。为了可读性建议拆分语句如下：

TreeSet<Student> tempSet = new TreeSet<>(Comparator.comparing(Student::getName));
tempSet.addAll(studentList);
studentList = new ArrayList(tempSet); // 或者 studentList = tempSet.stream().collect(Collectors.toList())；

xinhochen

Jul 6, 2022

我觉得这样写更容易理解一些：
Set<String> filter = new HashSet<>();
List<Student> result = studentList.stream().filter(o -> filter.add(o.getName())).collect(Collectors.toList());