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(...);
}

calculateMethod返回一个包含TypeA对象的列表，这对于method1（90％的调用）都没有问题，但对于method2则没有，因为我需要第二个包含TypeB对象的列表来进一步验证。

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

以参考方式致电 而不是返回一个值，我可以将列表放在方法参数中，并期望在calculateMethod完成后填充它。这实际上是我来自哪里以及我不想再去的地方。类似的东西：

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。这就是方法2的情况，我希望bTypes在calculateMethod之外做进一步的检查。

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个选项，具体取决于您手中的实施情况;

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

flattenRight

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

1 个答案: