Javascript 無限ループに入る

Question

質問失礼いたします。表題の通りjsでシステムを作っていた際に無限ループに入ってしまい、その原因と対処法がわからずにいます。原因と対処方法をご教授お願い申し上げます。

作りたいもの:
10, 11, 12 のように、1 の差で連続しているカードを 1 つのグループにする。例えば、5, 10, 11, 12, 24, 25 というカードがあったら「5」というグループと「10, 11, 12」というグループと「24, 25」というグループに分かれる。
・各グループのスコアは、グループ内の最大の数であり、総合スコアは各グループのスコアの和である。例えば上記の例では、「5」というグループのスコアは 5 で、「10, 11, 12」というグループのスコアは 12 で、「24, 25」というグループのスコアは 25 なので、総合スコアは 5+12+25 = 42 である。
上記のようなものを作成しております。

let lines1 = ["6", "5 10 11 12 24 25"]
    let arry1 = lines[1].split(" ");
    let num1 = parseInt(arry1[0]);
    for(let i = 1; i <= arry1.length; i++){
        arry1[i] = parseInt(arry1[i]);
        if(arry1[i+1] == false){
            num1 += arry1[i];
            break;
        }else if(arry1[i] - arry1[i-1] !== 1 && arry1[i] - arry1[i+1] !== 1){
            num1 = arry1[i];
        }
        
        console.log(num1);
    }

Answer 1

var a = [5, 10, 11, 12, 24, 25];
a[a.length] = parseInt(a[a.length]);
console.log(a); // [5, 10, 11, 12, 24, 25, NaN]
a[a.length] = parseInt(a[a.length]);
console.log(a); // [5, 10, 11, 12, 24, 25, NaN, NaN]

i===arry1.lengthのときにa[i]に代入することで配列の長さが増えていると思います。

var a = [5, 10, 11, 12, 24, 25];
console.log(a[a.length]===false); //false
console.log(a[a.length]===undefined); //true

Comments

[takasima20]

おーなるほど。そこで増えてたんスね～

[koko122102]

ありがとうございます！！となると、それを防ぐために下記のコードを記載しているのですが、これでは何かおかしいということなのでしょうか。
if(arry1[i+1] == false){
num1 += arry1[i];
break;
}

[querykuma]

配列の範囲外を参照したときの値はfalseではなくundefinedです。

Answer 2

const a = ["6", "5 10 11 12 24 25"];
const b = a[1].split(" ").map(Number);
const c = b.reduce((x,y)=>(x.splice(x.slice(-1)[0]==y-1?-1:x.length),[...x,y]),[]).reduce((x,y)=>x+y);
console.log(c);

Answer 3

エラーになるかと思ったら無限ループになるんですねえ

    for(let i = 0; i < arry1.length; i++){

いまのコードだと無条件に配列の範囲外を参照するのでなんとかしましょう。