Question

我正在尝试从作为参数传递的函数的数组中删除元素。我试图从数组中删除的元素是等于函数的以下参数的元素。像这样：destroyer([1, 2, 3, 1, 2, 3], 2, 3);所以如果数组（驱逐舰的第一个参数）包含“2”和“3”，我希望它们被删除。所以调用函数应该返回[1,1]。

function destroyer(arr) {

  var args = [];

  for(var j = 1; j<arguments.length; j++){

      args.push(arguments[j]);

    for(var i = 0; i<arr.length; i++){

        if(args.indexOf(arr[i]) > -1){

            arr.splice(i, 1)
        }
    }
  }
  return arr;
}

destroyer([1, 2, 3, 1, 2, 3], 2, 3);

我不明白为什么我的代码适用于destroyer([1, 2, 3, 1, 2, 3], 2, 3);（它返回[1,1]）但不适用于destroyer([2, 3, 2, 3], 2, 3);（它应该返回[]，但返回[3]）。

请帮助我理解为什么这不起作用。

我在https://repl.it/BTu5

创建了一个repl.it

Answer 1

在循环播放集合时更新集合是一种糟糕的编程习惯：对#available的调用可能会更改arr.splice()的值。尝试添加arr.length语句，以查看程序运行时会发生什么。

这是一个从输入数组中删除不需要的元素的版本

debugger;

Answer 2

不正确的答案是对的，但是我得到了一些有用的东西，它与问题的代码更相似，帮助他理解：代码：

 function destroyer(arr) {

 var args = [];
  //You want add all the nums you want remove from array, so you start from 1, which means second arg,first arg is the array you want to perform
  for(var j = 1; j<arguments.length; j++){ 
  //You store the args to an arg array
      args.push(arguments[j]);
    //Once you have the arg, you want loop the target array, see if the newly added arg exist in the target array, if it is, then remove it
    for(var i = 0; i<arr.length; i++){
        //Found it, remove it now! note: your found the index, then you need take it out, check the doc for slice function arr.slice([begin[, end]]) https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/slice
        if(args.indexOf(arr[i]) > -1){
            //key change, remove the element 
            arr.splice(i, i+1)
           }
       }
   }
  return arr;
  }
 destroyer([2, 3, 2, 3], 2, 3);

`

另外，为你创造了一个游戏

https://repl.it/BTu5/5

示例：slice（1,4）将第二个元素提取到第四个元素（元素索引为1,2和3）。

Answer 3

你为什么不只是.filter（）？

function destroyer(arr) {
  var badValues = Array.prototype.slice.call(arguments); //Construct an array of arguments
  badValues.shift(); //Get rid of the first ("good") array element
  return arr.filter(function(x) {return badValues.indexOf(x) == -1;}); //Make "bad" values false and thus — filter them.
}

destroyer([1, 2, 3, 1, 2, 3], 2, 3);

Answer 4

我已经使用过滤功能完成了这项挑战，但建议也使用＆＃39; indexOf＆＃39;为了比较数组中的值与要过滤的值。试试这个） ````

function destroyer(arr) {
    // Remove all the values
    var temp = [];
    for (var i = 1; i < arguments.length; i++) {
        temp.push(arguments[i]);
        arr = arguments[0].filter(function(value) {
            return ( value !== temp[i - 1]) ;
        });
    }
    return arr;
}
destroyer([1, 2, 3, 1, 2, 3], 2, 3);

Answer 5

我使用Arguments Object和Array.prototype.filter()在freeCodeCamp完成了这个基本算法脚本编写挑战。希望这有帮助

function destroyer(arr) {
var args = [];
//Creating array of arguments
for(var j=1;j<arguments.length;j++){
    args.push(arguments[j]);
}
//Filtering the arguments from array
arr = arr.filter(function(val){
    if(args.indexOf(val)==-1)
        return true;
return false;
});
return arr;

}

驱逐舰（[1,2,3,1,2,3]，2,3）;

如何使用arguments对象从数组中删除元素 - 我的代码出了什么问题？

5 个答案: