CHECKLIST的不实用之处
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的,而是直接凭经验对代码评头论足的:)
评论