stevenkang
V2EX  ›  问与答

探讨三种方式哪种效率高、可读性、可维护性好: if-else、switch、和另外一种

  •  
  •   stevenkang · Aug 15, 2019 · 4833 views
    This topic created in 2639 days ago, the information mentioned may be changed or developed.

    我们现在有一个参数 type,需要根据 type 进行相应的代码处理( type 大概有 20+ 种情况),用题目中的三种方式如下

    if-else 方式

    if (type == 'type1') {
        // todo something
    } else if (type == 'type2') {
        // todo something
    }
    ...
    else if (type == 'typeN') {
        // todo something
    } else {
        // otherwise
    }
    

    switch 方式

    swtich(type) {
        case 'type1': /* todo something */ break;
        case 'type2': /* todo something */ break;
        ...
        case 'typeN': /* todo something */ break;
        default: /* otherwise */ break;
    }
    

    另外一种(我不知道这叫什么方式)

    static {
        // 这里的 put 可以通过别的方式装载,可以一次性写好了之后,需要扩展时主动添加到 map 里面(也可以用注解注入等方式)
        map.put('type1', new Type1Process())
        map.put('type2', new Type2Process())
        ...
        map.put('typeN', new TypeNProcess())
        map.put('default', new DefaultProcess())
    }
    proc = map.containsKey(type) ? map.get(type) : map.get('default')
    proc.exec()
    

    从执行效率、可读性、可维护性等多个方面来看,哪种方式综合评分更高?

    24 replies  •  2019-08-16 10:12:19 +08:00
    z42514
    z42514
    Aug 15, 2019
    1
    我选三
    TomVista
    TomVista
    Aug 15, 2019
    2
    20 种 type 可以排除 if else 了吧
    mara1
    mara1
    Aug 15, 2019
    3
    放到字典里,差不多就是 3
    hmellochan
    hmellochan
    Aug 15, 2019
    4
    那么多类型,明显三好。
    Vegetable
    Vegetable
    Aug 15, 2019
    5
    2 和 3 在底层差不多是一样的吧,条件这么多我选 3.
    672795574
    672795574
    Aug 15, 2019❤️1
    6
    1,2 我选 1,踩过忘记加 break 的坑(自己的问题)
    3 应该是抽象过了,看着也有点符合开闭原则
    因此理论上新加一个 type 和 Processor 不需要测试原来的逻辑。
    fuxiao11
    fuxiao11
    Aug 15, 2019 via Android
    7
    3 其实是责任链模式的一种变形实现
    kkkkkrua
    kkkkkrua
    Aug 15, 2019
    8
    spring 直接 getBean("xxx_TYPE1").exec()不更好么,
    其他方式直接 new 对应的实例,不用 if..
    Justin13
    Justin13
    Aug 15, 2019 via Android
    9
    当然是第三种字典啦。
    zael
    zael
    Aug 15, 2019 via iPhone
    10
    性能上哪个最高呢
    ohmycodeape
    ohmycodeape
    Aug 15, 2019
    11
    没看出来是什么语言,但是确定第三种方法是在 get 时才去实例化的吗?看着写法有点担心 put 时直接 new 了.
    autoxbc
    autoxbc
    Aug 15, 2019
    12
    2 好,switch 故意不写 break 也是一种写法

    语句由上而下叠加执行,在需要共享一部分处理逻辑时有奇效。当然要防止别人打你
    ianva
    ianva
    Aug 15, 2019
    13
    表驱动
    jadec0der
    jadec0der
    Aug 15, 2019
    14
    第三种在代码大全 18 章叫「 表驱动法」
    Leammin
    Leammin
    Aug 15, 2019 via Android
    15
    少的时候 2,多的时候 3
    murmur
    murmur
    Aug 15, 2019
    16
    我投 gotoy 一票
    ywcjxf1515
    ywcjxf1515
    Aug 15, 2019 via iPad
    17
    谷歌搜 if 策略模式 map v2ex,或者搜 if 状态模式 map v2ex。
    zw1one
    zw1one
    Aug 15, 2019 via Android
    18
    如果你 if 里的条件全长这个样子,那肯定 3 好。如果 if 条件会突然多两个奇怪的判断,用 3 就不好改。
    pastgift
    pastgift
    Aug 15, 2019
    19
    2 到 5 个分支用,每个分支代码量中等( 20 行以内) if-else
    3 到 10 个分支,每个分支代码量很少( 10 行以内) switch-case
    2 个分支以上,每个分支代码量很多,表驱动方式

    具体操作按个人喜好,项目要求,统一性洁癖,美观等因素调整
    写代码不要太死板
    geelaw
    geelaw
    Aug 15, 2019 via iPhone
    20
    第三种叫做 branch/jump table。
    woscaizi
    woscaizi
    Aug 15, 2019 via iPhone
    21
    第三种是 查表法?
    Xbluer
    Xbluer
    Aug 15, 2019
    22
    @tomoya92 #10 这里几乎不用考虑性能问题。
    stevenkang
    stevenkang
    OP
    Aug 16, 2019
    23
    @sun2920989 这里应该可以不限语言,各个语言都可以这样写。put 的时候是初始化 type 和处理的映射,可以用多种方式初始化,不一定 new。

    @kkkkkrua 对,加上 spring 的一些特性,写出来更优雅。这里没有专门突出 spirng 是方便其他语言的使用者也可以参考、讨论研究。

    @jadec0der 表驱动法,优势是不是更方便增、减 type 以及热插拔?
    @pastgift 对,灵活应用更重要。这里探讨一下这三种方式也是比较各自的优缺点,方便在使用时灵活应对。

    @geelaw C 语言里面的知识吗?
    lllllliu
    lllllliu
    Aug 16, 2019
    24
    EventBus/Notification Center 可以么。 对于多特性的同一种事件,我大部分用的都是用的 EventBus + Factory,或者干脆都变成一个独立的事件。。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   2155 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 73ms · UTC 15:53 · PVG 23:53 · LAX 07:53 · JFK 10:53
    ♥ Do have faith in what you're doing.