我有以下for循环，当我使用splice()删除一个项目时，我得到'seconds'是未定义的。我可以检查它是否未定义，但我觉得可能有一种更优雅的方式来做到这一点。他们的愿望是简单地删除一个项目，然后继续前进。
for (i = 0, len = Auction.auctions.length; i < len; i++) {
auction = Auction.auctions[i];
Auction.auctions[i]['seconds'] --;
if (auction.seconds < 0) {
Auction.auctions.splice(i, 1);
}
}
当前回答
普通的for循环对我来说更熟悉，我只需要在每次从数组中删除一个项时递减索引
//5个正确，5个错误 Var arr1 =[假，假，真，真，假，真，假，真，真，假]; //从数组中删除false For (var I = 0;I < arr1.length;我+ +){ If (arr1[i] === false){ arr1。拼接(我,1); I——;//如果item被移除，则递减索引 ｝ ｝ Console.log (arr1);//应该是5个true
其他回答
重新计算每次循环的长度，而不是一开始就重新计算，例如:
for (i = 0; i < Auction.auctions.length; i++) {
auction = Auction.auctions[i];
Auction.auctions[i]['seconds'] --;
if (auction.seconds < 0) {
Auction.auctions.splice(i, 1);
i--; //decrement
}
}
这样就不会超过上界。
EDIT:在if语句中增加了一个减量。
当您执行.splice()时，数组正在重新索引，这意味着当一个索引被删除时，您将跳过一个索引，并且缓存的.length已过时。
要修复它，你要么需要在.splice()后面递减i，要么简单地反向迭代…
var i = Auction.auctions.length
while (i--) {
...
if (...) {
Auction.auctions.splice(i, 1);
}
}
这样，重新索引就不会影响迭代中的下一项，因为索引只影响从当前点到数组末尾的项，并且迭代中的下一项低于当前点。
为什么在.splice上浪费CPU周期?该操作必须一次又一次地执行整个循环以删除数组中的一个元素。
为什么不只是在一个循环中使用传统的2个旗帜?
Const元素= [1,5,5,3,5,2,4]; Const remove = 5 I = 0 For(令j = 0;J < elements.length;j + +) { If(元素[j] !==删除){ 元素[i] =元素[j] 我+ + ｝ ｝ 元素。长度= I
如果你正在使用ES6+ -为什么不直接使用数组。过滤方法?
Auction.auctions = Auction.auctions.filter((auction) => {
auction['seconds'] --;
return (auction.seconds > 0)
})
注意，在过滤器迭代过程中修改数组元素只对对象有效，而对基元值数组无效。
普通的for循环对我来说更熟悉，我只需要在每次从数组中删除一个项时递减索引
//5个正确，5个错误 Var arr1 =[假，假，真，真，假，真，假，真，真，假]; //从数组中删除false For (var I = 0;I < arr1.length;我+ +){ If (arr1[i] === false){ arr1。拼接(我,1); I——;//如果item被移除，则递减索引 ｝ ｝ Console.log (arr1);//应该是5个true
