o6z说code review

[quote="ozzzzzz在agilechina上"]下面是我关于code review的论述。

[b]code review其实就是发现code里面的疑问，并标注下来。[/b]

code review的三个原则：[b]尽快；随时随地随手；非正式更有效。[/b]虽未（所谓）尽快就是要趁代码还是热的时候就code review。所谓随时随地随
手，就是只要有时间，有地方，有精力就code review，并且应该随手就做；就如同你扫完地随手就把笤帚放回原位一样。所谓非正式更有效，就是要
多注重平时这些随处存在的非正式的code review，而绝不是依靠把代码积攒起来，大家板起面孔，正襟危坐的code review会议。

具体的几个方法和注意事项：
1、写完就code review。
2、写的时候也code review。[b]其实很多时候我们写代码的时候就有写不好的感觉，有这个感觉就标记下来。代码注释就是用来做这个的，而不是用了（来）
解释代码的。我经常上（说）好代码没有注释，也有这个意思。[/b]
3、问别人问他（题）的时候，就code review。
4、商量做接口的时候，也code review。
5、每日必在下班前作一个全天的code review。有些组织坚持每日建造，可以结合起来。
6、争论最终的解决依靠测试更有效。
7、code review关键在于标注问题，而不是对问题分类，更不是对问题划分优先级。这些是code review的后续工作，当然这些工作也必
须有专门的方法和约定进行解决。
8、code review要不预设范围，不要说我要找一个多线程的错误，我要找一个代码命名错误。
9、鼓励互相code review，但是以此和由此得出的成果作惩罚和奖励的依据。
10、定期对code review工作进行review是必要的。
11、注重工具的应用。比如注释的改变，并不会造成一次新的构建，具备这样功能的构建和集成工具是必要的。
12、注重 code review的实际效果，[b]避免落入形式主义。[/b]其实在国内我发现，很多组织的code review会议都是在够过场，当然有些组
织更过分成了纯粹的扯淡会议，更加有一些堕落到一群人攻击另外一群人的手段。[/quote]
oz老大的错别字老多了……