如何避免if/else条件直接用数字判断？是否有对应ESLint规则？

你遇到的问题本质是JS的隐式类型转换——数值0会被转为false，导致合法的0值被错误跳过。下面是几种实用的解决思路：

一、修正条件判断逻辑，明确区分合法值与空值 #

不要直接把数值变量丢进if条件，而是显式判断变量是否存在且合法：

• 若只需排除undefined和null：

  if (viewState.latitude !== undefined && viewState.latitude !== null) {
    params.set(QUERY_PARAM_LATITUDE_KEY, viewState.latitude.toFixed(4).toString());
  }
  

• 若还要排除NaN（比如接口返回的非法数值）：

  if (typeof viewState.latitude === 'number' && !isNaN(viewState.latitude)) {
    params.set(QUERY_PARAM_LATITUDE_KEY, viewState.latitude.toFixed(4).toString());
  }
  

• 现代JS环境可简化为可选链+空值合并：

  const lat = viewState.latitude;
  if (lat ?? false) { // 仅当lat不是null/undefined时执行
    params.set(QUERY_PARAM_LATITUDE_KEY, lat.toFixed(4).toString());
  }
  

二、自定义ESLint规则，从根源禁止这类写法 #

现成ESLint规则没有完全匹配的，你可以自己写个简单规则，强制禁止直接将数值变量作为if条件：

1. 新建自定义规则文件（比如rules/no-number-condition.js）：

module.exports = {
  meta: {
    type: 'problem',
    docs: {
      description: '禁止直接将数值类型变量/属性作为条件判断表达式',
      category: 'Possible Errors',
      recommended: true
    },
    schema: []
  },
  create(context) {
    return {
      IfStatement(node) {
        const test = node.test;
        // 匹配直接引用变量或对象属性的情况
        if (test.type === 'Identifier' || test.type === 'MemberExpression') {
          context.report({
            node: test,
            message: '请勿直接将数值类型变量作为条件，请显式写出判断逻辑'
          });
        }
      }
    };
  }
};

2. 在ESLint配置里引入这个规则：

{
  "plugins": ["your-custom-plugin"], // 若为单独插件包则配置，否则可直接用文件路径
  "rules": {
    "your-custom-plugin/no-number-condition": "error"
  }
}

如果是TS项目，结合@typescript-eslint/parser还能精准识别数值类型，只针对number类型变量触发警告，避免误报。

三、团队规范约束 #

• 统一编码约定：所有数值类型的条件判断必须显式写清逻辑，禁止依赖隐式转换。
• 代码评审时重点关注这类写法：看到if (num)或if (obj.num)就确认是否故意忽略0值，不是的话要求修改为显式判断。

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