[WIP] 展開されているサブカテゴリノード数の制限

[daiiz]

No description provided.

[daiiz]

論理削除だと少し遅い

[mizunashi-mana]

https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach

forEachでは、thisObjectを二番目の引数で指定することができます。
thisの参照を増やさないで、この機能は積極的に使っていきましょう

[daiiz]

なるほど，ありがとうございます。

この部分は，下のコードをbabelで変換したら出てきたのもので，アロー関数を使えば上手くthisを補足してくれるものと思っていたのですが，

appLoad () {
    miil_categories.forEach(cate => {
        if (cate.category_id !== 588 && cate.category_id !== 589) {
            this.parseMiilCategories(cate, [this.getNodeIdxById('miilroot')]);
        }
    });
    this.drawGraph();
}

やはり，

arr.forEach(elem, self => { });

という2引数で書いた方が良いということですね。

[mizunashi-mana]

なるほど、babelで変換したのですね。

通常は、そういう場合for of構文を使います。

https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Statements/for...of

[1, 2, 3].forEach(elem => console.log(this.a));

は、

for (let elem of [1,2,3]) {
  console.log(this.a);
}

と同値です。インデックスをとらないようなforEachをbabelで使うときは、for ofを使っていきましょう。

ついでに、forEachよりJSではインデックスアクセスの方がArrayオブジェクトにとっては効果的（まず、関数オブジェクトがforEachは必ず作成されますし、関数呼び出しのオーバーヘッドもかかるので）なため、for ofの場合、for(var i = 0; ...) { ... }といった感じで変換されます。

https://babeljs.io/repl/#?experimental=false&evaluate=true&loose=false&spec=false&code=function%20A%28%29%20{%0A%0Afor%20%28let%20i%20of%20[1%2C2%2C3]%29%20{%0A%20%20console.log%28this.a%29%3B%0A}%0A%0A}