代码中大量的if/else,你有什么优化方案?

代码中大量的if/else,你有什么优化方案?在产品快速迭代的中,由于追求开发速度,我们往往忽略代码的可读性与扩展性,不合理的使用if-else条件判断会使我们的程序复杂度大大提升,同时也会

大家好,欢迎来到IT知识分享网。

作者 | 星野丶
责编 | 欧阳姝黎

代码中大量的if/else,你有什么优化方案?

前言

在产品快速迭代的中,由于追求开发速度,我们往往忽略代码的可读性与扩展性,不合理的使用if-else条件判断会使我们的程序复杂度大大提升,同时也会使代码的可读性急速下降,后期维护难度也大大提高,真的让人脑壳疼。比如下方示例:

// 贷款申请操作的处理
function check() {
// 是否输入正确用户名
if (this.checkUsername(this.username)) {
// 是否输入正确身份证号
if (this.checkIdCard(this.idCard)) {
// 请输入正确的电话号码
if (this.checkTel(this.tel)) {
// 担保人是本人
if (this.dbr === '担保人是本人') {
// 是否存在身份证正面
if (document.querySelector('.sfzzm img')) {
console.log('存在身份证正面')
// 是否存在身份证反面
if (document.querySelector('.sfzfm img')) {
console.log('存在身份证反面')
// 是否存在学历证书
if (document.querySelector('.xlzs img')) {
console.log('存在学历证书')
if (this.ydxy) {
this.tijiaoIsShow = false
}
} else {
Toast('请上传学历证书')
this.tijiaoIsShow = true
}
} else {
Toast('请上传身份证反面')
}
} else {
Toast('请上传身份证正面')
}
} else if (this.dbr == '担保人不是本人') {
if (this.checkUsername(this.dbrname)) {
if (this.checkIdCard(this.dbridCard)) {
if (this.checkTel(this.dbrzyzh)) {
if (document.querySelector('.sfzzm img')) {
console.log('存在身份证正面')
if (document.querySelector('.sfzfm img')) {
console.log('存在身份证反面')
if (document.querySelector('.xlzs img')) {
console.log('存在学历证书')
this.tijiaoIsShow = false
} else {
Toast('请上传学历证书')
}
} else {
Toast('请上传身份证反面')
}
} else {
Toast('请上传身份证正面')
}
} else {
Toast('请输入担保人展业证号')
}
} else {
Toast('请输入担保人身份证号')
}
} else {
Toast('请输入担保人姓名')
}
} else {
Toast('请选择担保人是否为本人')
}
} else {
Toast('请输入正确的电话号码')
}
} else {
Toast('请输入正确的身份证号')
}
} else {
Toast('请输入正确的姓名')
}
}

如果你接手这样一段代码,相信大家和我的心情是一样的:

代码中大量的if/else,你有什么优化方案?

本文归纳以下几种优化if-else场景,希望对大家有所帮助。

  • 单个if语句优化
  • if/else语句优化
  • 单个if多条件优化
  • 多个else if分支优化

代码中大量的if/else,你有什么优化方案?

单个if语句优化



优化前

if (flag) {
this.handleFn()
}

优化后

flag && this.handleFn()
// handleFn是普通函数

这种写法就比较清晰,简洁,好读!

另外如果遇到有很多的if语句,但是执行的功能函数却是一致的情况,我们可以用”逻辑与“或者”逻辑或“来把他们合并成一个表达式。如果我们这些彼此独立的条件判断可以被视为同一次检查的场景时,一次检查的意图明显在可读性上优于多次的条件检查。我们来看一段代码片段:

if (!(staffInfo.patientName && staffInfo.phone)) {
// doSomething
}
...
if (!(staffInfo.phone && staffInfo.idCardNum)) {
// doSomething
}

代码中大量的if/else,你有什么优化方案?

我们可以通过合并逻辑,将上面代码修改为如下:


if(!(staffInfo.patientName && staffInfo.phone) || !(staffInfo.phone && staffInfo.idCardNum)){
// doSomething
}


代码中大量的if/else,你有什么优化方案?

if/else语句优化



if/else可以说在项目中遇到频率是最高,通常可以这两种策略优化

  • 排非策略
  • 三元运算符

排非策略

比如用户登录场景,如果用户名和密码输入框为空,那么我们就提示用户”用户名和密码不能为空”;如果有值,就执行登录的操作。

优化前

 if (user && password) {
// 逻辑处理
} else {
throw('用户名和密码不能为空!')
}

优化后

if (!user || !password) return throw('用户名和密码不能为空!')
// 逻辑处理

表单提交时,需要提前排除那些提交不规范的内容,通常情况下,表单提交遇到不符合我们要求大于我们提交成功的情形,排非策略是个很不错的选择。

三元运算符

示例一

let allow = 
if(age >= 18){
allow = '通过';
} else {
allow = '拒绝';
}

// 优化后
let allow = age >= 18 ? '通过' : '拒绝'

示例二

if (flag) {
success();
} else {
fail();
}

//优化后
flag ? success() : fail();

三元运算符相比if/else来说,只需一行语句,代码简练精炼。

代码中大量的if/else,你有什么优化方案?

