-
【强制】在一个
switch
块内,每个 case 要么通过 break / return 等来终止,要么注释说明程序将继续执行到哪一个 case 为止 ; 在一个 switch 块内,都必须包含一个 default 语句并且放在最后,即使它什么代码也没有。 -
【强制】在
if
/else
/for
/while
/do
语句中必须使用大括号。即使只有一行代码,避免采用单行的编码方式:if (condition) statements;
如果整个条件语句(条件和主体)适合放在同一行,那么您可以(但不是必须)将其全部放在一行上
// ① 我们接受以下样式 if (condition) { body(); } // ② 或者 if (condition) body(); - - - - - - - - - - - - - - - - - - // ③ 但不接受以下样式: if (condition) body(); // bad!
- 【推荐】表达异常的分支时,少用 if-else 方式 ,这种方式可以改写成:
if (condition) {
...
return obj;
}
// 接着写 else 的业务逻辑代码;
说明:如果非得使用 if()...else
if()...else...
方式表达逻辑,【强制】避免后续代码维护困难,请勿超过 3 层。
正例:超过 3 层的 if-else 的逻辑判断代码可以使用卫语句、策略模式、状态模式等来实现,其中卫语句示例如下:
public void today() {
if (isBusy()) {
System.out.println(“change time.”);
return;
}
if (isFree()) {
System.out.println(“go to travel.”);
return;
}
System.out.println(“stay at home to learn Alibaba Java Coding Guidelines.”);
return;
}
写程序一定要遵守红花绿叶原则,主逻辑放在主方法中,这是红花,子逻辑封装成小方法调用,这是绿叶,不要把不同层次的逻辑写在一个大方法体里,很难理解,就像绿叶把红花挡住了,谁还能看到。
- 【推荐】除常用方法(如 getXxx/isXxx)等外,不要在条件判断中执行其它复杂的语句,将复杂逻辑判断的结果赋值给一个有意义的布尔变量名,以提高可读性。
说明:很多 if 语句内的逻辑相当复杂,阅读者需要分析条件表达式的最终结果,才能明确什么样的条件执行什么样的语句,那么,如果阅读者分析逻辑表达式错误呢?
正例:
// 伪代码如下
final boolean existed = (file.open(fileName, "w") != null) && (...) || (...);
if (existed) {
...
}
反例:
if ((file.open(fileName, "w") != null) && (...) || (...)) {
...
}
- 【推荐】循环体中的语句要考量性能,以下操作尽量移至循环体外处理,如定义对象、变量、获取数据库连接,进行不必要的 try - catch 操作 ( 这个 try - catch 是否可以移至循环体外 ) 。
切记,循环体内尽量不要获取资源、不要处理异常。
- 【推荐】接口入参保护,这种场景常见的是用于做批量操作的接口。
接口入参保护,“保护”的是服务端应用,即接口提供方,最常见的做法就是校验入参的有效值范围和设置批量操作白名单;
比如,接口入参中包含日期时,校验日期必须在N天范围内,或者请求返回的记录总数必须在X条以内(比如10W条,否则缩小请求查询的记录范围),或者请求返回的记录必须分页查询返回;
开发手册中,尤其提到的场景就是批量操作,因为批量操作非常耗时耗资源(服务端),批量操作的批量数应该有上限,而不是无限的。
用白话说,就是控制批量参数的数量,一次不能太多,否则内存溢出。
- 【参考】下列情形,需要进行参数校验:
1 ) 调用频次低的方法。
2 ) 执行时间开销很大的方法。此情形中,参数校验时间几乎可以忽略不计,但如果因为参数错误导致中间执行回退,或者错误,那得不偿失。
3 ) 需要极高稳定性和可用性的方法。
4 ) 对外提供的开放接口,不管是 RPC / API / HTTP 接口。
5) 敏感权限入口。
根据业务适当调整是可以的。
- 【参考】下列情形,不需要进行参数校验:
1 ) 极有可能被循环调用的方法。但在方法说明里必须注明外部参数检查要求。
2 ) 底层调用频度比较高的方法。毕竟是像纯净水过滤的最后一道,参数错误不太可能到底
层才会暴露问题。一般 DAO 层与 Service 层都在同一个应用中,部署在同一台服务器中,所以 DAO 的参数校验,可以省略。
3 ) 被声明成 private 只会被自己代码所调用的方法,如果能够确定调用方法的代码传入参数已经做过检查或者肯定不会有问题,此时可以不校验参数。
根据业务适当调整是可以的。
参考
- 《编写高质量代码:改善Java程序的151个建议》
白话阿里巴巴Java开发手册(安全规约) - 李艳鹏 - 简书(https://www.jianshu.com/p/9528c4ea1504)
网友评论