使 JavaScript 代码变得干燥的最优雅的方法

Question

这是我想要干燥的 JavaScript 代码片段：

if(agency == 'abcd')map.entities.push(abcd);
if(agency == 'efgh')map.entities.push(efgh);
if(agency == 'xyz')map.entities.push(xyz);
if(agency == 'pqrs') map.entities.push(pqrs);
if(agency == 'values')map.entities.push(values);

现在将来可能会有更多针对不同键的 if。JavaScript 不提供我可以在此处使用的 HashMap 构建。使用数组和 id 之类的东西来使其干燥也太俗气了。 有没有更简单的解决方案？可能是这样的

if(agency == 'abcd')map.entities.push(stringToVariable('abcd'));

，然后我可以使用 for 并遍历键。我不确定这在 JavaScript 中是否可能。

Answer 1

也许吧：

map.entities.push({abcd: abcd, efgh: efgh, xyz: xyz}[agency]);

对我来说，这种情况需要退后一步，重新思考更多情况，因为它本质上是丑陋的。为什么存在所有这些单独的变量，而不是具有与“代理”值相对应的键的单个对象？

Answer 2

只要您知道 agency 值是干净的，就可以在此处使用 eval：

map.entities.push(eval(agency));

Answer 3

javascript 中的每个对象都是一个 hashmap（至少是一个 hashmap），这就是这段代码返回 true 的原因：

var obj = {};
obj.test = "someVarVal";
obj.test === obj['test'];

因此，您可以做的是构建一个返回值/函数的对象，如下所示：

var returns = {};
returns['abcd'] = function() {return 'abcd';};
returns['efgh'] = function() {return 'efgh';};

然后执行

map.entities.push(returns[agency]());

如果所有变量都是静态的你可以做得更简单，就像这样：

var returns = {};
returns['abcd'] = 'abcd';
map.entities.push(returns[agency]);

Answer 4

function _inArr(arr,itm)
{
  for (var i=0; i < arr.length; i++)
  {
     if (arr[i] == itm) return true;
  }
  return false;
}

var allowedAgencies = array['abcd','efgh','xyz','pqrs','values'];
if (_inArr(allowedAgencies, agency))
{
  map.entities.push(window[agency]);
}

_inArr 函数在每个主要的 javascript 包中都可用（例如 $.inArray）。

我认为重点是 JavaScript 是一个很大的哈希图。例如，全局函数名称只是窗口对象的一个​​键。对于任何命名空间项目也是如此。

Answer 5

var agency='foo',
    wanted_words=new Regexp(/^abcd|efgh|xyz|pqrs|values$/);

agency.match(wanted_words) && map.entities.push(eval(agency));

假设代理的所有可能值都作为变量存在。