我现在必须重构一些代码,它基本上是一个方法(大约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())
只会让人难以阅读.
我会花费任何必要的时间来确保我理解算法实际上做了什么.然后,我试图找出如何我真的解决这个问题,如果我可以选择,其中可能会涉及到一些类和概念.然后我尝试将这些概念逐一引入现有代码中,确保在我介绍它时为每个概念获得适当的测试覆盖率.