一段代码被老大要求重构了六次,我心态崩了

爱写Bug的麦洛
• 阅读 1996

一段代码被老大要求重构了六次,我心态崩了

前言

Hi,大家好,我是麦洛。我又回来啦🙈

进来给大家八卦一段,看看我自己都去干啥了?话说最近公司接了一个农产品交易网站新项目,因为一段代码重构问题差点和老大干起来,本来以为是老大故意刁难我。最后还是发现是我太菜了😏,事情是这个样子滴!

在周例会上,老大告知我们最近接了一个农产品交易平台,主要用于全省农产品线上交易。首当其中,就是要把我们甘肃省的黄河蜜推销出去,我就被安排卖瓜!嗷,不,卖瓜这个功能我负责开发🤣;很快我设计下面的类来定义瓜 Melon 类:

/**
 * 瓜
 * @author Milo Lee
 * @date 2021-04-07 13:21
 */
public class Melon {
    /**品种*/
    private final String type;
    /**重量*/
    private final int weight;
    /**产地*/
    private final String origin;

    public Melon(String type, int weight, String origin) {
        this.type = type;
        this.weight = weight;
        this.origin = origin;
    }
    // getters, toString()方法省略
}

经过一顿CRUD骚操作,写完了瓜类增删改查工作,交工下班🤗。

第一次 按类型筛选瓜类

第二天,老大给我提了一个问题,说增加能够按瓜类型对瓜进行过滤。这不很简单吗?于是,于是我创建了一个 Filters 类, 实现了一个filterMelonByType方法

/**
 * @author Milo Lee
 * @date 2021-04-07 13:25
 */
public class Filters {

    /**
     * 根据类型筛选瓜类
     * @param melons 瓜类
     * @param type 类型
     * @return
     */
    public static List<Melon> filterMelonByType(List<Melon> melons, String type) {

        List<Melon> result = new ArrayList<>();
        for (Melon melon: melons) {
            if (melon != null && type.equalsIgnoreCase(melon.getType())) {
                result.add(melon);
            }
        }
        return result;
    }
}

搞定,我们来测试一下

    public static void main(String[] args) {
        ArrayList<Melon> melons = new ArrayList<>();
        melons.add(new Melon("羊角蜜", 1, "泰国"));
        melons.add(new Melon("西瓜", 2, "三亚"));
        melons.add(new Melon("黄河蜜", 3, "兰州"));
        List<Melon> melonType = Filters.filterMelonByType(melons, "黄河蜜");
        melonType.forEach(melon->{
        System.out.println("瓜类型:"+melon.getType());
        });
    }

一段代码被老大要求重构了六次,我心态崩了

没毛病,拿给老大看看去,老大看了我的代码说:如果我让你在增加一个按重量筛选瓜类,你打算怎么写?回去考虑一下吧,这家伙不会故意找我茬把?😪

第二次 按重量筛选瓜类

回到座位的我心想,上次我已经实现了按类型筛选瓜类,那我给他copy一份改改吧!

如下所示:

    /**
     * 按照重量过滤瓜类
     * @param melons
     * @param weight
     * @return
     */
    public static List<Melon> filterMelonByWeight(List<Melon> melons, int weight) {

        List<Melon> result = new ArrayList<>();
        for (Melon melon: melons) {
            if (melon != null && melon.getWeight() == weight) {
                result.add(melon);
            }
        }
        return result;
    }
public static void main(String[] args) {
         ArrayList<Melon> melons = new ArrayList<>();
        melons.add(new Melon("羊角蜜", 1, "泰国"));
        melons.add(new Melon("西瓜", 2, "三亚"));
        melons.add(new Melon("黄河蜜", 3, "兰州"));
        List<Melon> melonType = Filters.filterMelonByType(melons, "黄河蜜");
        melonType.forEach(melon->{
            System.out.println("瓜类型:"+melon.getType());
        });

        List<Melon> melonWeight = Filters.filterMelonByWeight( melons,3);
        melonWeight.forEach(melon->{
            System.out.println("瓜重量:"+melon.getWeight());
        });
    }

一段代码被老大要求重构了六次,我心态崩了

