代码规范及CodeReview要点

作者: go4it | 来源:发表于2015-05-22 07:57 被阅读2919次

一、代码规范的要点

代码规范主要分为风格规范与设计规范两大类:

1、代码风格规范

主要是文字上的规定,看似表面文章,实际上非常重要。

具体有如下几个方面:

(1)缩进

(2)行宽

(3)断行/空白行

(4)括号

(5)命名(字母、下划线、大小写)

(6)注释

A、单行注释

B、多行注释

C、变量/方法/类/包注释

2、代码设计规范

牵涉到程序设计、模块之间的关系、设计模式等方方面面的通用原则。

主要有如下几个方面:

(1)方法/函数的写法

A、方法命名

B、方法参数(入参/返回值)

C、方法的职责

比如:避免out型参数、用枚举替代boolean、同类型参数最好间隔开、超过4个参数最好抽象成一个类、参数和返回值最好不传null、用卫述句减少if嵌套、方法连续调用要注意空指针、for循环优于while

(2)异常处理原则

比如:异常的抽象层次应该与方法所在的层次一致,业务层方法要对底层异常进行转译为业务异常

(3)分层/类设计原则

比如:在某一个层进行防御式校验,某一层按约定不做参数校验;比如调用到的外部接口封装为facade防腐层;

(4)单测原则

比如:快速/及时、独立、可重复、覆盖主要代码路径、无副作用

二、CodeReview注意事项

主要根据团队设定的代码规范,来review团队成员的代码,大致有以下几个方面:

1、代码有没有不符合代码规范的

比如:命名、注释

2、代码有没有(业务/算法)逻辑错误

比如:功能与需求有偏差;参数传递顺序出错;方法的边界条件有没有考虑等

3、代码有没有回归错误

比如:之前的功能回归测试不通过

4、代码有没有潜在性能问题

比如:考虑大数据量、大并发量下的性能下sql是否有问题?是否会有内存泄露?死锁等

5、代码有没有其他待改进的地方

比如:可扩展性/过度设计

——参考《构建之法》

相关文章

  • iOS Code Review

    CodeReview iOS App 稳定性指标及监测(转载) 代码规范及CodeReview要点 iOS-程序员...

  • 代码规范及CodeReview要点

    一、代码规范的要点 代码规范主要分为风格规范与设计规范两大类: 1、代码风格规范 主要是文字上的规定,看似表面文章...

  • CodeReview规范

    目标和原则 提高代码质量,及早发现潜在缺陷,降低修改/弥补缺陷的成本 促进团队内部知识共享,提高团队整体水平 评审...

  • 代码审核

    引用: iOS - CodeReview 代码评审

  • 关于CodeReview

    什么是CodeReview CodeReview的目的是提升代码质量,尽早发现潜在缺陷与BUG,降低修复成本,同时...

  • 代码风格及规范

    JavaScript Standard Style[https://standardjs.com/rules-zh...

  • iOS 代码评审

    1、CodeReview Code Review 中文应该译作 “代码审查” 或是 “代码评审”,这是一个流程,当...

  • CodeReview常见代码问题

    常见代码问题 常见的潜在代码问题是当前直接会导致BUG、故障或者产品功能不能正常工作的类别。 空值 空值恐怕是最容...

  • iOS小组代码分支管理

    代码分支管理规范 为了规范代码库分支管理和版本管理,使代码分支及版本结构清晰,方便维护,并避免由于维护造成的错误的...

  • View层组织和调用方案

    View层组织和调用方案 viewController的代码规范: 要点: 所有的属性都是用getter和sett...

网友评论

    本文标题:代码规范及CodeReview要点

    本文链接:https://www.haomeiwen.com/subject/fylutttx.html