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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

已注册用户请登录

这是一个创建于 2929 天前的主题，其中的信息可能已经有所发展或是发生改变。

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

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

instanceof

else

重构

Java

20 条回复 • 2016-04-12 14:28:54 +08:00

knightdf

2016-04-12 09:23:15 +08:00

换成 Scala 用 match case, 哈哈

ahmiao

2016-04-12 09:24:15 +08:00

统一接口＋子类化？

wanghaa

2016-04-12 09:31:49 +08:00

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

gqlxj1987

2016-04-12 09:34:01 +08:00

工厂+策略模式

zhuangzhuang1988

2016-04-12 09:40:53 +08:00

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

2016-04-12 09:41:52 +08:00

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

SpicyCat

2016-04-12 09:42:57 +08:00

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