程序员最喜欢的方式,CV搞定,哈哈。但是我发现filterByWeight()filterByType() 非常相似,就是过滤条件不同。我心想,老大不会让我写一个按类型和重量筛选瓜类吧。拿着我的代码去给老大看,果不其然,怕什么来什么😟。

第三次 按类型和重量筛选瓜类

为了满足完成老大的任务,我将上面的代码进行了糅合,很快写了如下的代码

    /**
     * 按照类型和重量来筛选瓜类
     * @param melons
     * @param type
     * @param weight
     * @return
     */
    public static List<Melon> filterMelonByTypeAndWeight(List<Melon> melons, String type, int weight) {

        List<Melon> result = new ArrayList<>();
        for (Melon melon: melons) {
            if (melon != null && type.equalsIgnoreCase(melon.getType()) && melon.getWeight() == weight) {
                result.add(melon);
            }
        }
        return result;
    }

一段代码被老大要求重构了六次,我心态崩了

老大看了我的代码说,看来你还是没有理解我的意思。假如今天不光我,还有客户继续这样提需求。

那么 Filters 将会有很多这样类似的方法,也就是说写了很多样板代码(代码冗余但又不得不写);

在我们程序员看来,这是不能接受的。如果继续添加新的过滤条件,则代码将变得难以维护且容易出错。你去了解一下lambda表达式函数式接口知识点,再修改一下你的代码。我已经确定了,他就是和我过不去😤

第四次 将行为作为参数传递

经过上面的三番折腾。我发现理论上Melon类的任何属性都有可能作为过滤条件,这样的话我们的Filter类将会有大量的样板代码,而且有些方法会非常复杂。

其实我们可以发现,我们每写一个方法,都对应一种查询行为,查询行为必然对应一种过滤条件。有没有办法我们写一个方法,将查询行为作为参数传递进去,从而返回我们的结果呢?

那么给它取了一个名字:行为参数化,在下图中进行了说明(左侧显示了我们现在拥有的;右侧显示了我们想要的),有没有发现样板代码会明显减少🤔

一段代码被老大要求重构了六次,我心态崩了

如果我们将过滤条件视为一种行为,那么将每种行为视为接口的实现是非常直观的。经过分析我们发现以上所有这些行为都有一个共同点:过滤条件boolean 类型的返回 。抽象一个接口如下

public interface MelonPredicate {
  boolean test(Melon melon);
}

例如,过滤 黄河蜜可以这样写: HHMMelonPredicate

public class HHMMelonPredicate implements MelonPredicate {

     @Override
     public boolean test(Melon melon) {
       return "黄河蜜".equalsIgnoreCase(melon.getType());

     }

}

以此类推,我们也可以过滤一定重量的瓜:

public class WeightMelonPredicate implements MelonPredicate {

     @Override
     public boolean test(Melon melon) {
       return melon.getWeight() > 5000;
     }

}

其实熟悉设计模式的同学应该知道这就是:策略设计模式

主要思想就是让系统在运行时动态选择需要调用的方法。所以我们可以认为 MelonPredicate 接口统一了所有专用于筛选瓜类的算法,并且每种实现都是一种策略,我们也可以把它理解为一种行为。

目前,我们利用策略设计模式,将查询行为进行了抽象。我们还需要一个方法接收 MelonPredicate 参数。于是我定义了 filterMelons() 方法,如下所示:

public static List<Melon> filterMelons(List<Melon> melons, MelonPredicate predicate) {

    List<Melon> result = new ArrayList<>();
    for (Melon melon: melons) {
      if (melon != null && predicate.test(melon)) {
        result.add(melon);
      }

    }  
    return result;
}

大功告成,测试一下,果然比之前好用多了,再让老大瞅瞅去

List<Melon> hhm = Filters.filterMelons(melons, new HHMMelonPredicate());

List<Melon> weight = Filters.filterMelons(melons, new WeightMelonPredicate());

第五次 一次性加了100个过滤条件

就在我沾沾自喜时候,老大又给他泼了一盆冷水。他说你以为我们的平台就买黄河蜜啊,如果前前后后有几十种瓜品种,我给你列出100种过滤条件,你怎么办?

我的心里一万个草泥马在奔腾啊😫!老大是不是存心和我过不去啊!虽然经过上次改造,我的代码已经足够灵活,但是如果突然增加100个过滤条件,我仍然需要编写100个策略类来实现 每一个过滤条件。然后我们需要将策略传递给 filterMelons() 方法。