单个if多条件优化



优化前

function test(type) {
if (type === 'jpg' || type === 'png' || type === 'gif' || type === 'svg') {
console.log("该文件为图片");
}
}

优化后

function test(type) {
const imgArr = ['jpg', 'png', 'gif', 'svg']
if (imgArr.includes(type)) {
console.log("该文件为图片")
}
}

代码中大量的if/else,你有什么优化方案?

多个else if分支优化



多个else if通常是一个糟糕的选择,它导致设计复杂,代码可读性差,并且可能导致重构困难。

if (this.type === 'A') {
this.handleA();
} else if (this.type === 'B') {
this.handleB();
} else if (this.type === 'C') {
this.handleC();
} else if (this.type === 'D') {
this.handleD();
} else {
this.handleE();
}

我们经常遇到多个else if条件判断代码,随着逻辑复杂性的增加,else if的代码将变得越来越肿。

代码中大量的if/else,你有什么优化方案?

看一下逻辑绘制为流程图从上面的流程图可以看出,不同条件分支的代码具有很高的耦合度。先前的条件判断将影续的代码流,并且此类代码在后续开发中难以维护。我们可以通过switchkey-valueMap来优化代码。

switch

 switch(val){
case 'A':
handleA()
break
case 'B':
handleB()
break
case 'C':
handleC()
break
case 'D':
handleD()
break
}

看一下逻辑绘制为流程图

代码中大量的if/else,你有什么优化方案?

switch

流程图显然更简单。而且,不同的条件分支之间没有嵌套,并且它们彼此独立。逻辑很清楚。

key-value

虽然switch语句在逻辑上确实比else if语句简单,但是代码本身也有点多。

其实我们对象枚举,将条件与特定操作相关联的键值。

let enums = {
'A': handleA,
'B': handleB,
'C': handleC,
'D': handleD,
'E': handleE
}
function action(val){
let handleType = enums[val]
handleType()
}

流程图:

代码中大量的if/else,你有什么优化方案?

这种方式消除了所有条件语句,并使用键值对象存储条件和操作之间的关系。当我们需要根据条件执行代码时,我们不再需要使用else ifswitch语句来处理相应的动作,我们只需从中提取相应的函数handleType并执行它即可。

Map

实际上我们还可以通过Map来进一步的优化我们的代码。

对比Object的话,Map具有许多优点

  • 对象的键只能是字符串或符号,而Map的键可以是任何类型的值。
  • 我们可以使用Map size属性轻松获取Map的键/值对的数量,而对象的键/值对的数量只能手动确定。
  • 具有极快的查找速度。

上面的例子可以优化如下:

let enums = new Map([
['A', handleA],
['B', handleB],
['C', handleC],
['D', handleD],
['E', handleE]
])

function action(val){
let handleType = enums(val)
handleType()
}

如果我们遇到多层复杂条件,Map语句优势就更明显了!

if (mode == 'kline') {
if (this.type === 'A') {
this.handleA()
} else if (this.type === 'B') {
this.handleB()
} else if (this.type === 'C') {
this.handleC()
} else if (this.type === 'D') {
this.handleD()
}
} else if ((mode = 'depth')) {
if (this.type === 'A') {
this.handleA()
} else if (this.type === 'B') {
this.handleB()
} else if (this.type === 'C') {
this.handleC()
} else if (this.type === 'D') {
this.handleD()
}
}

对于上述如此复杂的场景,是否可以通过Map来进行优化?

其实我们只需要将不同的判断语句连接成一个字符串,以便我们可以将条件和操作以键值格式关联在一起。

let enums = new Map([
['kline_A', handleKlineA],
['kline_B', handleKlineB],
['kline_C', handleKlineC],
['kline_D', handleKlineD],
['kline_E', handleKlineE],
['depth_A', handleDepthA],
['depth_B', handleDepthB],
['depth_C', handleDepthC],
])

function action(mode, type){
let key = `${mode}_${type}`
let handleType = enums(key)
handleType()
}

瞬间简单明了很多,有木有~~~

参考文章

  • if-else重构优化
  • if-else逻辑判断优化
  • How to Use if-else
  • Javascript条件语句优化策略
  • 深入浅出代码优化﹣if/else

代码中大量的if/else,你有什么优化方案?

生于2001年的《程序员》曾陪伴了无数开发者成长,影响了一代又一代的中国技术人。时隔20年,《新程序员》带着全球技术大师深邃思考、优秀开发者技术创造等深度内容回来了!同时将全方位为所有开发者呈现国内外核心技术生态体系全景图。扫描下方小程序码即可立即订阅!

免责声明:本站所有文章内容,图片,视频等均是来源于用户投稿和互联网及文摘转载整编而成,不代表本站观点,不承担相关法律责任。其著作权各归其原作者或其出版社所有。如发现本站有涉嫌抄袭侵权/违法违规的内容,侵犯到您的权益,请在线联系站长,一经查实,本站将立刻删除。 本文来自网络,若有侵权,请联系删除,如若转载,请注明出处:https://yundeesoft.com/46774.html

(0)

相关推荐

发表回复

您的电子邮箱地址不会被公开。 必填项已用 * 标注

关注微信