开源中国
2019-09-09 00:00:00 发布
【谷歌开源代码评审规范：好坏代码应该这样来判断】网页链接谷歌开源了一套代码评审（Code Review）规范，它是谷歌一套通用的工程实战指南，几乎涵盖了所有编程语言与各种类型的项目，这个规范代表了谷歌长期发展以来最佳实战经验的集合，谷歌表示希望开源项目或其他组织能够从这套规范中受益。根据谷歌的项目描述，代码审核规范为两套独立文档组成，代表了两方面内容的最佳实践：1.代码评审者的指南——如何进行代码评审2.更改列表作者指南——更改列表作者批准代码的评审指南在谷歌看来，代码审核的目的是确保谷歌代码库的整体代码健康程度。谷歌将以下规则作为代码评审的标准：一般来说，一旦 CL 能提升整体代码的健康程度，那么即使 CL 不完善，评审者同样也应该倾向于批准该列表。这是所有代码评审指南中的高级原则。它也会有一些限制，例如，如果 CL 添加了一些评审者不需要的特性，那么即使代码做了不错的设计，评审者也应该不予通过。没有所谓的“完美”代码，只有更好的代码。评审人员不应要求作者在批准前对 CL 的每一小部分过分完美。相反，评审者应该权衡向前继续开发的需求和修改建议的重要性。评审者要求的是持续性地改进，而不是追求完美的代码。CL 作为一个整体，如果它能提升系统的可维护性、可读性和可理解性，那么就不要因为它还不完美而推迟数天或数周更新。评审者应该经常留下一些评论，以表达能导致更好性能的做法。如果这些做法并不是非常重要的，那么需要加上前缀「Nit:」，从而令代码作者知道这些内容是可以忽略的。深入了解这套代码规范，请见网页链接
2019-09-09 00:00:00 发布
【谷歌开源代码评审规范：好坏代码应该这样来判断】网页链接谷歌开源了一套代码评审（Code Review）规范，它是谷歌一套通用的工程实战指南，几乎涵盖了所有编程语言与各种类型的项目，这个规范代表了谷歌长期发展以来最佳实战经验的集合，谷歌表示希望开源项目或其他组织能够从这套规范中受益。根据谷歌的项目描述，代码审核规范为两套独立文档组成，代表了两方面内容的最佳实践：1.代码评审者的指南——如何进行代码评审2.更改列表作者指南——更改列表作者批准代码的评审指南在谷歌看来，代码审核的目的是确保谷歌代码库的整体代码健康程度。谷歌将以下规则作为代码评审的标准：一般来说，一旦 CL 能提升整体代码的健康程度，那么即使 CL 不完善，评审者同样也应该倾向于批准该列表。这是所有代码评审指南中的高级原则。它也会有一些限制，例如，如果 CL 添加了一些评审者不需要的特性，那么即使代码做了不错的设计，评审者也应该不予通过。没有所谓的“完美”代码，只有更好的代码。评审人员不应要求作者在批准前对 CL 的每一小部分过分完美。相反，评审者应该权衡向前继续开发的需求和修改建议的重要性。评审者要求的是持续性地改进，而不是追求完美的代码。CL 作为一个整体，如果它能提升系统的可维护性、可读性和可理解性，那么就不要因为它还不完美而推迟数天或数周更新。评审者应该经常留下一些评论，以表达能导致更好性能的做法。如果这些做法并不是非常重要的，那么需要加上前缀「Nit:」，从而令代码作者知道这些内容是可以忽略的。深入了解这套代码规范，请见网页链接