参加代码审查的培训,也实践过一两次代码审查,感受仍是有用的,记录下。ide
代码审查的角色:开发
a,主持人:须要作会前的准备,让读者和评审者都提早看代码,预约会议室,主持会议,注意控制会议流程,尽量不发散,保证高效,作会后改进的监督。it
b,读者:须要会前作代码熟悉,不明白的和做者沟通,在会上逐行的读做者的代码,不作过多的评论。class
c,做者:负责给读者解释本身的代码,会上少解释,会后改进bug
d,评审者:会前大体了解做者的代码,会上在读者读代码的过程当中发表本身的见解。分享
代码审查的频次是一到两周,人数在三到四人为佳。di
代码审查不是批斗会,目的是分享提升,保证风格统一,规范代码,在开发阶段就减小bug风险。view