Question

我有两个基本相同的函数 - 它们遍历一些数组。不同之处在于循环内部发生的事情。

    func1: function () {
        var result = "";
        for (var i = 0; i < array1.length; i++) {
            result = result + array1[i].field1 + ', ';

        }
        if (result.length > 0) {
            return result.substring(0, result.length - 2);
        } else return ""
    },

    func2: function () {
        var result = "";
        for (var i = 0; i < array2.length; i++) {
            result = result + array2[i].field2 + ', ';

        }
        if (result.length > 0) {
            return result.substring(0, result.length - 2);
        } else return ""
    },

说它只是一个函数的正确方法是什么，或者让它们都继承一些公共代码？我看到一个解决方案，使它只是一个功能，并提供不同的参数：

func: function (array, fieldName) {
    var result = "";
    for (var i = 0; i < array.length; i++) {
        result = result + array[i].[fieldName] + ', ';
    }
    if (result.length > 0) {
        return result.substring(0, result.length - 2);
    } else return ""
},

但有些东西告诉我它很难看。什么是正确的方法呢？

Answer 1

比连接和截断更容易......

func: function(array, fieldName) {
    return array.map(a => a[fieldName]).join(', ');
}

Answer 2

旧学校：

function pluckAndJoin(arr, attr) {
  var i,
      len = arr.length,
      res = [];
  for(i=0; i<len; i++){
        res.push(arr[i][attr]);
  }
  return res.join(', ');
}

console.log(pluckAndJoin([{name:'Hello'}, {name:'World'}], 'name'));

如何优化代码？

2 个答案: