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

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 对象的列表来进行进一步验证。

我想出了以下解决方案，但我遇到了一些问题。

引用调用 而不是返回一个值，我可以将列表放入方法参数中，并期望在计算方法完成后填充它。这实际上是我来自的地方，也是我不想再去的地方。比如：

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 是否有重构模式，以便我可以重构一个使用引用调用的方法来处理多个返回类型？

编辑 第二个列表包含布尔值，用于验证列表一（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;
}