CHECKLIST的不实用之处

八月 15, 2008

CHECKLIST多是类似如下的东西，举一个代码CHECKLIST的例子：

- 参数的书写是否完整？不要贪图省事只写参数的类型而省略参数名字。
- 参数命名、顺序是否合理？
- 参数的个数是否太多？
- 是否使用类型和数目不确定的参数？
- 是否省略了函数返回值的类型？
- 函数名字与返回值类型在语义上是否冲突？

我们常常遇到的一个问题就是在进行source peer review的时候是根据每一个CHECK item去从头到尾看一遍代码(如果有50个CHECK items的话，那就从头到尾看50遍代码)还是记住所有CHECK items，然后只看一遍代码，显然我觉得后者在目前实施的可能性是最大的，也是实施最普遍的。

但是效果呢？估计还是看50遍代码较好，但是的确不太具备可操作性，投入的工作量太大，很多人也不会接受。

也有很多人采用折中的方式，比如说一共有10个人参与source peer review，每个人只关注其中的5项check item，然后一起walk through一遍代码。甚至在有些公司采取强制每个人必须能针对自己负责的check item提出问题，否则影响个人绩效之类的方法。

以上是看到公司的一个文档的CHECKLIST时突然想到的，没想出更好的solution。我想可能更多的人是不去记忆Checklist的，而是直接凭经验对代码评头论足的:)