重构非常组合的代码

Question

我现在必须重构一些代码,它基本上是一个方法(大约1000行代码),它做了很多计算并且有很多变量.我不确定如何重构它.代码是否像

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

看起来不错？

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

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

或者我可以将所有东西放在一个大物体上并制作它

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

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

你会怎么做？

PS计算并不简单,所以做的事情就像

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

只会让人难以阅读.

Answer 1

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