美文网首页学习编程TDD(测试驱动开发)
Kata07:给你一段过去的代码,看看那时我们的幼稚

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

作者: 梁杰_numbbbbb | 来源:发表于2014-12-04 20:13 被阅读168次

    Kata07地址

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

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

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

    • 第一遍把代码作者看作大牛,找他写的好的地方
    • 第二遍把代码作者看作新手,找他写的烂的地方
    • 第三遍把代码作者看作二逼,找他出BUG的地方

    我之前的代码在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

    相关文章

      网友评论

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

      本文链接:https://www.haomeiwen.com/subject/pkngtttx.html