我有以下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);
}
}
当前回答
虽然你的问题是关于从被迭代的数组中删除元素，而不是关于有效地删除元素(除了一些其他处理)，但我认为如果遇到类似情况，应该重新考虑它。
这种方法的算法复杂度是O(n^2)作为拼接函数和for循环都遍历数组(在最坏的情况下，拼接函数移位数组的所有元素)。相反，您可以将所需的元素推入到新数组中，然后将该数组赋值给所需的变量(该变量刚刚被迭代)。
var newArray = [];
for (var i = 0, len = Auction.auctions.length; i < len; i++) {
auction = Auction.auctions[i];
auction.seconds--;
if (!auction.seconds < 0) {
newArray.push(auction);
}
}
Auction.auctions = newArray;
自ES2015以来，我们可以使用Array.prototype.filter将所有内容都放在一行中:
Auction.auctions = Auction.auctions.filter(auction => --auction.seconds >= 0);
其他回答
试试吧
RemoveItems.forEach((i, j) => {
OriginalItems.splice((i - j), 1);
});
重新计算每次循环的长度，而不是一开始就重新计算，例如:
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语句中增加了一个减量。
你可以浏览一下，然后使用shift()
如果你正在使用ES6+ -为什么不直接使用数组。过滤方法?
Auction.auctions = Auction.auctions.filter((auction) => {
auction['seconds'] --;
return (auction.seconds > 0)
})
注意，在过滤器迭代过程中修改数组元素只对对象有效，而对基元值数组无效。
尝试在循环时将数组中继到newArray:
var auctions = Auction.auctions;
var auctionIndex;
var auction;
var newAuctions = [];
for (
auctionIndex = 0;
auctionIndex < Auction.auctions.length;
auctionIndex++) {
auction = auctions[auctionIndex];
if (auction.seconds >= 0) {
newAuctions.push(
auction);
}
}
Auction.auctions = newAuctions;
