Question

我试图创建一个从多维数组中删除空值的函数，但它并没有很好地工作。它不会穿透到最后一层数组，并且在两个空值相邻时不会删除。

function isArray(obj) {
    // http://stackoverflow.com/a/1058753/1252748
    return Object.prototype.toString.call(obj) === '[object Array]';
}

function removeEmptyArrayElements(obj) {

    for (key in obj) {
        if (obj[key] === null) {

            obj = obj.splice(key, 1);

        }
        var isArr = isArray(obj[key]);
        if (isArr) {
            removeEmptyArrayElements(obj[key]);
        }

    }
    return obj;
}

Answer 1

在迭代对象时，请勿从对象中删除元素。

不要在数组上使用for ... in。

利用filter和map等高阶函数。试试这个：

function removeEmptyArrayElements(arr) { 
   if (!isArray(arr)) {
      return arr;
   } else {
       return arr.filter( function(elem) { 
          return elem !== null
       } ).map(removeEmptyArrayElements)
   }
}

说明：

“高阶”函数是使用其他函数作为值的函数 - 将它们作为参数接受，或者将它们作为参数接收，或者两者兼有。在这种情况下，我使用了两种Javascript的Array类方法，它接受一个函数作为参数，并用它来构建和返回一个新数组。

someArray.filter( function )在someArray的每个元素上调用 function 并返回一个只包含其元素的新数组 function 返回true。我在这种情况下传递的函数是function(elem) { return elem !== null } - 也就是说，如果元素不为null，则返回true的匿名函数。因此，filter调用的结果是删除了所有空值的原始数组。

现在我们要从结果的任何元素中删除任何空值，这些元素本身就是数组。我使用了map。 someArray.map( function )也会在数组的每个元素上调用 function ，但不会根据这些结果返回其输入的子集，而是只返回结果本身。例如，[1,2,3].map(function(x){return x*2})返回数组[2,4,6]。在这种情况下，我们传递的函数是我们正在定义的函数..递归调用回{{1}}。

我们确实需要一种方法来阻止这种无休止的自我调用循环，这就是外部removeEmptyElements的用途 - 当调用不是数组的元素时，我们只是按原样返回它。

Answer 2

您正在循环删除要删除的数组。这为您提供“当两个空值相邻时不会删除”错误。 “它没有穿透到最后一层数组”部分可能是因为你没有在obj[key]重新分配值。数组按值复制，未引用。

我会这样做：

function isArray(obj) {
    // http://stackoverflow.com/a/1058753/1252748
    return Object.prototype.toString.call(obj) === '[object Array]';
}

function removeEmptyArrayElements(arr) {
    var i;
    for (i = 0; i < arr.length; i += 1) {
        if (arr[i] === null) {
            arr = arr.splice(i, 1);
            i -= 1; // decrement counter!

        } else if (isArray(arr[i])) {
            arr[i] = removeEmptyArrayElements(arr[i]);
        }
    }
    return arr;
}

使用splice从数组中删除空值

2 个答案: