我必须重构一个方法,并遇到一个问题,即该方法被调用时对返回值的期望不同。这基本上就是我现在所拥有的(从真实的方法和类名中抽象出来)。
public static List<TypeA> calculationMethod(...)
{
List<TypeA> aTypes = new ArrayList<TypeA>;
//this one is used to calculate some stuff inside this method
List<TypeB> bTypes = new ArrayList<TypeB>;
// generation of objects of TypeA and TypeB and more
// use bTypes to check if aTypes is fine
return aTypes;
}
public void method1()
{
List<TypeA> aTypes = calculationMethod(...)
//do other stuff with aTypes
}
public void method2()
{
List<TypeA> aTypes;
List<TypeB> bTypes;
// here is the problem, i need to get both lists from calculationMethod
aTypes = calculationMethod(...);
}
calculationMethod 返回一个包含 TypeA 对象的列表,这对于方法 1(我的 90% 的调用)来说很好,但不适用于方法 2,因为我需要第二个包含 TypeB 对象的列表来进行进一步验证。
我提出了以下解决方案,但我遇到了一些问题。
通过引用调用 我可以不返回值,而是将列表放入方法参数中,并期望在calculationMethod完成后填充它。这实际上是我来自的地方,也是我不想再去的地方。像这样的东西:
public static void calculationMethod(List<TypeA> aTypes, List<TypeB> bTypes, ...)
{
aTypes.add(...);
bTypes.add(...);
}
切换计算方法的返回类型 我可以返回一个包含两个列表的元组,而不是返回一个列表。但由于在大多数情况下我只需要其中一个列表,这似乎削弱了我的方法契约和代码的可读性。
public static Tuple<List<TypeA>, List<TypeB>> calculationMethod(...)
{
// do stuff
return new Tuple<>(aTypes,bTypes);
}
分割方法 我还可以拆分该方法(并重命名它们),这样我就有两种方法,并且可以调用我目前需要的方法。但这会产生重复的代码,至少在某种程度上是这样,因为在calculationMethod中使用的代码不能那么容易地放入另一个方法中。
public static List<TypeA> calculationMethod1(...)
{
//code
return aTypes;
}
public static Tuple<List<TypeA>, List<TypeB>> calculationMethod2(...)
{
// same code
return new Tuple<>(aTypes,bTypes);
}
TL;DR 是否存在重构模式,以便我可以重构一个方法,该方法使用按引用调用其参数来处理多种返回类型?
编辑 第二个列表包含 boolean 值,用于验证列表一(TypeA)的对象的初始化。通常,如果我知道 bTypes 仅包含 true 就足够了。但有一种情况我希望 bTypes 包含 false。这是 method2 的情况,我希望 bTypes 在 CalculationMethod 之外进行进一步检查。
public static List<TypeA> calculationMethod(...)
{
List<TypeA> aTypes = new ArrayList<TypeA>;
List<Boolean> bTypes = new ArrayList<TypeB>;
TypeA typeA = new TypeA(...);
bTypes.add(validateInit(typeA));
Boolean initOK = true;
for(Boolean b: bTypes)
{
if(!b)
initOK = false;
}
if(initOK)
{
return aTypes;
}else
return null;
}
最佳答案
这是一个设计问题,当你必须从一个方法返回 2 个对象时,这意味着该方法做了比它应该做的更多的事情。根据您手中已有的实现,您可能有 2 个选择;
拆分方法只做一件有意义的事情,或者将列表封装在一个对象中,将其填充到方法中并返回。
public class Types {
List<TypeA> typeAs;
List<TypeB> typeBs;
}
关于java - 具有不同返回值的一种方法的重构模式,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/41629915/