优化过多的if-else

开发中使用到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

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 203,772评论 6 477
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,458评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 150,610评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,640评论 1 276
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,657评论 5 365
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,590评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 37,962评论 3 395
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,631评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,870评论 1 297
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,611评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,704评论 1 329
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,386评论 4 319
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,969评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,944评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,179评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 44,742评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,440评论 2 342

推荐阅读更多精彩内容

  • 第一部分 HTML&CSS整理答案 1. 什么是HTML5? 答:HTML5是最新的HTML标准。 注意:讲述HT...
    kismetajun阅读 27,406评论 1 45
  • 原文出处: Jun.M 几天前的一次上线,脑残手抖不小心写了bug,虽然组里的老大没有说什么,但心里面很是难过。同...
    Albert陈凯阅读 1,068评论 0 2
  • 作者:慕容千语 juejin.im/post/5b893eaef265da434816006c 平时开发中if-e...
    Ernest_Chou阅读 626评论 0 0
  • 划过的光辉未必炫目坠落尘埃亦无声但,曾游过时间的河流飞越过风雨的群山 仅,闪烁过瞬息的光华但在时间的长流中确曾来过...
    俗然阅读 253评论 0 14
  • 不知不觉已经入群2周了,这是我第一篇感赏日记。今天孩子终于去学校了,这是他旷课三周后第一次回学校,感觉一切一如往常...
    Lucy_4807阅读 344评论 2 5