从数组（forEach）生成代码块（函数、变量）？

Question

我有这段代码：

foo.bar('run', function () {       
  var text1 = foo.url('./src/url/text1.txt')
          .go(bar.do({ something }))
          .also(bar({
              variable: 1,
              else: 2,
          }).on('exception', doSomething ))
          .also(bar.url('.src/url/source'))

  var textLong = foo.url('./src/url/textLong.txt')
          .go(bar.do({ something }))
          .also(bar({
              variable: 1,
              else: 2,
          }).on('exception', doSomething ))
          .also(bar.url('.src/url/source'))

  var text = foo.url('./src/url/text.txt')
          .go(bar.do({ something }))
          .also(bar({
              variable: 1,
              else: 2,
          }).on('exception', doSomething ))
          .also(bar.url('.src/url/source'))

  return myCustomFunction(text1, textLong, text);
}

您可以看到它尽可能地抗 DRY，每个变量都是相同的，除了变量名和文件名 (foo.url)。

我一直想知道是否有可能使它更简单，例如：

var files = [ 'text1', 'textLong', 'text'];

然后：

foo.bar('run', function () {    
  files.forEach(function(fileName){
        var fileName = foo.url('./src/url/'+ fileName + '.txt')
        .go(bar.do({ something }))
        .also(bar({
            variable: 1,
            else: 2,
        }).on('exception', doSomething ))
        .also(bar.url('.src/url/source'))
  })

  return myCustomFunction(text1, textLong, text);
})

但我可以通过第二种方法获得的最佳效果是text1 is not defined。 SO上有一些关于动态创建变量的问题，但它们通常具有相同的名称+我从未见过它们与自定义函数结合使用。

有什么提示吗？

Answer 1

我已经用来自 Ates Gora、Thomas、Bergi 的提示更新了这个答案。

使用 ES2016 中实现的一些功能，解决方案可能如下所示：

foo.bar('run', function () {

  return myCustomFunction(

    ...[ 'text1', 'textLong', 'text' ].map(fileName => {

      return foo.url(`./src/url/${fileName}.txt`)
        .go(bar.do({ something }))
        .also(bar({
          variable: 1,
          else: 2,
        })
        .on('exception', doSomething ))
        .also(bar.url('.src/url/source'));

     });

  );

});

Answer 2

您可以使用.map() 收集对数组中的项目执行操作的结果，然后使用.apply() 使用一组可能的参数调用函数：

foo.bar('run', function () {    
  var results = files.map(function(fileName){
    return foo.url('./src/url/'+ fileName + '.txt')
      .go(bar.do({ something }))
      .also(bar({
        variable: 1,
        else: 2,
      })
      .on('exception', doSomething ))
      .also(bar.url('.src/url/source'));
  });

  return myCustomFunction.apply(null, results);
});

Answer 3

永远不要使用动态变量。您想改用数组：

foo.bar('run', function () {    
    var files = [ 'text1', 'textLong', 'text'];
    var texts = files.map(function(fileName) {
//                    ^^^ get back the results
        return foo.url('./src/url/'+ fileName + '.txt')
//      ^^^^^^ don't assign to anything
        .go(bar.do({ something }))
        .also(bar({
            variable: 1,
            else: 2,
        }).on('exception', doSomething ))
        .also(bar.url('.src/url/source'))
  })

  return myCustomFunction(texts[0], texts[1], texts[2]);
})