[原创]CodeReview的一些想法

CodeReview的一些想法
    昨天的codereview后，想到了一些问题：

1. 发codereview之前
1. 注释。注释会告诉同事代码想做什么，在做什么。好的注释会让同事更容易地理解为什么那样做。尤其是同事不知道的一些细节问题，好的注释将会节省codereview的时间，也会节省产品的时间。
2. 自我检查。发codereview之前应该全面检查代码的改动及其影响。实现预期功能或fix相关bug是重要的，对其他部分的影响也同样重要，尤其是bug的shcedule非常紧张的时候，很多时候这部分可能会被降低优先级。这个是不对的。
3. 好的设计。好的设计任何时候都重要，fix bug的时候也不例外。不要仅仅关注bug本身，同样需要跳出这个bug分析，设计。
2. 收到codereview的回复之后
1. 收到疑问：可能因为注释或者文档的不清楚，可能因为同事对问题本身的不了解，也可能是设计或者实现本身是有问题的。认真分析原因后，有的放矢。
2. 反对意见：对反对意见首先要在心理上持open态度，这个非常重要。然后认真地去思考是什么原因。可能是思考的角度不同，可能是自已或同事忽视了某点，也可能是自己根本没有意识到的问题(昨天的bug就是这样的问题，尽管是以另一种方式fix bug,但并没有从现象追溯到真正的根源，仅仅是医表而已。印度同事的问题很好：“我知道这样fix可以工作，但我想知道为什么我们需要这样的fix”，跳出bug本身去分析，尤其是跳出bug描述，因为很多时候现象是片面的。)认真，客观地对待反对意见，如果这个意见不合理，耐心地解释，同时要注意解释的方法，要有效而且有力;如果合理，应该虚心接受并且改正，同时从中吸取经验和教训。