This topic created in 1990 days ago, the information mentioned may be changed or developed.
JavaScript 循环好像哪里不对，哪位老哥帮看下。
price = ['apple', 10, 'orange', 6, 'apple', 10, 'apple', 10, 'banana', 5];
for (i=0; i<price.length; i++) {
if (price[i] == 'apple') {
pair = price.splice(i, 2);
console.log(i, pair);
console.log(price);
};
};
期望得到的结果是：
0 ["apple", 10]
["orange", 6, "apple", 10, "apple", 10, "banana", 5]
2 ["apple", 10]
["orange", 6, "apple", 10, "banana", 5]
2
["orange", 6, "banana", 5]
运行当前代码得到的结果是：
0 ["apple", 10]
["orange", 6, "apple", 10, "apple", 10, "banana", 5]
2 ["apple", 10]
["orange", 6, "apple", 10, "banana", 5]
 |
1
noe132 Nov 28, 2020  1
边循环边改数组是大忌。
|
 |
2
des Nov 28, 2020 1
不要在循环中修改数组！不要在循环中修改数组！不要在循环中修改数组！
|
 |
4
sugarsalt Nov 28, 2020 1
0 ["apple", 10]
["orange", 6, "apple", 10, "apple", 10, "banana", 5] 2 ["apple", 10] ["orange", 6, "apple", 10, "banana", 5] 此时 i 已经等于 2 了，再++就是 3，自然找不到 apple 了 |
 |
5
hoyixi Nov 28, 2020 1
filter 一下，返回一个新数组好了，有 JS 内置实现
|
 |
6
luob Nov 28, 2020
你这不都找到原因了吗……
|
 |
7
aaronlam Nov 28, 2020 via iPhone
实在不济，你在外面搞个新数组，不就好了。。
|
 |
8
bQ3u9RAQt4125PUK Nov 28, 2020 via iPhone 1
都 ES2020 了，用 array 内置的 api 吧，这玩意用 reduce 正合适
|
 |
9
Caballarii Nov 28, 2020
从右向左循环就 ok 了
|
 |
10
xiangyuecn Nov 28, 2020
这玩意跟语言无关， 什么 filter ES2020 “不要在循环中修改数组”，没有一毛钱关系
|
 |
11
TommyDx Nov 28, 2020
两边一起减，少了循环次数
|
 |
12
sudoy OP 谢谢大家回复，已经找到解决办法了：就是搞个新的数组
|
 |
13
xqin Nov 28, 2020 2
倒序遍历 可解.
|
 |
15
chenyu8674 Nov 28, 2020 1
price = ['apple', 10, 'orange', 6, 'apple', 10, 'apple', 10, 'banana', 5];
for (i=0; i<price.length; i++) { if (price[i] == 'apple') { pair = price.splice(i, 2); i -= 2; }; }; |
|
16
chenyu8674 Nov 29, 2020
@chenyu8674 #15 困脑抽了，i--就行
|
 |
17
hive Nov 29, 2020 1
@chenyu8674 #16
不光需要 i-- ``` let price = ['apple', 10, 'orange', 6, 'apple', 10, 'apple', 10, 'banana', 5]; for (i = price.length - 1; i >= 0; i--) { if (price[i] == 'apple') { pair = price.splice(i, 2); console.log(i, pair); console.log(price); }; } ``` |
 |
18
autoxbc Nov 29, 2020 1
for (i = 0; i < price.length; ) {
if (price[i] == 'apple') { ... } else { i++ } }; 只需要把 for 句首的 i++ 转移到 else 分支里就够了 |
 |
20
pwn Nov 30, 2020 1
不止 JavaScript，任何语言（大概）都是如此，price.length 是每次循环开始的时候会计算的，也就是说在修改了数组 price 之后，price.length 是会变的！会变的！
我通常是在循环体外面把这个值固定下来，let length = price.length，然后才开始循环。 |
Select Language