学习编程TDD(测试驱动开发)

Kata07:给你一段过去的代码,看看那时我们的幼稚

2014-12-04  本文已影响168人  梁杰_numbbbbb

Kata07地址

一般我是晚上做Kata,第二天早晨写文章记录。不过今天这个Kata比较简单,我又比较闲,做完干脆就把文章写了。

经历过一段(10行的)算法训练之后,我们又可以偷懒——哦不是,我们又可以进行思维训练了。

这次的任务很简单,找到一段自己一年前写的代码,几百行左右,然后阅读三遍,每遍角度不同。

我之前的代码在GitHub上,所以直接找了一段看,那个项目的名称叫worldline,感兴趣的可以去看看,虽然我觉得肯定没人看,哈哈。

那个项目大概是两年前写的,当时用了canvas,工作量主要在前端。印象中实现功能的时候遇到了很多坑,比如jQuery的animation并发的时候必须手动stop之前的动作什么的。说起来似乎也算不上是坑,只是自己不理解jQuery罢了,不过当时很是痛苦了一段时间。

功能实现之后代码烂到自己也看不下去了,就认真地重构了一下——当然,是基于我两年前的实力进行重构——当时自己还感觉挺叼的,提取了好多公共函数,重构之后大概减少了40%代码,一直以此为荣,虽然从来没告诉过别人。

这次打开这段代码一看,果然,就像看自己一年前的QQ空间一样,图样图森破。当然,幸运的是比两年前的QQ空间好点,两年前就不是幼稚了,是SB,恨不得穿越回去抽丫几嘴巴子。

具体代码我就不贴了,直接说结论:

写的好的地方主要是提取了公共函数,用canvas画图时候方便了许多

写的烂的地方主要有三点:

  1. 代码没有格式化。当时的我可能还不知道有自动格式化插件,所以代码乱七八糟的。
  2. 抽象程度不够。只是提取公共函数,其实代码仍然很乱,应该用类或者原型继承方式重构的。这次切实体会到了类的重要性,类看起来比较有调理,从init开始一步一步深入。函数的话只能一大片代码慢慢往下看,很难建立整体的逻辑关系。
  3. 细节做得不好。具体来说就是命名不规范,很多函数直接就叫a、b,还有就是赋值不恰当,应该加var的地方没有加,可能会污染全局空间。

这个练习就像“好好学习天天向上”一样,说起来简单做起来难。做起来难的原因很简单——自以为是,觉得没必要做,其实错过了许多提高的机会。

以后有机会的话要经常温故而知新,最后把CodeKata作者的一句话送给大家:

Moving Forward By Looking Back

上一篇 下一篇

猜你喜欢

热点阅读