美文网首页
优化过多的if-else

优化过多的if-else

作者: 小尾巴1024 | 来源:发表于2019-01-24 17:34 被阅读0次

    开发中使用到if-else是再正常不过的了,如果需要判断的条件比较少,使用少量的if-else是最简单不过的,但是如果代码中出现一堆if-else,if-else中又嵌套了一大堆if-else,想必看到这么多代码,你估计会怀疑人生!优秀的编程者想必会去优化或是使用其它方式去代替它,那我们就开始优化之旅吧!

    如果你看到如下这段代码,估计你绝对没有看下去的欲望

          if (city.equals("BJ")){
                List<String> list = Arrays.asList("东城区","西城区","朝阳区");
                list.forEach(item->{
                    if (item.equals("东城区")){
                        //do something
                    }else if (item.equals("西城区")){
                        //do something
                    }else {
                        //....
                    }
                });
            }else if (city.equals("SH")){
                List<String> list = Arrays.asList("黄浦区","徐汇区","长宁区");
                list.forEach(item->{
                    if (item.equals("黄浦区")){
                        //do something
                    }else if (item.equals("徐汇区")){
                        //do something
                    }else {
                        //....
                    }
                });
            }else if(city.equals("SZ")){
                List<String> list = Arrays.asList("罗湖区","福田区","南山区");
                list.forEach(item->{
                    if (item.equals("罗湖区")){
                        //do something
                    }else if (item.equals("福田区")){
                        //do something
                    }else {
                        //....
                    }
                });
            }else if (city.equals("GZ")){
                List<String> list = Arrays.asList("荔湾区","越秀区","海珠区");
                list.forEach(item->{
                    if (item.equals("荔湾区")){
                        //do something
                    }else if (item.equals("越秀区")){
                        //do something
                    }else {
                        //....
                    }
                });
            }else {
                //......
            }
    

    但是,如果你是个有追求的编程者,想必你肯定会想去优化这段代码

    那么,如何重构if-else过多的代码块呢

    • 继续用 if-else,只达到剥离执行代码块
    • 用工厂模式去耦合

    对于这两种其实不是非此即彼的关系,而是优化深度不同。第一种相对比较简单,可以重构成下面这样子

            if (city.equals("BJ")){
                processBJ();
            }else if (city.equals("SH")){
                processSH();
            }else if(city.equals("SZ")){
                processSZ();
            }else if (city.equals("GZ")){
                processGZ();
            }else {
                //......
            }
    

    这样看着顿时像吃了一口薄荷清爽许多
    上面的优化思路是只关注城市的不同,而处理逻辑移交到其它地方处理,这样即使出了错,排查的时候也方便。

    进一步优化,工厂模式

    接下来就是本文的重点知识点了,使用工厂模式优化过多的if-else。
    根据城市的不同,会有不同的处理逻辑,可以把这个处理逻辑抽象出来,使用java多态去实现它。


    image.png
            TargetExecutor executor =null;
            if (city.equals("BJ")){
                executor = new ExecutorBJ();
            }else if (city.equals("SH")){
                executor = new ExecutorSH();
            }else if(city.equals("SZ")){
                executor = new ExecutorSZ();
            }else if (city.equals("GZ")){
                executor = new ExecutorGZ();
            }else {
                //......
            }
            executor.process();
    

    上面这段代码还可以进一步优化,可以定义一个工厂来实现不同条件下new出不同的处理者


    image.png

    在经过使用工厂模式这一轮重构之后,我们之前在一个类里面写的那堆代码已经抽离到多个不同的类里了
    我们只需要在刚开始那里使用以下简单的代码,就可以替换换之前那一堆if-else了,是不是清爽很多!
    TargetExecutor executor = ExecutorFactory.getExecutor(city);
    executor .process();
    简单的现行代码,替换掉了繁多的if-else,而且降低了主类和逻辑类的耦合度。

    参考:https://mp.weixin.qq.com/s/1-Sn4NtJqLImtUohr7qt_g

    相关文章

      网友评论

          本文标题:优化过多的if-else

          本文链接:https://www.haomeiwen.com/subject/qgijjqtx.html