真实记录一次对项目中代码的重构过程
这份记录来自对项目中下单接口重构，详细记录了每一步操作、以及运用到的一些方法。力求能够最大程度将当时的过程展现出来。
准备
重构中的每次修改都需要进行测试，用来验证修改是否正确，因此单元测试是一个非常好的选择。
单元测试
单元测试中可以进行mock操作，从烦人的token中摆脱出来，做到在任何人的IDE里面都能跑起来，从而任何人都能进行重构。当然前期可能工作量会稍微大一些。
接口处理逻辑
- 下单的入参校验。
- 乘客校验。
- 根据订单类型判断是否满足下单条件，例如是否已有在进行中的订单。
- 根据订单类型塞入想要的值到订单表相应字段。
- 其他根据订单类型进行的操作。
- 返回。
存在的问题
- 神秘的命名（Mysterious Name）
- 重复代码（Duplicated Code）
- 过长函数（Long Function）
- 重复的Switch（Repeated Switches）
主流程重构
抛开Controller里面的代码，直接从Service代码开始。
消除重复代码
最开头有一个判断用户是否存在的代码。首先，这个代码相信只要是与用户相关的接口，肯定是需要做这样的判断；再者，这是一个相对独立的功能，可以提成一个函数，用恰当的命名，可以让代码更容易理解。
创建新的函数，然后将带提炼的代码拷贝得到新的函数中，确保对其逻辑没有实质的改动。提炼之后的代码位置为：fbb4117030c143d86b0b7f58adff4ea910c39d15，效果如下：
运行单元测试，检测是否改出了新的问题。
消除变量的神秘命名
对变量、函数的命名是一门玄学，是个仁者见仁智者见智的问题。就像一千个人眼中有一千个哈姆雷特，每个人眼中正确的命名可能都有不同。
但是评判的标准都应该是：是否容易理解，是否能够一眼看出它的含义，诸如int a,b,c之类的肯定就不可取了。
JetBrains IDEA有很方便的重命名的方法，基本可以保证要改的地方都改了，不该改的不会动。如果实在不放心IDE的修改结果，可以用git diff逐项排查。
还有另外一处函数的局部变量也进行了重命名操作，如下：
重构之后的代码位置为：6fc2330d0ead0429bb8be683c847f40bc338683c，效果如下：
运行单元测试，检测是否改出了新的问题。一定要运行，虽然我也觉得改个名并不会有什么影响。
消除重复的switch语句
在检验当前用户是否可以下单的时候，发现里面有一堆if-else的判断，并且在后续的填充订单数据等也有正对订单类型的if-else判断逻辑。
首先要声明的是，在代码中并没有真实的switch语句，但是有嵌套的if-else语句，这个与switch语句在逻辑上是一样的。重复的switch语句做的判断是与订单类型相关，并且这样的判断出现了很多次。想象一下出现这种情况的场景：对某些类型的订单，做了特殊要求的时候，按照尿性，我们都会顺手写个if-else，如果两个if还不够，两个不够那就多来几个if就完事了，但是这样显然是不友好、不优雅的、不利于阅读代码。
用多态取代条件表达式。
- 新增了
OrderAction抽象类，包含一个待子类实现的verify()方法。 - 新增四种分别表示订单类型的类，继承自
OrderAction，实现verify()方法。
因此，对订单的校验变成了下面这种情况：
具体的校验策略均拷贝自原来的校验函数中，暂时不做逻辑上的修改。
重构之后的代码位置为：2329017ba0f420306eae4b443139785c05bc525b。新增实时拼车单、预约不拼车单的单元测试，运行单元测试，检测是否改出了新的问题。
重复运用提炼代码
对于刚消除了switch之后的代码，继续对其进行重构。梳理其业务逻辑如下：
- 查询是否有进行中的订单。
- 根据需求判断是否可继续下单。
首先从实时单下单前校验开始：
虽然这个if没有bug，但是总是感觉很难受，直接删掉了后面那个判断条件。接着看获取进行中订单列表的函数。
这里又进行了一次判断该用户是否存在的逻辑，然而在下单最开始的时候已经做了该判断，所以，这个if判断是不需要的。但是
调用的地方只有两处，并且另外一处就是预约单下单前校验的时候调用了该函数。虽然后面代码中，还用到了memberInfo的手机号码，去订单表里查询该用户的订单，但是订单表里的每一笔订单后面都加上了user_id（也就是memberId）。因此后面也不需要通过memberInfo去拿手机号码，所以这一次数据库查询完全可以去掉，可以节约一点接口处理时间。
在处理完上面的SQL查询后，对于这种局部变量，可以直接用IDEA的提示进行Inline variable操作，去掉不必要的局部变量。
所以重构后的代码如下：
另外，在Java中基本类型(primitive)优先于装箱基本类型(boxed primitive)。
对变量重命名、并删掉局部变量：
运行单元测试，检测是否改出了新的问题，发现单元测测试过不了。单元测试终于派上用场了，报错是因为StringUtils.join()得到的字符串并不符合SQL语法，如下：
修改成base-support下面的StringUtil.join2SqlInStr即可修改好改问题。
可见，单元测试在保证每步重构的正确性上面，是有一定的帮助的。
实时单下单前校验重构后，代码的位置为：4dcd17826ddd67f711151e28d569970e00ff0327。
预约单下单前校验与实时单类似，这里不再展示。
重构数据填充
先去掉重复的switch
将原有的dealWithByOrderType()函数放到OrderAction的fillOrderData()中，
并在该类中新建fillOrderDetail()抽象函数，用来分别处理不同订单类型的数据填充方式，在4个子类中分别实现该函数，将相应的填充代码拷贝进相应的函数中，先不做逻辑修改，暂时先对所需的函数、变量进行调整，让它能正常运行即可。
运行单元测试，检测是否改出了新的问题。代码位置为：b0fc69718184ffa543a57df0210e2086225172dd。
对各子类中的fillOrderDetail()进行重构
后续发现，在填充数据的函数里面，充斥了大量的校验，这个时候函数做的事情与函数名称是不相符合的，这时候就应该使用提炼函数方法，将不同的工作提到另外一个函数中。因为是拷贝过来的，所以这就是之前代码中的问题。
但是考虑到后面的流程中还有一个填充数据的流程，所以对这部分的重构，等将整体流程重构完成之后，再一起重构。
发送消息
完成初步填充数据后，后面还有一个填充数据的流程，这个if-else里面都有unifiedCreateOrderProcess()这个函数，所以可以直接提出来，放到OrderAction里面。这个if-else实质上只做了一件事情，那就是根据不同的订单发送不同的消息。
这个if-else真的是有点辣眼睛啊，最起码也得放到orderList初始化那一行。但是下单接口改的人确实是多，也可以理解。
在OrderAction里面，新建一个发送消息的方法，并将上面if中的代码拷贝进去。根据原代码，得出的结论是只有孩子单没有发送消息的必要，所以只需要在表示孩子单的那个子类中覆盖这个方法，然后函数里面什么都不做，就可以了。但是这里好像把预约单忘了，不过也没关系，目前并没有与预约单相关的业务。
所以修改后的代码如下：
下单流程代码：
OrderAction代码：
其函数中的具体重构事项，暂时不管，留着后续再次做重构。此次重构后的代码位置为：30b45bdef6dd69f1737b7d6bde747030a69da5cd。
运行单元测试，检测是否改出了新的问题。
存储上下车站点位置
存储上下车站点位置的这段代码看着好像没有啥问题，理解起来也没什么问题，但是始终还是觉得有点辣眼睛。
因为这个needRecordSites()函数还是依据订单类型来做判断，返回是true或者false。
这里可以考虑让每个子类返回一个参数，表示是否需要进行上下车站点的记录。形式可以是将主要的代码移到OrderAction中，构建一个模板方法；或者直接用运行时的多态即可。这里我选择了后面这种，主代码不移走，仍然放在下单的主流程里面。
删掉之前的neeadRecordSites(short orderType)，运行单元测试，检测是否改出了新的问题。代码位置为：2f05795039824d0ddd1d4cfd912069a8bbdf4c79。
主流程扫尾重构
①变量命名：OrderIds。改名。
②这段小代码里面明显存在公用的代码。去掉公用的代码。
③是网约车平台监控推送代码提炼成一个函数。
④去掉orderIds这个变量。
当前代码位置为：5b6bac8da。运行单元测试，检测是否改出了新的问题。
各函数重构
目前的主流程中如下：
①有一个非常刺眼的orderType在其中
②并且记录上下车站点的函数名也很是奇怪
③然后就是校验这一块可以细分成入参校验+用户状态是否可下单校验。因为多种类型的订单，公用一个接口进行下单，在Form表单里面不能一概而论的使用框架提供的参数校验。
④在createOrder()中调用insertTravellingOrder()函数看起来很多余，因为前者只是做一个转发作用，这是可以将后者内联回前者中。
修改后的主流程代码如下，整体比较清楚明白，但是仍然有点瑕疵，那就是fillOrderData()与unifiedCreateOrderProcess()这个函数是不是可以合并？这个可以待后续观察。
当前代码位置为：5b6bac8da。运行单元测试，检测是否改出了新的问题。
当前的下单逻辑已经分成几块明显的模块，其中verifyMemberStatus()是由之前的verify()改名而来，之前已做过重构。接下来就对fillOrderData()模块进行重构。
fillOrderData()模块
这部分的修改主要集中在孩子单的处理上，孩子单的填写参数模块中夹杂着校验参数和填写订单信息、校验乘客状态着三部分的代码，使得每个函数的分工不再明确，因此需要将相应的逻辑，移到相应的函数中去。
在调用这个方法之前有一个查询用户是否存在的函数，如果用户存在，但是在这里参数校验过不了，那么在查询用户是否存在的代码中进行的SQL查询就白费了。因此将参数判断放在最开始的地方是有好处的。
当前代码位置为：698cb047d。运行单元测试，检测是否改出了新的问题。
unifiedCreateOrderProcess()
①这个函数参数很奇怪。
这个appointmentTimeList到底来自哪里？
到头来还是来自orderCreateForm，可是这个参数已经被传进来了。所以这个appointmentTimeList可以直接被干掉。
②这个函数里面大部分是执行set操作。只有后面有一个对孩子单生成多日订单的代码，不然整个代码使用于所有的订单类型。因此只需要将公共的代码放入fillOrderDataCommon()中即可，生成订单列表的部分，可以交给不同的子类来实现。
当前的代码位置：f30ec83f7a427bd299b40ed7a0cfe4816faef15d。运行单元测试，检测是否改出了新的问题。
发送消息
在发送消息模块，有一个VIP通道，意思大致是：后端自己去找合适的车辆，找到了就不用传给算法进行计算，没找到再传给算法计算。这段代码可以提炼。是否需要发送给算法，可以交给子类。
当前的代码位置：3ea9c14331d260a6de8fd7966ec8b84af43744bf。运行单元测试，检测是否改出了新的问题。
到目前为止，下单接口的代码已经重构完成了。可以等发布开发环境后，再次用司机端、客户端再次进行测试。
后记
最终还是翻车了，出错的代码在孩子单生成多日订单的时候。
这个是因为for循环里面的order原来也叫orderCreateForm，然后函数的入参变量名也被我改成了orderCreateForm，所以这两个一开始的没开始搬家之前，就已经相同了，然后通过IDEA的Refactor-rename修改时，两个都被改成了一样的名称，但是主流程还是没有报错。
真实记录一次对项目中代码的重构过程
