当函数参数与外部同名函数重名时，会形成变量遮蔽（shadowing），导致递归调用时报错“typeerror: cityname is not a function”。本文详解该问题成因，并提供可直接运行的修复方案。
在 Node.js 命令行应用中，使用 readline-sync 与 axios 实现交互式天气查询时，一个常见却隐蔽的错误是：将函数参数命名为与外部函数同名的标识符，从而意外覆盖（shadow）了函数声明本身。
在原始代码中，getWeatherData 函数接收的第一个参数名为 cityname：
function getWeatherData(cityname, apiKey) { ... }而与此同时，外部已定义同名函数 cityname()。由于 JavaScript 的作用域规则，该参数 cityname 在函数体内优先级高于同名函数声明，因此当执行 case 1: cityname(); 时，JavaScript 尝试调用的是参数值（一个字符串），而非函数 —— 自然抛出 TypeError: cityname is not a function。
✅ 正确做法是：为参数选择语义清晰且不与函数名冲突的名称，例如 cityName、city 或 myCity。以下是修复后的完整可运行代码（已优化可读性与健壮性）：
import axios from "axios";
import readlineSync from "readline-sync";
// 启动入口
cityname();
function cityname() {
const apiKey = 'xyz';
const name = readlineSync.question("Enter City Name:\n");
return getWeatherData(name, apiKey);
}
function getWeatherData(cityName, apiKey) {
axios
.get(`https://api.openweathermap.org/data/2.5/weather?q=${cityName}&appid=${apiKey}`)
.then((res) => {
const data = res.data;
const options = readlineSync.questionInt(
`Select an option below for ${cityName}:\n` +
`1. Change City\n` +
`0. Exit\n`
);
switch (options) {
case 1:
cityname(); // ✅ 此时 cityname 指向函数，无遮蔽
break;
case 0:
console.log("Exiting the program...");
break;
default:
console.log("Invalid option. Please try again.");
// 可选：递归重试当前菜单，避免退出
getWeatherData(cityName, apiKey);
break;
}
})
.catch((err) => {
console.error("Failed to fetch weather data:", err.message);
console.log("Please check the city name and try again.");
cityname(); // 查询失败也允许重新输入城市
});
}? 关键注意事项：
- 永远避免参数/变量名与函数名相同，尤其在需递归调用的场景；
- switch 中每个 case 后务必显式写 break（原代码 case 1 缺失 break，会导致穿透执行 case 0）；
- 添加 .catch() 处理网络或 API 错误（如城市不存在），提升用户体验；
- 若用户输入无效选项，建议保持当前上下文（如再次调用 getWeatherData）而非强制退出，增强交互友好性。
通过命名隔离与错误防御，即可安全实现“输入城市 → 查天气 → 重选城市”的闭环递归逻辑。
