JavaScript新手求优化：判断数字能否被7和11整除的代码改进建议

原始代码 #

//TASK 5 PART 1

//Request input from user
let x   = Number(prompt("Enter a number:"));

// using if...else if
if      (x % 7 === 0 && 
         x % 11 === 0) {
         console.log
        (x + " is divisible by 7 and 11");
}

else if (x % 7 === 0 || 
         x % 11 === 0) {
         console.log
        (x + " is divisible by either 7 or 11");
}

else                   {
         console.log
        (x + " is divisible by neither 7 or 11");
}

需求说明 #

我是JavaScript新手，编写了上述代码用于提示用户输入数字，判断其是否同时被7和11整除、仅被其中之一整除或都不整除。现寻求代码优化方案，包括精简行数、提升效率、增强可读性，以及排查潜在错误，希望得到建设性改进意见。感谢各位的时间。

一、潜在错误排查 #

• 非数字输入无处理：若用户输入字母、符号或空值，Number(prompt())会返回NaN，后续取模判断全部失效，直接进入else分支导致错误判断。
• 语法表述错误：最后一句提示的neither 7 or 11是语法错误，正确表述应为neither 7 nor 11。
• 冗余函数写法：console.log换行调用属于不必要的写法，直接写成一行更符合规范。

二、可读性与精简优化 #

1. 提前存储整除状态：把x%7===0和x%11===0的结果存为变量，避免重复计算，同时让判断逻辑更直观。
2. 使用模板字符串：替代传统字符串拼接，写法更简洁，可读性更强。
3. 统一代码格式化：保持缩进一致（推荐2或4空格），去掉不必要的换行，让代码结构更清晰。
4. 调整分支顺序：把“同时整除”的判断放在最前面，逻辑顺序更符合常规思考路径。

三、效率优化 #

核心优化点是避免重复取模运算：原始代码中x%7和x%11各计算了两次，提前存为变量后仅需计算一次，虽对小数字影响不大，但能养成高效编码的习惯。

优化后的完整代码 #

// 处理用户输入并验证有效性
const input = prompt("Enter a number:");
const x = Number(input);

// 提前计算整除状态，减少重复运算
const divisibleBy7 = x % 7 === 0;
const divisibleBy11 = x % 11 === 0;

// 先判断输入是否为有效数字
if (isNaN(x)) {
  console.log("Please enter a valid number!");
} else if (divisibleBy7 && divisibleBy11) {
  console.log(`${x} is divisible by both 7 and 11`);
} else if (divisibleBy7 || divisibleBy11) {
  console.log(`${x} is divisible by either 7 or 11`);
} else {
  console.log(`${x} is divisible by neither 7 nor 11`);
}

额外小建议 #

• 若面向普通用户，可用alert替代console.log，让结果直接展示在页面弹窗中，无需打开浏览器控制台。
• 可优化提示语，比如prompt("Please enter an integer:")，引导用户输入正确类型的内容。

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