查看原文
其他

Lambda 代码像屎山?三个方法解决!

The following article is from 古时的风筝 Author 风筝


说 Lambda 写的代码像屎山,其实就是代码不够干净嘛。说到底并不是不会用 Lambda 本身的 API,而是用的方式不对。

Java Lambda 本身提供了非常丰富的方法库,大多数时候我们常用的方法也就为数不多的那几个。

之所以被diss,也大概并不是你用了其中某个不常用的方法(那样别人可能觉得你见多识广)。更多的时候可能是因为这几个原因:

  1. 代码写太乱了。一个人说乱那可能是那个人的问题,如果大家都说乱,不好意思,那基本上就是你的问题了。
  2. Lambda 函数式本身的写法,尤其是对于从未接触过函数式编程的开发来说,这种写法本身就不太习惯,甚至不喜欢。负面情绪先入为主了,自然就觉得乱了。
  3. 还有就是一直被诟病调试问题,Lambda 公认的不便于调试。

先来看一段代码,也就是经常被人(除自己外的所有人)说的屎山代码。

private static List<SimpleUser> dirtyLambda(){
 List<User> userList = User.buildUserList(30);
 List<SimpleUser> 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<Address> 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等等,就是各种能想到的维度统统算一遍。有经验的同学一看就知道,这妥妥的体力活儿啊,但是时间只有一天,没办法,越写越烦躁,直接躺平了,比上面这种更屎的代码一段接一段的写啊。写完别说改了,看都不敢看啊。

说回正题,上面那个代码的逻辑是这样的:

  1. 在一个 User列表中筛选男性,且年龄为18到45岁之间的;
  2. 然后将 User转换为 SimpleUser类型,获取姓名、年龄,以及地址列表(假设一个人有多个地址)中第一个的省份字段;
  3. 然后排序,按照年龄正序排序;
  4. 最后返回一个 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<Address> addressOptional = user.getAddressList().stream()
   .findFirst();
 if (addressOptional.isPresent()) {
  su.setProvince(addressOptional.get().getProvince());
 }
 return su;
})

直接将一段抽取成方法,在 IDEA 中操作也非常方便。选中花括号中的代码,然后右键->Refactor->Extract Method,直接抽取出方法,连名字都帮忙取号了。

同样的,filter()中的三个条件判断也抽离出来。然后效果就是下面这样,每一行的意图都很清晰,谁还会说不能理解。

    private static List<SimpleUser> dirtyLambda(){
        List<User> userList = User.buildUserList(30);
        List<SimpleUser> 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<Address> addressOptional = user.getAddressList().stream()
                .findFirst();
        if (addressOptional.isPresent()) {
            su.setProvince(addressOptional.get().getProvince());
        }
        return su;
    }

最好连 -> 都不要出现

再进一步,就是将 ->也干掉,虽然 ->后面没有花括号已经很简洁了,但是去掉->就不只是简洁了,而是优雅了。

不用->,取而代之的是 ::,最终,去掉->后的代码是下面这样子。

private static List<SimpleUser> dirtyLambda(){
 List<User> 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<Address> addressOptional = user.getAddressList().stream()
   .findFirst();
 addressOptional.ifPresent(address -> su.setProvince(address.getProvince()));
 return su;
}

最后

本文只是抛砖引玉,并没有介绍太细节的 Lambda 用法。授人以鱼不如授人以渔,聪明人早就这样写了,更聪明的人已经去改代码了。


往期推荐

2.5 万人!编程导航到底是什么?

我的新网站上线啦!

为什么我不建议大学生接公司单?

时代变了,Spring 官方抛弃了 Java 8!

我做了个网站,帮你写出满分简历

用 Java 做个命令行程序,太简单了!

继续滑动看下一个

Lambda 代码像屎山?三个方法解决!

向上滑动看下一个

您可能也对以下帖子感兴趣

文章有问题?点此查看未经处理的缓存