• 2008-11-10

    代码看多了

    版权声明:转载时请以超链接形式标明文章原始出处和作者信息及本声明
    http://www.blogbus.com/dreamhead-logs/31212384.html

    Clean Code
    Implementation Patterns

    按照惯例,空闲的状态下,负责招聘的同事就会找上门来。在这个校园招聘如火如荼进行的时段,他们更是要充分利用“beach”上的每一个人。去年,我荣膺了公司的“Code Review Machine”,所以,当之无愧就成了他们找人的首选。这不,今天一口气做了14个Code Review。

    我们的招聘环节中,有一个环节,就是让应聘开发角色的人去写代码,然后,我们会根据内部的标准对代码进行评审,这就是Code Review的由来。有兴趣的人,不妨搜搜,我知道,我们公司最近用的这几道题在网上都是可以找到的。其实,仔细一读便不难发现,这几道题并不是那种难到让人摸不到头脑的题。按照我当初应聘时,冰云给我的说法,两个小时就可以完全搞定。

    或许你会疑问,如果试题不难,那如何区分应聘者的编码能力呢?事实上,仅仅完成功能是不够的。我们更多的是在看代码中展现出来的内容。

    当然,我无意在这里讨论公司Code Review的标准。

    之所以会想起写这个话题,完全是今天的代码看多了,好吧,找一些精采的部份分享在这里:
    * 一个类的名字叫做A(我不记得我们的题里面有这样的概念……)
    * 一个C#/Java程序只有一个文件(我们的题没有简单到只有一个概念吧?)
    * 所有的方法都是static的(确定用的是OO语言?)
    * 长长的方法(虽然鼠标有滚轮,看着也麻烦)
    * 太多的switch..case(很勤劳,写那么多一样的switch都不嫌烦)
    * 嵌套许多层(看来是用有自动格式化的工具了,要不然,打tab或是空格,自己就会觉得麻烦)
    ……

    我曾经和同事开玩笑,我应该写一篇《ThoughtWorks应聘代码过关指南》,争取让自己在今后的Code Review过程中稍为舒服一些。

    其实,ThoughtWorks所遵循的评审标准完全是业界公认的一些标准。所以,真正的《过关指南》已经有了,比如Martin Fowler用《重构》告诉我们什么是不好的代码,比如这篇blog开头列着的那两本书告诉你怎么写好代码。

    如果这算泄密的话,那么就让我去评审更多的好代码,累累自己。
    分享到:

    历史上的今天:

    探索Antlr 2005-11-10
    引用地址:

    评论

  • 只想小小说一句,实现模式的中文版已经快要上市了,China-pub上有预订
  • 呵呵,现在看到也不算晚,向博主学习
  • hi,风暴,你好。
    看你的blog,已经有近一年了。第一次给你留言。
    从你的blog开始喜欢上tw。
    我参加了tw2009校园招聘的考试,也参加了code review环节。
    16号的时候已经提交了,不知道是不是有运气让你来review我的代码,我自己也常常review别人的代码,自认为代码风格较好。
    能知道你的email 地址不? 希望您review一下我的代码,谢谢。
  • 辛苦啊,以后我多注意哈~
    回复眼影说:
    不是你的问题,是代码的问题。
    2008-11-15 09:54:58