这种去重方法真的有用么

[luxinfl]

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

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

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

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

[AoEiuV020CN]

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

[luxinfl]

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

[nothingistrue]

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

[nothingistrue]

尝试换一种写法：

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

[zhuangzhuang1988]

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

[codingadog]

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

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

[Leviathann]

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

[nothingistrue]

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

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

[xinhochen]

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

[rpish]

好奇问下,大家项目开发中 stream 用得多吗?

[Red998]

用的很多 jdk8 stream 流用的很多

[chendy]

studentList = new ArrayList<>(new HashSet<>(studentList));

[mazai]

只是因为 students 中 name 有 null 的学生吧。 还有只是去重不用这么麻烦，重写 Student 对象的 hashcode 和 equals 方法，studentList.stream().distinct().collect(Collectors.toList()); 就行了

[git00ll]

这么写可读性贼差

[golangLover]

@mazai 同意有 null 的解释
但不同意重写 equal. 可能我只是需要比较这个属性一次

[potatowish]

@git00ll 可以抽一个常用的方法出来，每次这么写一堆确实难受

[lmshl]

归根结底还是 Java 设计的 API 太丑了，丑破天际，搁 Scala 里就一行的事
val result = studentList.distinctBy(_.name)

[wolfie]

骚操作 new ArrayList(toMap(getName()).values())

[Anshay]

直接 studentList.stream().distinct.toCollect.....即可，前提是有自己编写了 studentList 中的 equals 方法和 hashCode 方法。

[mazai]

@golangLover equals 和 hashcode 具体选取哪些字段重写还是看业务，哪几个字段能代表学生的唯一性，那就重写哪几个字段

[issakchill]

streamEx 有些方法还挺好用的 也自带根据某字段去重的方法

[golangLover]

@mazai 。。你是不可能一两次比较就重写字段的