如何优化代码?

Bat*_*lug 0 javascript

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

    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 ""
    },
Run Code Online (Sandbox Code Playgroud)

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

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 ""
},
Run Code Online (Sandbox Code Playgroud)

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

Phi*_*hil 6

比连接和截断容易得多......

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

  • @ K3N这是一种新的`Array.prototype`方法.它们是迭代数组以获取特定值的新的最佳方式.您可以期望它们在大多数实现中针对速度进行优化,因为它们是内置的,并且将来会经常使用它们.因此,`map`从长远来看将具有非常快的性能. (2认同)