java - 具有不同返回值的一种方法的重构模式

Question

我必须重构一个方法，并遇到一个问题，即该方法被调用时对返回值的期望不同。这基本上就是我现在所拥有的(从真实的方法和类名中抽象出来)。

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;
} 

Answer 1

这是一个设计问题，当你必须从一个方法返回 2 个对象时，这意味着该方法做了比它应该做的更多的事情。根据您手中已有的实现，您可能有 2 个选择；

拆分方法只做一件有意义的事情，或者将列表封装在一个对象中，将其填充到方法中并返回。

public class Types {
    List<TypeA> typeAs;
    List<TypeB> typeBs;
}