有没有不需要创建这些类的办法那?聪明的我很快发现可以使用java匿名内部类

如下所示:

List<Melon> europeans = Filters.filterMelons(melons, new MelonPredicate() {

     @Override

     public boolean test(Melon melon) {

       return "europe".equalsIgnoreCase(melon.getOrigin());

     }

});

虽然向前跨了一大步,但好像无济于事。我还是需要编写大量的代码实现此次需求。设计匿名内部类的目的,就是为了方便 Java 程序员将代码作为数据传递。有时候,匿名内部类看这比较复杂,这时候,我突然想起来老大让我学的lambda表达式,我可以用它来简化

List<Melon> europeansLambda = Filters.filterMelons(
  melons, m -> "europe".equalsIgnoreCase(m.getOrigin())
);

果然看这帅多了!!!,就这样,我又一次成功完成了任务。我兴冲冲的拿着代码让老大去看看。

第六次 引入泛型

老大看着我的代码说,嗯,不错!脑袋终于开窍了。现在你考虑一下,我们的平台是做农产品的,也就是肯定不止瓜这一类水果,如果换做其他的水果,你的代码如何修改?

目前我们的MelonPredicate仅支持 Melon 类。这家伙怎么搞?说不定哪天他要买蔬菜、海参可怎么搞,总不能给他再创建好多类似MelonPredicate的接口吧。这个时候突然想起老师讲过的泛型,该它发挥作用了!

于是我定义了一个新接口Predicate

@FunctionalInterface
public interface Predicate<T> {

  boolean test(T t);

}

接下来,我们重写该 filterMelons() 方法并将其重命名为 filter()

public static <T> List<T> filter(List<T> list, Predicate<T> predicate) {

    List<T> result = new ArrayList<>();

    for (T t: list) {

      if (t != null && predicate.test(t)) {

        result.add(t);

      }

    }  

    return result;

}

现在,我们可以这样过滤瓜类 :

List<Melon> watermelons = Filters.filter(

  melons, (Melon m) -> "Watermelon".equalsIgnoreCase(m.getType()));

同样的,我们也可以对数字做同样的事情:

List<Integer> numbers = Arrays.asList(1, 13, 15, 2, 67);

List<Integer> smallThan10 = Filters.filter(numbers, (Integer i) -> i < 10);

回过头来复盘一下,我们发现自从使用Java 8函数式接口和lambda表达式后,代码发生了明显的变化。

不知道细心的伙伴有没有发现我们上面的 Predicate 接口上面多了一个@FunctionalInterface 上的注解,它就是标记函数式接口的。

至此,我们通过一个需求的演变过程,了解了lambda和函数式接口的概念,同时也加深对它们的理解。其实熟悉java8的朋友都知道,在我们的 java.util.function 包下包含40多个此类接口

一段代码被老大要求重构了六次,我心态崩了

函数式接口lambda表达式组成了一个强大的团队。根据上面的例子,我们知道函数式接口是我们行为的高度抽象,lambda表达式我们可以看出这种行为的具体实现的一个实例。

Predicate<Melon> predicate = (Melon m)-> "Watermelon".equalsIgnoreCase(m.getType());

简而言之Lambda

lambda表达式由三部分组成,如下图所示:

一段代码被老大要求重构了六次,我心态崩了

以下是lambda表达式各部分的描述:

  • 在箭头的左侧,是在lambda表达式主体中使用的参数。
  • 在箭头的右侧,是lambda主体 。
  • 箭头只是lambda参数和主体的分隔符。

lambda的匿名类版本如下:

List<Melon> europeans = Filters.filterMelons(melons, new Predicate<Melon>() {

 @Override

 public boolean test(Melon melon) {

   return "Watermelon".equalsIgnoreCase(melon.getType());

 }

});

现在,如果我们查看lambda表达式及其匿名类版本,可以从下面四方面来描述lambda表达式

一段代码被老大要求重构了六次,我心态崩了

我们可以将 lambda 表达式定义为一种 简洁、可传递的匿名函数,首先我们需要明确 lambda 表达式本质上是一个函数,虽然它不属于某个特定的类,但具备参数列表、函数主体、返回类型,甚至能够抛出异常;其次它是匿名的,lambda 表达式没有具体的函数名称;lambda 表达式可以像参数一样进行传递,从而简化代码的编写。

