有时候帮人改代码,我就提醒他注意代码规范,结果被喷我有强迫症!
我是觉得代码规范真的很重要,良好的缩进,命名,注释是必须的,你的代码不只是给自己看的,也是给别人和将来的自己看的,我现在看我以前写的东西都会有陌生感,怀疑这不是我写的,这还是我保持良好的规范的前提下。
除了上面这三种规范,其实还有很多不像是规范但是应该遵循的原则,比如一个简单的例子:
boolean flag = false;
if (flag == false) {
// do something
}
flag
本身就是条件变量,又何必再用==
去比较呢?
boolean flag = false;
if (flag) {
// do something
}
这样的例子很多,也是很多人不在意的,但是我在意。我想说这不是强迫症,作为一个程序员,代码对于我,就像雕像对于雕刻家,我把它看做一件艺术品,精雕细琢,这是一种精益求精的工匠精神,才不是强迫症。
这篇文章只要总结一下如何优化代码中大量if/else,更多的优化会在以后慢慢补充。
1. 提前return
这是判断条件取反的做法,代码在逻辑表达上会更清晰,看下面代码:
if (condition) {
// do something
} else {
return xxx;
}
其实,每次看到上面这种代码,我都心里抓痒,完全可以先判断!condition
,干掉else。
if (!condition) {
return xxx;
}
// do something
2. 策略模式
有这么一种场景,根据不同的参数走不同的逻辑,其实这种场景很常见。
最一般的实现:
if (strategy.equals("fast")) {
// 快速执行
} else if (strategy.equals("normal")) {
// 正常执行
} else if (strategy.equals("smooth")) {
// 平滑执行
} else if (strategy.equals("slow")) {
// 慢慢执行
}
看上面代码,有4种策略,有两种优化方案。
2.1 多态
interface Strategy {
void run() throws Exception;
}
class FastStrategy implements Strategy {
@Override
void run() throws Exception {
// 快速执行逻辑
}
}
class NormalStrategy implements Strategy {
@Override
void run() throws Exception {
// 正常执行逻辑
}
}
class SmoothStrategy implements Strategy {
@Override
void run() throws Exception {
// 平滑执行逻辑
}
}
class SlowStrategy implements Strategy {
@Override
void run() throws Exception {
// 慢速执行逻辑
}
}
具体策略对象存放在一个Map中,优化后的实现
Strategy strategy = map.get(param);
strategy.run();
上面这种优化方案有一个弊端,为了能够快速拿到对应的策略实现,需要map对象来保存策略,当添加一个新策略的时候,还需要手动添加到map中,容易被忽略。
2.2枚举
发现很多同学不知道在枚举中可以定义方法,这里定义一个表示状态的枚举,另外可以实现一个run方法。
public enum Status {
NEW(0) {
@Override
void run() {
//do something
}
},
RUNNABLE(1) {
@Override
void run() {
//do something
}
};
public int statusCode;
abstract void run();
Status(int statusCode){
this.statusCode = statusCode;
}
}
重新定义策略枚举
public enum Strategy {
FAST {
@Override
void run() {
//do something
}
},
NORMAL {
@Override
void run() {
//do something
}
},
SMOOTH {
@Override
void run() {
//do something
}
},
SLOW {
@Override
void run() {
//do something
}
};
abstract void run();
}
通过枚举优化之后的代码如下
Strategy strategy = Strategy.valueOf(param);
strategy.run();
3. 学会使用 Optional
Optional主要用于非空判断,由于是jdk8新特性,所以使用的不是特别多,但是用起来真的爽。
使用之前:
if (user == null) {
//do action 1
} else {
//do action2
}
如果登录用户为空,执行action1,否则执行action 2,使用Optional优化之后,让非空校验更加优雅,间接的减少if操作
Optional<User> userOptional = Optional.ofNullable(user);
userOptional.map(action1).orElse(action2);
4. 数组小技巧
来自google解释,这是一种编程模式,叫做表驱动法,本质是从表里查询信息来代替逻辑语句,比如有这么一个场景,通过月份来获取当月的天数,仅作为案例演示,数据并不严谨。
一般的实现:
int getDays(int month){
if (month == 1) return 31;
if (month == 2) return 29;
if (month == 3) return 31;
if (month == 4) return 30;
if (month == 5) return 31;
if (month == 6) return 30;
if (month == 7) return 31;
if (month == 8) return 31;
if (month == 9) return 30;
if (month == 10) return 31;
if (month == 11) return 30;
if (month == 12) return 31;
}
优化后的代码
int monthDays[12] = {31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
int getDays(int month){
return monthDays[--month];
}
if/else作为每种编程语言都不可或缺的条件语句,在编程时会大量的用到。一般建议嵌套不要超过三层,如果一段代码存在过多的if else嵌套,代码的可读性就会急速下降,后期维护难度也大大提高
2 条评论
eslint标准校验的格式它不对?各种框架有些格式不一样就不必要吹毛求疵(强迫症除外),且,一般来说循环if性能最差,多重嵌套不建议使用(站在es6角度)
eslint只是一个代码检查工具,完全可按照你自己的喜好配置:缩进几个空格、行尾加不加分号等,它不是一个规范;不同语言可以按照各自的格式去写;循环是循环,if是条件语句,不太明白循环if是什么;多重嵌套确实不建议的。