如何修改代码消除JSHint「循环内函数引用外部变量」警告？

嘿，我来帮你搞定这个JSHint的警告！这个提示本质上是在提醒你：在循环内部定义的函数如果引用了外部作用域的变量，在旧JS环境里可能因为变量提升导致语义混淆——虽然你的代码里引用的是外部函数的operator参数，理论上不会出问题，但JSHint还是会出于安全原则给出提示。下面给你几个靠谱的修改方案：

方案一：改用ES6块级作用域变量（最推荐） #

把代码里的var全部换成let/const，因为ES6的let/const有块级作用域，JSHint会识别到变量不会被意外共享，直接消除警告：

const getPrecedence = function getPrecedence(operator, operators) { 
    const keys = Object.keys(Object(operators)); 
    for (let i = 0, len = keys.length; i < len; i++) { 
        const check = Object.keys(operators[keys[i]]).some(function (item) { 
            return item === operator; 
        }); 
        if (check) return operators[keys[i]][operator]; 
    } 
};

方案二：将回调函数移出循环，避免重复创建 #

把some方法的回调函数提到循环外面，这样每次循环不会重新生成函数实例，同时也能让JSHint放下顾虑：

var getPrecedence = function getPrecedence(operator, operators) { 
    // 把回调函数抽离到外部，明确关联需要比较的operator
    function matchesOperator(item) {
        return item === operator;
    }
    var keys = Object.keys(Object(operators)); 
    for (var i = 0, len = keys.length; i < len; i++) { 
        var check = Object.keys(operators[keys[i]]).some(matchesOperator); 
        if (check) return operators[keys[i]][operator]; 
    } 
};

方案三：重构代码逻辑，简化循环（更优雅） #

直接用ES6的Object.entries遍历，配合in操作符判断operator是否存在，完全去掉显式for循环，代码更简洁，也从根源上消除了警告：

const getPrecedence = (operator, operators) => {
    for (const [_, ops] of Object.entries(operators)) {
        if (operator in ops) {
            return ops[operator];
        }
    }
};

这个版本不仅解决了警告，还让代码可读性更高，逻辑更清晰哦~

内容的提问来源于stack exchange，提问作者Kasia