如何改造以下场景 if-else 的代码？

V2EX = way to explore

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

现在注册

已注册用户请登录

推荐关注

Meteor

JSLint - a Javascript code quality tool

jsFiddle

D3.js

WebStorm

推荐书目

Javascript 权威指南第 5 版

Closure: The Definitive Guide

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

有一批物品数据，每个物品都有 id 字段,同时可能有 config 字段（如果没有这个字段则跳过忽略），用户每次会添加不同的物品现在需要在用户添加的物品列表中，取出权重最高的那个 config 字段，并最后返回，其中： id 为 1、3 的物品权重最，2、4 次之，5、7 最低。。以此类推，可能还会有还有很多其他的 id

现在我个人的做法是，循环遍历

var cOnfig= null; for (var item of items){ if (item['id'] === 1 || item['id'] === 3){ cOnfig= config ? (item['config']?item['config']:null) : null }else if (item['id'] === 2 || item['id'] === 4){ cOnfig= config ? (item['config']?item['config']:null) : null }... }

但是这样太蠢。。有什么更好的模式可以取代这种操作吗，或者这种场景应该如何处理

22 条回复 2017-05-31 23:36:56 +08:00

ferrum

2017 年 5 月 30 日 via iPhone

看看你这里写的代码，已经有一部分重复了，就把重复那部分提取出来，作为一个函数。

EXDestroyer

2017 年 5 月 30 日

@ferrum 重复部分是可以优化，现在我比较头疼的是大量的 if else，不知道怎么处理更好

4641585

2017 年 5 月 30 日

写成 Switch ？
印象中哪个课本上提到过条件嵌套的优化，然而已经记不清了

sumhat

2017 年 5 月 30 日

大约是这个样子:

https://gist.github.com/sumhat/268b560d8c89c4e7cf3be6a7d5f00fdc

nutting

2017 年 5 月 30 日 via Android

map?然后排序?

EXDestroyer

2017 年 5 月 30 日

@sumhat 这个思路挺好的，哈哈

gmywq0392

2017 年 5 月 30 日 via Android

这么个法子不知道对不对。
先按照你业务约束的 id 权重顺序写个数组*。
然后做 table(或者说是 object)给他个 comparable 函数，具体就是用 item 的 id 去按照*的索引排序。然后再取该 table(或者说是 object)的 first item 不就…

8qwe24657913

2017 年 5 月 30 日

@sumhat #4
var idLevel = Infinity;
function getIdLevel(id) {
return Math.ceil(id / 4) * 2 - id % 2
}

EXDestroyer

2017 年 5 月 30 日

@8qwe24657913 这个权重是我假设的哈~其实真实情况不一定是有规律的权重

8qwe24657913

2017 年 5 月 30 日

@EXDestroyer #9
var levels = {
id1:level1,
id2:level2,
};
function getIdLevel(id) {
return levels[id];
}

sagaxu

2017 年 5 月 30 日

2000 个 id 你就写 1000 个 if-else?

wangxiaoer

2017 年 5 月 30 日 via Android

这不就是 Items 排序？看样子是 js，那就写个排序函数 fun 不就完了？ Fun 里面按你的 id 规则来

Items.sort(fun)

sensui7

2017 年 5 月 31 日

if 语句可以用真值表简化

autoxbc

2017 年 5 月 31 日

var cOnfig= null ;
[1,3,2,4,5,7].some(function(level){
return items.some(function(item){
if(item.config && item.id === level)
return cOnfig= item.config ;
});
});
alert(config);

只需要维护一个权重数组。

autoxbc

2017 年 5 月 31 日

为什么 V2EX 歧视 Tab 缩进派

var cOnfig= null ;
[1,3,2,4,5,7].some(function(level){
return items.some(function(item){
if(item.config && item.id === level)
return cOnfig= item.config ;
});
});
alert(config);

autoxbc

2017 年 5 月 31 日

全角空格缩进

var cOnfig= null ;
[1,3,2,4,5,7].some(function(level){
　　 return items.some(function(item){
　　　　 if(item.config && item.id === level)
　　　　　　 return cOnfig= item.config ;
　　});
});
alert(config);