This slide contains a number of points should be noted that in the refactoring

1. Refactoring
dreampuf

2. Insight

5. Compare

6. 400
UbbParser
nutshell/gksite/views_index
320 Text2pic
Django:core/cache/backend/memcache
240
160
80
0
SLOC

7. 8 UbbParser
nutshell/gksite/views_index
Text2pic
Django:core/cache/backend/memcache
6
4
2
0
maxBlockDepth

8. 300
UbbParser
nutshell/gksite/views_index
240 Text2pic
Django:core/cache/backend/memcache
180
120
60
0
numBlocks

9. 50
UbbParser
nutshell/gksite/views_index
40 Text2pic
Django:core/cache/backend/memcache
30
20
10
0
DocString(%)

10. 一种修改，使软件的内部结构更容易理解，在不改
变软件的可见行为方式前提下使软件更容易变更…
它是一种有节制的整理代码、使bug产生几率最小
化的方法。
- Martin Fowler & Kent Beck

11. 设计原则

12. 设计原则
OCP open-closed principle
SRP single responsibility principle
ISP interface segregation principle
LSP Liskov substitution principle
DIP dependency inversion principle

13. 坏味道

14. 坏味道
重复
过长函数
过长的参数列表
散弹式修改
耦合
为未来着想太多
低内聚

15. 测试

16. 测试
测试自动化
频繁运行保值
为每一个Bug创建一个UT
集中火力对付边界条件
UT是昂贵的成本
为测试而设计

17. 技巧

18. 技巧
小步重构
同时只做一项重构
有TODO list
UT

19. 20%
其他
80%

20. 文档化
好样式，简单代码
有意义的名称
分解为原子函数
描述性类型
强调重要的代码
提供文件头
恰当的错误处理
有意义的注释

22. 沟通

23. Reference

24. Thanks!