尽量优化 if-else 语句,防止代码结构变得复杂混乱,防止过多的大括号,导致代码阅读困难。
一般禁止在 if 条件表达式中进行函数调用及运算,此时应先行计算出条件表达式的结果,避免 if 条件表达式过于庞大,导致代码阅读困难。允许简短的调用及运算、数据比较等单一简短的表达式。
原始代码:
if (StrUtil.equals(UserService.getUserStatus(user.getIp()), "online")) { // if (user.isConnect()) 简短的调用及运算可以允许
// TODO
}
优化为:
String userStatus = userService.getUserStatus(user.getIp()); // 先行计算条件表达式
if (StrUtil.equals(userStatus, "online")) { // 不建议将 boolean isOnline = StrUtil.equals(userStatus, "online"); 也先行计算出,意义不大。
// TODO
}
当 if-else 语句具有两个分支,且分支业务简短时,考虑使用三目运算符。
原始代码:
public boolean isOnline() {
if (StrUtil.equals(userStatus, "online")) {
return true;
} else {
return false;
}
}
优化为:
public boolean isOnline() {
return StrUtil.equals(userStatus, "online") ? true : false;
}
else 后面没有其他业务时,可省略最后的 else,使代码简洁。
原始代码:
public void addUser() {
if (StrUtil.equals(userStatus, "online")) {
// TODO 1
return;
} else {
// TODO 2
return;
}
// else 后面没有其他业务时,可省略最后的else,使代码简洁
}
优化为:
public void addUser() {
if (StrUtil.equals(userStatus, "online")) {
// TODO 1
return;
}
// TODO 2
return;
}
减少 if 语句包裹的内容,选择包裹代码少的条件业务,异常状态直接返回。
原始代码:
public void addUser() {
if (StrUtil.equals(userStatus, "online")) {
// TODO 1.1
// TODO 1.2
return;
}
// TODO 2
return;
}
优化为:
public void addUser() {
if (!StrUtil.equals(userStatus, "online")) {
// TODO 2
return;
}
// TODO 1.1
// TODO 1.2
return;
}
减少 if 语句包裹的内容,选择包裹代码少的条件业务,异常状态直接返回。
原始代码:
public void editUser() {
// 用户存在时,进行大量处理业务
if (dbUser != null) {
// 处理用户数据
...
...
...
}
优化为:
public void editUser() {
// 用户不存在时,直接返回
if (dbUser == null) {
return;
}
// 处理用户数据
...
...
...
}
避免 if 嵌套,防止代码结构复杂。
原始代码:
public void addUser(User user) {
if (ValidUtil.isIdCardNo(user.getIdCardNo())) {
// 处理用户数据
...
if (ValidUtil.isPhone(user.getPhone())) {
// 处理用户数据
...
if (ValidUtil.isEmail(user.getEmail())) {
// 处理用户数据
...
} else {
throw new RuntimeException("邮箱格式错误!");
}
} else {
throw new RuntimeException("手机号格式错误!");
}
} else {
throw new RuntimeException("身份证号格式错误!");
}
}
优化为:
public void addUser(User user) {
if (!ValidUtil.isIdCardNo(user.getIdCardNo())) {
throw new RuntimeException("身份证号格式错误!");
}
// 处理用户数据
...
if (!ValidUtil.isEmail(user.getEmail())) {
throw new RuntimeException("手机号格式错误!");
}
// 处理用户数据
...
if (!ValidUtil.isEmail(user.getEmail())) {
throw new RuntimeException("邮箱格式错误!");
}
// 处理用户数据
...
}
为什么写这个案例,因为大把的人在干这种事!
Optional.ofNullable
(T).orElseThrow
(() -> new Exception(“”));
原始代码:
if (user == null) {
throw new Exception("未查询到用户信息");
}
优化为:
Optional.ofNullable(user).orElseThrow(() -> new Exception("未查询到用户信息"));
Optional.ofNullable
(T).ifPresent
(x -> fun(x));
原始代码:
if (user != null) {
update(user); // 执行方法调用
}
优化为:
Optional.ofNullable(user).ifPresent(user -> update(user));
根据相应场景使用合适的优化方案。
Optional 类虽然简洁,不过采用这种链式编程,虽然代码优雅了。但是,逻辑性没那么明显,可读性有所降低,酌情使用。