Lambda支持行为参数化,在前面的例子中,我们已经证明这一点。最后,请记住,lambda表达式只能在函数式接口的上下文中使用。

总结

在本文中,我们重点介绍了函数式接口的用途和可用性,我们将研究如何将代码从开始的样板代码现演变为基于函数式接口的灵活实现。希望对大家理解函数式接口有所帮助,谢谢大家。

点赞
收藏
评论区
推荐文章
blmius blmius
3年前
MySQL:[Err] 1292 - Incorrect datetime value: ‘0000-00-00 00:00:00‘ for column ‘CREATE_TIME‘ at row 1
文章目录问题用navicat导入数据时,报错:原因这是因为当前的MySQL不支持datetime为0的情况。解决修改sql\mode:sql\mode:SQLMode定义了MySQL应支持的SQL语法、数据校验等,这样可以更容易地在不同的环境中使用MySQL。全局s
皕杰报表之UUID
​在我们用皕杰报表工具设计填报报表时,如何在新增行里自动增加id呢?能新增整数排序id吗?目前可以在新增行里自动增加id,但只能用uuid函数增加UUID编码,不能新增整数排序id。uuid函数说明:获取一个UUID,可以在填报表中用来创建数据ID语法:uuid()或uuid(sep)参数说明:sep布尔值,生成的uuid中是否包含分隔符'',缺省为
待兔 待兔
4个月前
手写Java HashMap源码
HashMap的使用教程HashMap的使用教程HashMap的使用教程HashMap的使用教程HashMap的使用教程22
Jacquelyn38 Jacquelyn38
3年前
2020年前端实用代码段,为你的工作保驾护航
有空的时候,自己总结了几个代码段,在开发中也经常使用,谢谢。1、使用解构获取json数据let jsonData  id: 1,status: "OK",data: 'a', 'b';let  id, status, data: number   jsonData;console.log(id, status, number )
项目实战,动态增删form表单
前言Hi,大家好,我是麦叔。今天老大让我做一个需求,我们的这个表单以前只支持录入一个检查器具。现在要求改为可以动态添加,满足录入多个器具。作为前
Wesley13 Wesley13
3年前
Java日期时间API系列31
  时间戳是指格林威治时间1970年01月01日00时00分00秒起至现在的总毫秒数,是所有时间的基础,其他时间可以通过时间戳转换得到。Java中本来已经有相关获取时间戳的方法,Java8后增加新的类Instant等专用于处理时间戳问题。 1获取时间戳的方法和性能对比1.1获取时间戳方法Java8以前
Stella981 Stella981
3年前
Android So动态加载 优雅实现与原理分析
背景:漫品Android客户端集成适配转换功能(基于目标识别(So库35M)和人脸识别库(5M)),导致apk体积50M左右,为优化客户端体验,决定实现So文件动态加载.!(https://oscimg.oschina.net/oscnet/00d1ff90e4b34869664fef59e3ec3fdd20b.png)点击上方“蓝字”关注我
Wesley13 Wesley13
3年前
00:Java简单了解
浅谈Java之概述Java是SUN(StanfordUniversityNetwork),斯坦福大学网络公司)1995年推出的一门高级编程语言。Java是一种面向Internet的编程语言。随着Java技术在web方面的不断成熟,已经成为Web应用程序的首选开发语言。Java是简单易学,完全面向对象,安全可靠,与平台无关的编程语言。
Wesley13 Wesley13
3年前
MySQL部分从库上面因为大量的临时表tmp_table造成慢查询
背景描述Time:20190124T00:08:14.70572408:00User@Host:@Id:Schema:sentrymetaLast_errno:0Killed:0Query_time:0.315758Lock_
Python进阶者 Python进阶者
10个月前
Excel中这日期老是出来00:00:00,怎么用Pandas把这个去除
大家好,我是皮皮。一、前言前几天在Python白银交流群【上海新年人】问了一个Pandas数据筛选的问题。问题如下:这日期老是出来00:00:00,怎么把这个去除。二、实现过程后来【论草莓如何成为冻干莓】给了一个思路和代码如下:pd.toexcel之前把这