我正在开发一个遗留的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对象中读取状态,并根据相应的类型创建其他类的实例,存储在数据库中等等 – 从我对回答中的问题回答我认为这很重要.
