被人说 Lambda 代码像...，那是没用下面这三个方法
说 Lambda 写的代码像屎山，其实就是代码不够干净嘛。说到底并不是不会用 Lambda 本身的 API，而是用的方式不对。
Java Lambda 本身提供了非常丰富的方法库，大多数时候我们常用的方法也就为数不多的那几个。Lambda 的使用方法之前专门写过文章，8000字，让你彻底了解 Java 8 的 Lambda、函数式接口、Stream 用法和原理。在掘金社区已经获得了将近600个赞，1200多个收藏。
之所以被diss，也大概并不是你用了其中某个不常用的方法（那样别人可能觉得你见多识广）。更多的时候可能是因为这几个原因：
- 代码写太乱了。一个人说乱那可能是那个人的问题，如果大家都说乱，不好意思，那基本上就是你的问题了。
- Lambda 函数式本身的写法，尤其是对于从未接触过函数式编程的开发来说，这种写法本身就不太习惯，甚至不喜欢。负面情绪先入为主了，自然就觉得乱了。
- 还有就是一直被诟病调试问题，Lambda 公认的不便于调试。
先来看一段代码，也就是经常被人（除自己外的所有人）说的屎山代码。
private static List dirtyLambda(){
List userList = User.buildUserList(30);
List simpleUserList = userList.stream()
.filter(user -> {
return user.getGender().equals(1)
&& user.getAge() >= 18 && user.getAge() <= 45;
})
.map(user -> {
SimpleUser su = new SimpleUser();
su.setName(user.getName());
su.setAge(user.getAge());
Optional addressOptional = user.getAddressList().stream()
.findFirst();
if (addressOptional.isPresent()) {
su.setProvince(addressOptional.get().getProvince());
}
return su;
})
.sorted(Comparator.comparingInt(SimpleUser::getAge))
.collect(Collectors.toList());
return simpleUserList;
} 如果不做解释，是不是脏话马上就要出来了。这其实在屎山代码中也最多拍个中等，最起码该换行的换行了，比如那个filter 中的三个并列条件，恐怕你是没见过与或非排列组合的写法，加上不怎么换行，那是真的让人抓狂。
如果你觉得这代码还可以，那有可能你也这么写过。
不瞒各位，这样的代码我曾经写过，而且一天之内不知道写了多少行。曾经有一个需求，一个很复杂的报表，100多个变量+图表+表格，什么最大值、最小值、环比、同比、正序、倒序、top3、top5、top10等等，就是各种能想到的维度统统算一遍。有经验的同学一看就知道，这妥妥的体力活儿啊，但是时间只有一天，没办法，越写越烦躁，直接躺平了，比上面这种更屎的代码一段接一段的写啊。写完别说改了，看都不敢看啊。
说回正题，上面那个代码的逻辑是这样的：
- 在一个 User列表中筛选男性，且年龄为18到45岁之间的；
- 然后将 User转换为 SimpleUser类型，获取姓名、年龄，以及地址列表（假设一个人有多个地址）中第一个的省份字段；
- 然后排序，按照年龄正序排序；
- 最后返回一个 SimpleUser 列表；
那怎么做才能让代码变得清晰易懂，告别屎山 Lambda 呢？
不管你用什么办法，只要做到下面这3点，Lambda 代码块立马变清晰，最后一点可以适度放宽。
不要超过 5 行
这其实没什么好说的，本身代码规约中就要求最好不写超大方法，也就是行数过多的方法，更何况是在 Lambda 中。在函数式编程中，你写的代码其实是在小括号中，作为参数的形式出现的，一个多行的参数，不敢想啊。
不超过5行可不是说把换行符去掉，把之前的100行直接边 1 行啊。而是下面这样子，stream()就算一行了， 之后每个.function()都算一行，加起来不超过5行。
userList.stream()
.filter()
.filter()
.map()
.collect(Collectors.toList());不要一个stream() 后面跟3个filter，4个map，再来个排序，再整个分组，有那么复杂的业务吗，如果有，想想可能在上层设计的时候就出现问题了。
不要出现花括号
不要出现花括号，这其实就是缩短代码行数的一个根本方法。用这个方法，强制你将逻辑抽离出来，这样，你的代码逻辑就会马上变清晰，立竿见影。
拿前面的那端代码举个例子，其中map方法将 User转换为SimpleUser，里面有赋值操作，还有一些判断逻辑。
.map(user -> {
SimpleUser su = new SimpleUser();
su.setName(user.getName());
su.setAge(user.getAge());
Optional addressOptional = user.getAddressList().stream()
.findFirst();
if (addressOptional.isPresent()) {
su.setProvince(addressOptional.get().getProvince());
}
return su;
})直接将一段抽取成方法，在 IDEA 中操作也非常方便。选中花括号中的代码，然后右键->Refactor->Extract Method，直接抽取出方法，连名字都帮忙取号了。
图片
同样的，filter()中的三个条件判断也抽离出来。然后效果就是下面这样，每一行的意图都很清晰，谁还会说不能理解。
private static List dirtyLambda(){
List userList = User.buildUserList(30);
List simpleUserList = userList.stream()
.filter(user -> filterUser(user))
.map(user -> getSimpleUser(user))
.sorted(Comparator.comparingInt(SimpleUser::getAge))
.collect(Collectors.toList());
return simpleUserList;
}
private static boolean filterUser(User user) {
return user.getGender().equals(1)
&& user.getAge() >= 18 && user.getAge() <= 45;
}
private static SimpleUser getSimpleUser(User user) {
SimpleUser su = new SimpleUser();
su.setName(user.getName());
su.setAge(user.getAge());
Optional addressOptional = user.getAddressList().stream()
.findFirst();
if (addressOptional.isPresent()) {
su.setProvince(addressOptional.get().getProvince());
}
return su;
} 最好连 -> 都不要出现
再进一步，就是将 ->也干掉，虽然 ->后面没有花括号已经很简洁了，但是去掉->就不只是简洁了，而是优雅了。
不用->，取而代之的是 ::，最终，去掉->后的代码是下面这样子。
private static List dirtyLambda(){
List userList = User.buildUserList(30);
return userList.stream()
.filter(CleanLambda::filterUser)
.map(CleanLambda::getSimpleUser)
.sorted(Comparator.comparingInt(SimpleUser::getAge))
.collect(Collectors.toList());
}
private static boolean filterUser(User user) {
return user.getGender().equals(1)
&& user.getAge() >= 18 && user.getAge() <= 45;
}
private static SimpleUser getSimpleUser(User user) {
SimpleUser su = new SimpleUser();
su.setName(user.getName());
su.setAge(user.getAge());
Optional addressOptional = user.getAddressList().stream()
.findFirst();
addressOptional.ifPresent(address -> su.setProvince(address.getProvince()));
return su;
} 最后
本文只是抛砖引玉，并没有介绍太细节的 Lambda 用法。授人以鱼不如授人以渔，聪明人早就这样写了，更聪明的人已经去改代码了。