XiaST
V2EX  ›  Java

在 Java 语言中有一堆 instanceof 的 else-if 语句,该如何重构才能使代码更加美观并且效率更快??

  •  
  •   XiaST · Apr 12, 2016 · 8274 views
    This topic created in 3868 days ago, the information mentioned may be changed or developed.

    在写 Java 程序的时候遇到了如下格式的逻辑: if (s instanceof FunctionDef) { ... } else if (s instanceof ClassDef) { ... } else if (s instanceof Return) { ... } ...

    对于这种代码该如何重构才能使得程序看上去更加简洁并且高效?

    20 replies  •  2016-04-12 14:28:54 +08:00
    knightdf
    knightdf
    Apr 12, 2016
    1
    换成 Scala 用 match case, 哈哈
    ahmiao
    ahmiao
    Apr 12, 2016❤️1
    2
    统一接口+子类化?
    wanghaa
    wanghaa
    Apr 12, 2016
    3
    gqlxj1987
    gqlxj1987
    Apr 12, 2016
    4
    工厂+策略模式
    zhuangzhuang1988
    zhuangzhuang1988
    Apr 12, 2016
    5
    import java.util.HashMap;
    import java.util.LinkedList;
    import java.util.List;
    import java.util.Map;

    abstract class Validate {
    abstract boolean validate(Object obj);
    abstract void process(Object val);
    }

    class A {

    }
    class B {

    }

    class AValidate extends Validate {

    @Override
    boolean validate(Object obj) {
    return obj instanceof A;
    }

    @Override
    void process(Object val) {
    A a = (A)val;
    // ...
    }
    }

    class BValidate extends Validate {
    @Override
    boolean validate(Object obj) {
    return obj instanceof B;
    }
    @Override
    void process(Object val) {
    B a = (B)val;
    // ...
    }
    }

    public class Main {
    public static void main(String[] args) {
    Object a = new A();
    Validate[] validates = {
    new AValidate(),
    new BValidate()
    };
    for(Validate v: validates ){
    if(v.validate(a)){
    v.process(a);
    break;
    }
    }
    }
    }
    试试下.
    beanlam
    beanlam
    Apr 12, 2016
    6
    用多态,一个分支对应一个实现。这种方法对于有大量分支的情况可能不适用,因为要写大量的类。
    另外一种模式可以参考《代码大全》第十八章提到的“表驱动模式”。
    SpicyCat
    SpicyCat
    Apr 12, 2016
    7
    @ahmiao 同意用接口加子类。 FunctionDef, ClassDef 和 Return 可以实现同一个接口或者继承同一个父类,假设父类叫 Father, 方法名叫 doSomething(), 然后那三个子类各自实现 doSomething(). 这样代码里只需要调用 Father#doSomething 就可以了。
    levn
    levn
    Apr 12, 2016❤️1
    8
    改成一种更加复杂难懂的方式
    cxshun
    cxshun
    Apr 12, 2016
    9
    面向接口编程,多态就是这个时候用的。
    fwrq41251
    fwrq41251
    Apr 12, 2016
    10
    访问者模式。
    或者用 java8 ,有个叫 javaslang 的类库,有用于类型匹配的类。
    SoloCompany
    SoloCompany
    Apr 12, 2016
    11
    首选是多态
    不适合多态的可以试一下 Kotlin 用 case when 来代替了 if else 而已也没啥区别
    wanttofly
    wanttofly
    Apr 12, 2016
    12
    谢谢回答,最近正好有这方面的修改。
    hrong
    hrong
    Apr 12, 2016 via Android
    13
    @beanlam 表驱动模式对取值还好,要取函数指针就不合适了, JAVA 不支持函数指针,不知道 java8 能否实现
    beanlam
    beanlam
    Apr 12, 2016
    14
    @hrong 是的, jdk1.7 我一般都用匿名内部类作为 value ,伪装函数。
    zhuangzhuang1988
    zhuangzhuang1988
    Apr 12, 2016
    15
    或者看这种解决方式, 当然有点难 `functional-programming-in-scala` 作者写的 http://blog.higher-order.com/blog/2009/08/21/structural-pattern-matching-in-java/
    kaneg
    kaneg
    Apr 12, 2016
    16
    这不就是多态要解决的问题吗
    twoyuan
    twoyuan
    Apr 12, 2016
    17
    我现在是建一个 Map<Class, Callback> map ,用的时候 map.get(object.class)。不过第一次创建时会慢点
    Jaylee
    Jaylee
    Apr 12, 2016
    18
    这应该是用多态吧
    laibin
    laibin
    Apr 12, 2016
    19
    访问者模式。
    haolly
    haolly
    Apr 12, 2016
    20
    重構裏面說過,當你看到大量 switch 或者 if-else 的時候,這表示你應該用多態了
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   2136 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 60ms · UTC 05:14 · PVG 13:14 · LAX 21:14 · JFK 00:14
    ♥ Do have faith in what you're doing.