在领导的极力推动下,最近部门开始大力进行代码评审。
有幸成为部门代码评审委员会一员,负责评审了许多同事的代码。在评审他人代码的同时,也对我自己是一个极大的进步。
现在coding的感觉,已经和半年前有了极大的差别。
然后评审委员会选出我的一小段代码作为“模范代码”让公司的首席科学家“老天”评审,通过其细致的评审,甚至让我对这段我曾经认为还不错的代码生出了愧疚之情,看来代码真的没有最好,只有更好。
最近对我的代码评审,总结为以下几点建议:
1。抛出异常需果断,绝不能对调用者手软
2。线程锁的关系需更仔细的设计
3。返回值不给不必要的bool
4。应该更加多应用语言特性,比如c#的using
5。异常处理机制需更仔细的设计
6。类成员实例尽量不要暴露,如需要,给出拷贝
7。注释给的不够精炼
附一段我的代码,C#写的(一个树状任务系统的Manager部分)