title: 2.1、规范导读 V2.0
2019 年,我们 1024 创新实验室 发布《1024 创新实验室-Java 规范 V1.0 和 前端规范 V1.0》,在业内收到了很多好评,时至今日过去了三年,我们实验室在与百余家企业沟通和交流中,以及在我们实验室团队的不断成长和不断的摸索中,我们又在 v1.0 的基础上做了更好的改进。规范的目的是为了编写高质量的代码,让你的团队成员每天得心情都是愉悦的,大家在一起是快乐的。
引自《阿里规约》的开头片段:
----现代软件架构的复杂性需要协同开发完成,如何高效地协同呢?无规矩不成方圆,无规范难以协同,比如,制订交通法规表面上是要限制行车权,实际上是保障公众的人身安全,试想如果没有限速,没有红绿灯,谁还敢上路行驶。对软件来说,适当的规范和标准绝不是消灭代码内容的创造性、优雅性,而是限制过度个性化,以一种普遍认可的统一方式一起做事,提升协作效率,降低沟通成本。代码的字里行间流淌的是软件系统的血液,质量的提升是尽可能少踩坑,杜绝踩重复的坑,切实提升系统稳定性,码出质量。
我们开源的代码规范基于阿里巴巴、华为的开发手册,添加了我们团队的风格和规范,补充了一些细节。感谢前人的经验和付出,让我们可以有机会站在巨人的肩膀上眺望星辰大海。
我们制定规则的目的:为了大多数程序员小伙伴可以得到更多的好处,如果在团队实际运作中认为某个规则无法遵循或有更好的做法,希望大家可以共同改进该规范。
在 V1.0 版本中,我们对好的代码引用了 Kent Beck 的简单设计四原则,大家反馈比较抽象。V2.0 我们决定用白话来简单讲讲好的代码原则:
以上四个原则的重要程度依次降低, 这是我们1024 创新实验室认为好代码的原则,即:简单的、好的、代码
无论前端代码还是后端代码、异或其他代码,都是由一个个单词组成的,所以一个好的单词影响着代码的本身,所以我们定义如下:
很多人认为自己英语不好就命名比较随意,但我们看来,一个有道词典或者百度翻译就能看好的解决这件事情,所以单词的命名必须使用正确。我们曾经遇到过这种起名字的,比如有一个双11业务,要求第一天业务、第二天、第三天业务的区别,有些小伙伴这样起名字:
第一天: di1day
第二天: di2day
第三天: treeday (三应该是 three,笑喷)
以上是真实发生的例子,故我们定义如下:
我们知道在大部分编程语言中, 项目、类Class、数据库、表名、url中的前半部分 等等 一个比较大的范围都是应该用名词,比如java中的class
、interface
、enum
等等都是名词,比如 OrderService
。
而具体的方法名应该是动词异或动名词,比如方法:创建订单:createOrder
,查询订单queryOrder
无论团队里的前端、后端、移动端、数据库、服务器、redis、docker等等一切对于 某个业务或者某个业务单元的命名必须高度保持一致。
比如之前遇到过这么一个功能,OA办公系统的`通知`功能,各个端定义为:
- 后端:`notice`, (NoticeController, 表:t_notice)
- 前端用成了`news`, (news-list.vue, /news/news-list)
- 移动端用成了`message`,(MessageFragement)
- 最后再对接的时候,懵逼了,懵了;
注释是我们披荆斩棘历经磨难翻越需求这座大山时,留下的踪迹和收获的经验教训,这些宝贵的知识除了证明我们曾经存在过,也提醒着后来的人们殷鉴不远、继往开来。
注释除了说明作用、逻辑之外。还有一个很重要的原因:当业务逻辑过于复杂,代码过于庞大的时候,注释就变成了一道道美化环境、分离与整理逻辑思路的路标。这是很重要的一点,它能有效得帮助我们免于陷入代码与业务逻辑的泥沼之中。
正例:
/**
* 开始抽奖方法
* 保存中奖信息、奖励用户积分等
* @param luckDrawDTO
* @return ResponseDTO 返回中奖信息
*/
public ResponseDTO<String> startLuckDraw(LuckDrawDTO luckDrawDTO) {
// -------------- 1、校验抽奖活动基本信息 ------------------------
xxx伪代码一顿操作
// -------------- 2、新增抽奖记录 -------------------------------
xxx伪代码一顿操作
// -------------- 3、如果需要消耗积分,则扣除钢镚积分 -------------
xxx伪代码一顿操作
// -------------- 4、获取奖品信息,开始翻滚吧 --------------------
xxx伪代码一顿操作
return ResponseDTO.succ(luckDrawPrizeVO);
}
注释并不是越多越好,当注释过多,维护代码的同时,还需要维护注释,不仅变成了一种负担,也与我们当初添加注释的初衷背道而驰。
首先:大家应该通过清晰的逻辑架构,好的变量命名来提高代码可读性;需要的时候,才辅以注释说明。注释是为了帮助阅读者快速读懂代码,所以要从读者的角度出发,按需注释。注释内容要简洁、明了、无二义性,信息全面且不冗余。
其次:无论是修改、复制代码时,都要仔细核对注释内容是否正确。只改代码,不改注释是一种不文明行为,破坏了代码与注释的一致性,会让阅读者迷惑、费解,甚至误解。
反例:
// 查询部门
EmployeeVO employee = employeeDao.listByDepartmenttId(deptId);
方法要尽量通过方法名自解释,不要写无用、信息冗余的方法头,不要写空有格式的方法头注释。
方法头注释内容可选,但不限于:功能说明、返回值,用法、算法实现等等。尤其是对外的方法接口声明,其注释,应当将重要、有用的信息表达清楚。
正例:
/**
* 解析转换时间字符串为 LocalDate 时间类
* 调用前必须校验字符串格式 否则可能造成解析失败的错误异常
*
* @param dateStr 必须是 yyyy-MM-dd 格式的字符串
* @return LocalDate
*/
public static LocalDate parseYMD(String dateStr){}
反例:
/**
* 校验对象
*
* @param t
* @return String
*/
public static <T> String checkObj(T t);
反例中出现的问题:
TODO/TBD(to be determined)
注释一般用来描述已知待改进、待补充的修改点,并且加上作者名称。
FIXME
注释一般用来描述已知缺陷,它们都应该有统一风格,方便文本搜索统一处理。如:
// TODO <author-name>: 补充XX处理
// FIXME <author-name>: XX缺陷
举例:
// TODO <卓大> 为了数据安全,此处应该对手机号进行加*处理
因为现在所有的项目都使用了代码管理工具,比如 git、svn 、ts等等,所以对于无用的代码,让我们尽情的删除掉吧!
重要的说三遍:
不要注释,不要注释,不要注释!
要删除代码,要删除代码,要删除代码!
若此次提交代码对应禅道中的任务或者 bug,格式如下:
task#[任务id] [任务标题] [具体事项]
bug#[bug id] [bug标题] [具体事项]
例子如下:
git commit -m 'task#1101 开发smartreload功能 完成线程池的编码'
git commit -m 'bug#1102 smartreload时间不正确 线程池的大小问题'
使用 git,必须添加 .gitignore 忽略配置文件。
不要提交与项目无关的内容文件:idea 配置、target 包等。
1024创新实验室-主任:卓大,混迹于各个技术圈,研究过计算机,熟悉点 java,略懂点前端。
1024创新实验室(河南·洛阳) 致力于成为中原领先、国内一流的技术团队,以技术创新为驱动,合作各类项目。
加 主任 “卓大” 微信 拉你入群,一起学习 |
关注 “小镇程序员” 分享代码与生活、技术与赚钱 |
请 “1024创新实验室” 喝咖啡 支持我们的开源与分享 |