switch (val) {
case 1:
func()
break
case 2:
let a = 1
break
case 3:
return 1
}
switch (val) {
case 1:
func()
break
case 2:
let a = 1
break
case 3:
return 1
}
如果是我写的话,我会把里面魔术数问题给解决了,结合上下文改改成类似这样的形式:
const APP_TYPE= {
GAME: 1,
TOOLS: 2,
SOCIAL: 3
};
switch (val) {
case APP_TYPE.GAME:
func()
break
case APP_TYPE.TOOLS:
let a = 1
break
case APP_TYPE.SOCIAL:
return 1
}
这样,读到代码就知道这段代码的意思是啥,比单纯写数字要好.
里面别的魔术数没改,只改了一个主要的部分.
个人看法.
// redux里面用的一种方式
reducer = {
'1': () => {return 1},
'2': () => {return 2},
'3': () => {return 3}
}
function action (type) {
return reducer[type]()
}
13 回答13k 阅读
7 回答2.1k 阅读
3 回答1.3k 阅读✓ 已解决
6 回答1.2k 阅读✓ 已解决
2 回答1.4k 阅读✓ 已解决
3 回答1.3k 阅读✓ 已解决
6 回答1.1k 阅读
这样写可读性好,别改了