重构:改善既有代码的设计
输入“/”快速插入
重构:改善既有代码的设计
重构是什么以及为什么?
1.
重构是在不改变软件可观测行为的前提下，调整代码结构，提高软件的可理解性，降低变更成本。
2.
重构是一种经济适用行为，而非道德使然，如果它不能让我们更快更好的开发，那么它是毫无意义。
3.
代码的写法应该使别人理解它所需要的时间最小化，进而变更代码需要的时间也会最小化。
4.
重构对个体程序员的意义是提高ROI。
a.
更快速的定位问题，节省调试时间。
b.
最小化变更风险，提高代码质量，减少修复事故的时间。
c.
得到程序员同行的认可，更好的发展机会。
5.
重构对整个研发团队的意义是战斗力的提升。
重构的原则
1.
重构的目标: 提高迭代效率。
2.
获得同行认可的方法: 每一次提交代码，都应该使代码变得更好，先重构，再开发。
3.
增量式重构 = 自动化测试+持续集成+TDD驱动重构。
代码/架构的坏味道
💡
所谓优雅的代码，就是易于理解，可扩展性良好。
依赖传递
交互产生依赖，多模块之间的影响被交叉放大。
变更放大
修改点被交叉放大，一次迭代需要修改N个位置，每一处修改点都可能带来调试，编译，构建，测试，上线等环节，致使效率低下，操作变得复杂，容易遗漏或失误。
func packVideo(longVideo, sortVideo)*videoInfo{
....
// logic
....
}
关注放大
关注点被交叉放大，为完成一次迭代任务，需要通读修改点附近上下文的代码，而由于依赖被传递，附近的代码会牵扯出更多需要关注的代码，通常为弄清楚修改点的影响范围，需要理解超出此次迭代任务10倍以上的代码量，需要关注的上下文过多，造成心智负担，难以测试，调试，上线极易出错
最终，演变成未知的未知(变更后不知道是否会出问题的状态)
// query: 1231232|23423|sdfs|342|dsfsdf|sdfsdfd
func ParseData(dataArray []string) string {
....
if len(dataArray) >= 10 {
arrLen := len(dataArray)
deviceID, _ := strconv.ParseInt(dataArray[5], 10, 64)
userID, _ := strconv.ParseInt(dataArray[arrLen-6], 10, 64)
itemID, _ := strconv.ParseInt(dataArray[arrLen-5], 10, 64)
xxxID, _ := strconv.ParseInt(dataArray[arrLen-4], 10, 64)
xxxID := dataArray[arrLen-3]
from := dataArray[arrLen-2]
if len(dataArray) >= 18 {
a, _ := strconv.ParseInt(dataArray[arrLen-7], 10, 64)
b = groupIdTmp
c, _ := strconv.ParseInt(dataArray[arrLen-6], 10, 64)
from = dataArray[arrLen-3]
}
}
....
}