Node.js中数字校验代码报错的原因及修复方法

嗨，我来帮你分析并解决这个问题～

报错的根本原因 #

你代码里的问题出在对Number.isInteger()方法的使用上！这个方法有个关键特性：它只会对「本身就是Number类型的整数」返回true。

看你的代码，test_port是字符串类型的"443"，哪怕它的内容是数字，直接传给Number.isInteger()也会返回false。所以你的判断条件!(Number.isInteger(test_port))就会变成true，直接触发错误抛出逻辑，这就是为什么你会看到Invalid port: "443"的报错。

修复方法 #

我们需要先把字符串形式的端口号转换成数字类型，再进行校验。这里有两种严谨的修复方案：

方案1：基础修复（适用于确定输入是数字字符串的场景）

先将字符串转为数字，再用Number.isInteger()判断：

let test_port = "443";
// 先把字符串端口转为数字类型
const portNum = Number(test_port);
// 再判断是否为正整数
if (!(Number.isInteger(portNum)) || portNum <= 0) {
  throw new Error(`Invalid port: "${test_port}"`);
}

方案2：增强版修复（兼容非数字输入的场景）

如果需要处理比如test_port = "abc"这类无效输入，我们可以新增isNaN()检查，排除转换失败的情况：

let test_port = "443";
const portNum = Number(test_port);
// 先检查转换是否有效（排除NaN），再判断是否为正整数
if (isNaN(portNum) || !Number.isInteger(portNum) || portNum <= 0) {
  throw new Error(`Invalid port: "${test_port}"`);
}

额外小提示 #

如果你用parseInt转换的话，要注意它会自动截断非数字部分（比如parseInt("443abc")会返回443），如果需要严格校验输入必须是纯数字字符串，用Number()转换配合isNaN()会更合适哦～

备注：内容来源于stack exchange，提问作者stackbiz