一般我是晚上做Kata,第二天早晨写文章记录。不过今天这个Kata比较简单,我又比较闲,做完干脆就把文章写了。
经历过一段(10行的)算法训练之后,我们又可以偷懒——哦不是,我们又可以进行思维训练了。
这次的任务很简单,找到一段自己一年前写的代码,几百行左右,然后阅读三遍,每遍角度不同。
- 第一遍把代码作者看作大牛,找他写的好的地方
- 第二遍把代码作者看作新手,找他写的烂的地方
- 第三遍把代码作者看作二逼,找他出BUG的地方
我之前的代码在GitHub上,所以直接找了一段看,那个项目的名称叫worldline,感兴趣的可以去看看,虽然我觉得肯定没人看,哈哈。
那个项目大概是两年前写的,当时用了canvas,工作量主要在前端。印象中实现功能的时候遇到了很多坑,比如jQuery的animation并发的时候必须手动stop之前的动作什么的。说起来似乎也算不上是坑,只是自己不理解jQuery罢了,不过当时很是痛苦了一段时间。
功能实现之后代码烂到自己也看不下去了,就认真地重构了一下——当然,是基于我两年前的实力进行重构——当时自己还感觉挺叼的,提取了好多公共函数,重构之后大概减少了40%代码,一直以此为荣,虽然从来没告诉过别人。
这次打开这段代码一看,果然,就像看自己一年前的QQ空间一样,图样图森破。当然,幸运的是比两年前的QQ空间好点,两年前就不是幼稚了,是SB,恨不得穿越回去抽丫几嘴巴子。
具体代码我就不贴了,直接说结论:
写的好的地方主要是提取了公共函数,用canvas画图时候方便了许多
写的烂的地方主要有三点:
- 代码没有格式化。当时的我可能还不知道有自动格式化插件,所以代码乱七八糟的。
- 抽象程度不够。只是提取公共函数,其实代码仍然很乱,应该用类或者原型继承方式重构的。这次切实体会到了类的重要性,类看起来比较有调理,从init开始一步一步深入。函数的话只能一大片代码慢慢往下看,很难建立整体的逻辑关系。
- 细节做得不好。具体来说就是命名不规范,很多函数直接就叫a、b,还有就是赋值不恰当,应该加var的地方没有加,可能会污染全局空间。
这个练习就像“好好学习天天向上”一样,说起来简单做起来难。做起来难的原因很简单——自以为是,觉得没必要做,其实错过了许多提高的机会。
以后有机会的话要经常温故而知新,最后把CodeKata作者的一句话送给大家:
Moving Forward By Looking Back
网友评论
http://www.jianshu.com/p/70e7ca584ff3