V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
sudoy
V2EX  ›  问与答

JavaScript 循环问题

  •  
  •   sudoy · 2020-11-28 14:45:57 +08:00 · 1916 次点击
    这是一个创建于 1217 天前的主题,其中的信息可能已经有所发展或是发生改变。

    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]
    
    21 条回复    2020-11-30 14:37:00 +08:00
    noe132
        1
    noe132  
       2020-11-28 14:55:17 +08:00   ❤️ 1
    边循环边改数组是大忌。
    des
        2
    des  
       2020-11-28 14:59:27 +08:00   ❤️ 1
    不要在循环中修改数组!不要在循环中修改数组!不要在循环中修改数组!
    sudoy
        3
    sudoy  
    OP
       2020-11-28 15:00:19 +08:00
    @noe132 嗯,index 好像出问题了。不过我需要做到一旦匹配到就要移除掉,避免重复匹配。所以不得不从数组里面移除掉
    sugarsalt
        4
    sugarsalt  
       2020-11-28 15:07:22 +08:00   ❤️ 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 了
    hoyixi
        5
    hoyixi  
       2020-11-28 15:08:26 +08:00   ❤️ 1
    filter 一下,返回一个新数组好了,有 JS 内置实现
    luob
        6
    luob  
       2020-11-28 15:08:35 +08:00
    你这不都找到原因了吗……
    aaronlam
        7
    aaronlam  
       2020-11-28 15:12:34 +08:00 via iPhone
    实在不济,你在外面搞个新数组,不就好了。。
    zackwan95
        8
    zackwan95  
       2020-11-28 15:19:08 +08:00 via iPhone   ❤️ 1
    都 ES2020 了,用 array 内置的 api 吧,这玩意用 reduce 正合适
    Caballarii
        9
    Caballarii  
       2020-11-28 16:18:13 +08:00
    从右向左循环就 ok 了
    xiangyuecn
        10
    xiangyuecn  
       2020-11-28 16:22:52 +08:00
    这玩意跟语言无关, 什么 filter ES2020 “不要在循环中修改数组”,没有一毛钱关系
    TommyDx
        11
    TommyDx  
       2020-11-28 16:29:34 +08:00
    两边一起减,少了循环次数
    sudoy
        12
    sudoy  
    OP
       2020-11-28 17:59:14 +08:00 via iPhone
    谢谢大家回复,已经找到解决办法了:就是搞个新的数组
    ddsfeng
        13
    ddsfeng  
       2020-11-28 18:08:59 +08:00   ❤️ 2
    倒序遍历 可解.
    sudoy
        14
    sudoy  
    OP
       2020-11-28 23:43:51 +08:00
    @ddsfeng 攒👍
    chenyu8674
        15
    chenyu8674  
       2020-11-28 23:59:21 +08:00   ❤️ 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;
    };
    };
    chenyu8674
        16
    chenyu8674  
       2020-11-29 00:03:07 +08:00
    @chenyu8674 #15 困脑抽了,i--就行
    lbyo
        17
    lbyo  
       2020-11-29 00:47:22 +08:00   ❤️ 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);
    };
    }
    ```
    autoxbc
        18
    autoxbc  
       2020-11-29 03:54:34 +08:00   ❤️ 1
    for (i = 0; i < price.length; ) {
      if (price[i] == 'apple') {
      ...
     } else {
       i++
     }
    };

    只需要把 for 句首的 i++ 转移到 else 分支里就够了
    sudoy
        19
    sudoy  
    OP
       2020-11-29 17:41:26 +08:00 via iPhone
    @autoxbc 谢谢
    pwn
        20
    pwn  
       2020-11-30 10:11:14 +08:00   ❤️ 1
    不止 JavaScript,任何语言(大概)都是如此,price.length 是每次循环开始的时候会计算的,也就是说在修改了数组 price 之后,price.length 是会变的!会变的!
    我通常是在循环体外面把这个值固定下来,let length = price.length,然后才开始循环。
    sudoy
        21
    sudoy  
    OP
       2020-11-30 14:37:00 +08:00
    @autoxbc 我最后用了这个方法😍
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   3400 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 32ms · UTC 10:39 · PVG 18:39 · LAX 03:39 · JFK 06:39
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.