if 条件判断放在哪里比较好？

推荐关注

› Meteor

› JSLint - a JavaScript code quality tool

› D3.js

推荐书目

› JavaScript 权威指南第 5 版

› Closure: The Definitive Guide

This topic created in 2450 days ago, the information mentioned may be changed or developed.

function fn(){
	if(type == 1){
    	fn2();
    }else{
    	fn3();
    }
}

我现在有个 fn 方法，执行的时候会根据 type 进行判断到底调用 fn2 和 fn3 哪一个方法，在调用这些方法之前，需要做一些判断，判断这个方法需要的值是否为空之类的，有些判断是相同的，是放在工共的 fn 中比较好，还是这些判断在 fn2 和 fn3 中都写一次比较好

function fn(){
	// if(a == ''){
    // 
    // }
	if(type == 1){
    	fn2();
    }else{
    	fn3();
    }
}

7 replies • 2019-10-15 16:37:14 +08:00

wb500

Oct 15, 2019

看你 type 侧重点，如果 type 比较重要：先用 type 来区分执行 f2 还是 f3，f2,f3 里面放公共 fn ；否则的话现有的就可以

markzyh

Oct 15, 2019

function fn() {
if (!a) {
if (type == 1) fn1();
else fn2();
}
}

waiaan

Oct 15, 2019

公共的吧，然后闭包返回。

ironMan1995

Oct 15, 2019

不知道其它地方用到 fn2 fn3 没，个人倾向只在 fn 中做根据 type 属性调用哪个函数的判断(后面也许需要有根据更多 type 属性值调用其它的方法可以使用 switch)，函数执行需要的变量只在自身作用域中做判断逻辑处理，低耦合

sheaned

Oct 15, 2019

一个函数自己要用到的参数自己去判断，干嘛让别的函数帮你做呢

leoskey

Oct 15, 2019

1. 尽量避免写重复的代码；
2. 错误优先执行；
3. 少写面条代码，该拆就拆。
```javascript
function fn() {
if(isChecked()){

}
}

```

leoskey

Oct 15, 2019

1. 尽量避免写重复的代码；
2. 错误优先执行；
3. 少写面条代码，该拆就拆。
```
function fn(type) {
if(isChecked()) {
return type == 1 ? fn1() : fn2();
}
}
```