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

[XiaST]

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

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

[knightdf]

换成 Scala 用 match case, 哈哈

[ahmiao]

统一接口＋子类化？

[wanghaa]

https://book.douban.com/subject/4262627/ 这本书里有讲到

[gqlxj1987]

工厂+策略模式

[zhuangzhuang1988]

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]

用多态，一个分支对应一个实现。这种方法对于有大量分支的情况可能不适用，因为要写大量的类。
另外一种模式可以参考《代码大全》第十八章提到的“表驱动模式”。

[SpicyCat]

@ahmiao 同意用接口加子类。 FunctionDef, ClassDef 和 Return 可以实现同一个接口或者继承同一个父类，假设父类叫 Father, 方法名叫 doSomething(), 然后那三个子类各自实现 doSomething(). 这样代码里只需要调用 Father#doSomething 就可以了。

[levn]

改成一种更加复杂难懂的方式

[cxshun]

面向接口编程，多态就是这个时候用的。

[fwrq41251]

访问者模式。
或者用 java8 ，有个叫 javaslang 的类库，有用于类型匹配的类。

[SoloCompany]

首选是多态
不适合多态的可以试一下 Kotlin 用 case when 来代替了 if else 而已也没啥区别

[wanttofly]

谢谢回答，最近正好有这方面的修改。

[hrong]

@beanlam 表驱动模式对取值还好，要取函数指针就不合适了， JAVA 不支持函数指针，不知道 java8 能否实现

[beanlam]

@hrong 是的， jdk1.7 我一般都用匿名内部类作为 value ，伪装函数。

[zhuangzhuang1988]

或者看这种解决方式, 当然有点难 `functional-programming-in-scala` 作者写的 http://blog.higher-order.com/blog/2009/08/21/structural-pattern-matching-in-java/

[kaneg]

这不就是多态要解决的问题吗

[twoyuan]

我现在是建一个 Map<Class, Callback> map ，用的时候 map.get(object.class)。不过第一次创建时会慢点

[Jaylee]

这应该是用多态吧

[laibin]

访问者模式。

[haolly]

重構裏面說過，當你看到大量 switch 或者 if-else 的時候，這表示你應該用多態了