javascript - JS ヒント - ループ内で関数を作成しないでください

Question

JSHint のエラー メッセージを回避できません。私が使用しているループは次のとおりです。

for (i = 0; i < Collection.length; i += 4) {
    data.push({
        items : Collection.slice(i, i + 4).map(function(item) {
            return {
                id: item[0],
                title: item[1],
            };
        })
    });
}

Answer 1

関数をループの外側に移動して、関数への参照を に渡すだけですmap。

function mapCallback(item) {
    return {
        id : item[0],
        title : item[1],
    };
}
for (i = 0; i < Collection.length; i += 4) {
    data.push({
        items: Collection.slice(i, i + 4).map(mapCallback)
    });
}

または、JSHint ディレクティブを使用して、ループ内の関数式を無視することもできます。これを問題のファイルの先頭に置くだけです：

/*jshint loopfunc: true */

Answer 2

ループ内で関数を宣言するのは面倒で、エラーが発生しやすい可能性があります。代わりに、関数を一度定義してから、ループに入ります。

var objMaker = function(item) {
    return {
        id : item[0],
        title : item[1],
    };
};

for (i = 0; i < Collection.length; i += 4) {
    data.push({
                  items : Collection.slice(i, i + 4).map(objMaker)
             });
}