重构非常组合的代码

Question

我现在必须重构一些代码，它基本上是一种方法（大约1000行代码），它进行了大量的计算并且有很多变量。我不确定如何重构它。 代码是否像

...
calculateSth(param1,param2,param3,param4,param5, params6);
calculateSthElse(param1,param2,param3);
...

看起来不错？

我可以引入参数对象，但这些对象只能用作某些方法的参数，所以它看起来像这样

...
calculateSth(calculateSthObject);
calculateSthElse(calculateSthElseObject);
...

或者我可以将所有东西都放在一个大对象中并使其成为

...
calculateSth(calculateObject);
calculateSthElse(calculateObject);
...

但是在该解决方案中，我必须在方法的开头提取私有方法中所需的所有内容并在最后设置，并且要找出私有方法中使用的值会更加困难。 。需要大约一半的变量作为输出。

你会怎么做？

P.S。计算并不简单，所以做了像

这样的事情

calculateObject.setMagicValue4((calculateObject.getMagicValue() * calculateObject.getMagicValue2() / calculateObject.getMagicValue3()) 

只会让你难以阅读。

Answer 1

我会花费任何必要的时间来确保我理解算法实际上做了什么。然后，我会试着找出如果我可以选择，我如何 解决问题，这可能涉及到许多课程和概念。然后我尝试将这些概念逐一介绍到现有代码中，确保在我介绍它时为每个概念获得适当的测试覆盖率。