在写 Java 程序的时候遇到了如下格式的逻辑: if (s instanceof FunctionDef) { ... } else if (s instanceof ClassDef) { ... } else if (s instanceof Return) { ... } ...
对于这种代码该如何重构才能使得程序看上去更加简洁并且高效?
在写 Java 程序的时候遇到了如下格式的逻辑: if (s instanceof FunctionDef) { ... } else if (s instanceof ClassDef) { ... } else if (s instanceof Return) { ... } ...
对于这种代码该如何重构才能使得程序看上去更加简洁并且高效?
|
knightdfApr 12, 2016
1
换成 Scala 用 match case, 哈哈
|
|
2
统一接口+子类化?
|
|
wanghaaApr 12, 2016
3
|
|
gqlxj1987Apr 12, 2016
4
工厂+策略模式
|
|
zhuangzhuang1988Apr 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; } } } } 试试下. |
|
beanlamApr 12, 2016
6
用多态,一个分支对应一个实现。这种方法对于有大量分支的情况可能不适用,因为要写大量的类。
另外一种模式可以参考《代码大全》第十八章提到的“表驱动模式”。 |
|
SpicyCatApr 12, 2016
7
@ahmiao 同意用接口加子类。 FunctionDef, ClassDef 和 Return 可以实现同一个接口或者继承同一个父类,假设父类叫 Father, 方法名叫 doSomething(), 然后那三个子类各自实现 doSomething(). 这样代码里只需要调用 Father#doSomething 就可以了。
|
|
8
改成一种更加复杂难懂的方式
|
|
cxshunApr 12, 2016
9
面向接口编程,多态就是这个时候用的。
|
|
fwrq41251Apr 12, 2016
10
访问者模式。
或者用 java8 ,有个叫 javaslang 的类库,有用于类型匹配的类。 |
|
SoloCompanyApr 12, 2016
11
首选是多态
不适合多态的可以试一下 Kotlin 用 case when 来代替了 if else 而已也没啥区别 |
|
wanttoflyApr 12, 2016
12
谢谢回答,最近正好有这方面的修改。
|
|
zhuangzhuang1988Apr 12, 2016
15
或者看这种解决方式, 当然有点难 `functional-programming-in-scala` 作者写的 http://blog.higher-order.com/blog/2009/08/21/structural-pattern-matching-in-java/
|
|
kanegApr 12, 2016
16
这不就是多态要解决的问题吗
|
|
twoyuanApr 12, 2016
17
我现在是建一个 Map<Class, Callback> map ,用的时候 map.get(object.class)。不过第一次创建时会慢点
|
|
JayleeApr 12, 2016
18
这应该是用多态吧
|
|
laibinApr 12, 2016
19
访问者模式。
|
|
haollyApr 12, 2016
20
重構裏面說過,當你看到大量 switch 或者 if-else 的時候,這表示你應該用多態了
|