我有以下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 (var i = Auction.auctions.length - 1; i >= 0; i--) {
Auction.auctions[i].seconds--;
if (Auction.auctions[i].seconds < 0) {
Auction.auctions.splice(i, 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语句中增加了一个减量。
另一个简单的方法是一次消化数组元素:
while(Auction.auctions.length){
// From first to last...
var auction = Auction.auctions.shift();
// From last to first...
var auction = Auction.auctions.pop();
// Do stuff with auction
}
当您执行.splice()时，数组正在重新索引，这意味着当一个索引被删除时，您将跳过一个索引，并且缓存的.length已过时。
要修复它，你要么需要在.splice()后面递减i，要么简单地反向迭代…
var i = Auction.auctions.length
while (i--) {
...
if (...) {
Auction.auctions.splice(i, 1);
}
}
这样，重新索引就不会影响迭代中的下一项，因为索引只影响从当前点到数组末尾的项，并且迭代中的下一项低于当前点。
试试吧
RemoveItems.forEach((i, j) => {
OriginalItems.splice((i - j), 1);
});
下面是另一个正确使用拼接的例子。本例将从'array'中删除'attribute'。
for (var i = array.length; i--;) {
if (array[i] === 'attribute') {
array.splice(i, 1);
}
}
