我正在开发一个遗留的Java应用程序,它可以解决“水果”和“蔬菜”问题.
它们在内部被视为不同的东西,因为它们没有共同的所有方法/属性,但是很多东西都完全类似于它们.
所以,我们有很多方法doSomethingWithAFruit(Fruit f)和doSomethingWithAVegetable(Veg v),即使用正确的doOtherStuffWithAFruit(Fruit f)/ doOtherStuffWithAVeg(Veg v).这些非常相似,只不过用水果做事的方法只能用水果做方法,对蔬菜做同样的事情.
我想重构这个以减少重复,但我不确定实现这一目标的最佳方法是什么.我已经阅读了一些关于某些设计模式的内容,但我不知道它是否让我更清楚. (我可以识别我使用的代码中的一些模式,但我真的不知道何时应该使用模式来改进周围的事情.也许我应该阅读更多有关重构自身的内容……)
我在考虑这两个选项：
1.创建一个可以具有Fruit或Vegetable实例的类,并将其传递给方法,尝试最小化重复.它会是这样的：
public void doSomething(Plant p) {
// do the stuff that is common,and then...
if (p.hasFruit()) {
doThingWithFruit(p.getFruit());
} else {
doThingWithVegetable(p.getVegetable());
}
}
这会让事情变得更好,但我不知道……它仍然感觉不对.
我想到的另一种选择是在水果和蔬菜中加入一个与它们相同的东西,然后用它来传递它.我觉得这是更清洁的方法,虽然我需要使用instanceof并在需要特定于他们的东西时施放到Fruit / Vegetable.
那么,我还能在这做什么呢？这些方法的缺点是什么？
更新：请注意,问题有点简化,我正在寻找使用“植物”做事的方法,即主要“使用”它们而不是对它们做事的代码.话虽如此,我所指的那些类似方法不能在“Plants”类中,并且它们通常有另一个参数,如：
public void createSomethingUsingFruit(Something s,Fruit f);
public void createSomethingUsingVegetable(Something s,Vegetable v);
也就是说,除了水果/蔬菜之外,这些方法还有其他问题,并且不适用于任何水果/蔬菜类.
更新2：这些方法中的大多数代码只从Fruit / Vegetable对象中读取状态,并根据相应的类型创建其他类的实例,存储在数据库中等等 – 从我对回答中的问题回答我认为这很重要.
例如,如果您的消费者被称为Eat,您可以执行以下操作：
Consumer e = new Eat();
Consumer w = new Water();
if( e.canProcess( myFruit ) )
e.doSomethingWith( myFruit );
else if ( w.canProcess( myFruit ) )
w.doSomethingWith( myFruit );
.... etc
但是你最终得到了很多/其他类,所以你自己创建一个工厂来确定你想要的消费者.您的工厂基本上执行if / else分支以确定哪个消费者可以处理您传递的对象,并将消费者返回给您.
所以它看起来像
public class Factory {
public static Consumer getConsumer( Object o ){
Consumer e = new Eat();
Consumer w = new Water();
if( e.canProcess( o ) )
return e;
else if ( w.canProcess( o ) )
return w;
}
}
然后你的代码变成：
Consumer c = Factory.getConsumer( myFruit );
c.doSomethingWith( myFruit );
当然,在消费者的canProcess方法中,它基本上是一个instanceof或一些其他函数,你可以确定它是否可以处理你的类.
public class Eat implements Consumer{
public boolean canProcess(Object o ){
return o instanceof Fruit;
}
}
因此,您最终将责任从类转移到Factory类,以确定可以处理哪些对象.当然,诀窍在于所有消费者都必须实现通用接口.
我意识到我的伪代码是非常基本的,但它只是指出了一般的想法.根据您的类的结构,这可能会或可能不会在您的情况下工作和/或变得过度,但如果设计得很好,可以显着提高您的代码的可读性,并真正保持每个类型的所有逻辑自包含在他们自己的类中没有instanceof,if / then散布在各处.
