这一段 JS 语句怎么写才能让它更通顺和简单

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

推荐关注

Meteor

JSLint - a Javascript code quality tool

jsFiddle

D3.js

WebStorm

推荐书目

Javascript 权威指南第 5 版

Closure: The Definitive Guide

这是一个创建于 2769 天前的主题，其中的信息可能已经有所发展或是发生改变。

 // meet [Boolean] 是否需要全部满足 // caps[Array] 需要满足的条件 // userCaps [Object] 当前用户拥有的权限 if (meet === false) { return caps.find((cap) => { if (userCaps[cap] === true) return true }) != null ? true : false } else { return caps.find((cap) => { if (userCaps[cap] !== true) return true }) != null ? false : true }

这是一段前端判断用户是否有权限的句子，如果条件是 meet 则需要全部满足，否则只要满足其一。

其实我是想吧这两个 return 合并为一个，因为只有中间 true 判断和结尾的 false : true 相反，但是可读性巨差无比

就像下面这样

 return caps.find((cap) => { if (meet ? userCaps[cap] !== true : userCaps[cap] === true) return true }) != null ? !meet : meet

在大厂这样写 js 会被辞退吗

8 条回复

UnluckyNinja

2018-03-15 14:38:21 +08:00

some() every()了解一下

hoythan

2018-03-15 14:41:55 +08:00

@UnluckyNinja 浏览器兼容有点高额，我虽然用了 ES6 但是最终还是要转 5 的

hoythan

2018-03-15 14:43:02 +08:00

@UnluckyNinja 我看错了兼容性

narcotics

2018-03-15 14:44:10 +08:00 via Android

@hoythan lodash 之类的库也不在考虑范围内吗？

hoythan

div class="badge op">OP

2018-03-15 14:48:26 +08:00

```
return meet ? caps.some(() => {

}) : caps.every(() => {

})
```

hoythan

2018-03-15 14:49:02 +08:00

```
return meet ? caps.every(() => {

}) : caps.some(() => {

})
```

UnluckyNinja

2018-03-15 14:49:41 +08:00

@hoythan #3 注意下 every 对空数组固定输出 true （这两个函数一个假定 false 一个假定 true，遇到不同值直接跳出遍历立即返回），应该就没什么坑了

hoythan

2018-03-15 14:54:38 +08:00

@UnluckyNinja