本文详解因函数参数名与函数名同名引发的变量遮蔽（shadowing）问题，导致 `typeerror: cityname is not a function` 错误，并提供可立即使用的修复方案与最佳实践。
在 Node.js 命令行应用中，使用 readline-sync 实现交互式城市天气查询时，若在异步回调中递归调用同名函数（如 cityname()），却意外报错 TypeError: cityname is not a function，根本原因并非作用域缺失或声明提升问题，而是参数名与函数名冲突造成的变量遮蔽（variable shadowing）。
JavaScript 中，当函数形参名为 cityname 时，该参数会在整个函数作用域内优先于同名的外部函数声明。因此，在 getWeatherData(cityname, apiKey) 内部，cityname 指向的是传入的字符串参数（如 "Beijing"），而非全局函数 cityname() —— 导致 case 1: cityname(); 实际尝试调用一个字符串，从而抛出类型错误。
✅ 正确修复方式：重命名冲突参数，避免与函数名重复。例如将 cityname 参数改为 myCity、city 或 cityNameInput 等语义清晰且无歧义的名称。
以下是修复后的完整可运行代码（已优化可读性与健壮性）：
分析 Java Spring Boot 后端 Controller 代码，自动生成 Markdown 格式的接口文档。当用户说“生成 Java 接口文档”、“生成 API 文档”、“从 Spring 代码生成文档”时触发。
立即学习“Java免费学习笔记（深入）”；
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(city, apiKey) { // ✅ 参数名已更正：city ≠ cityname
axios
.get(`https://api.openweathermap.org/data/2.5/weather?q=${city}&appid=${apiKey}`)
.then((res) => {
const data = res.data;
const options = readlineSync.questionInt(
`Select an option below for ${city}:\n` +
`1. Change City\n` +
`0. Exit\n`
);
switch (options) {
case 1:
cityname(); // ✅ 此处正确调用函数，无遮蔽
break;
case 0:
console.log('Exiting the program...');
break;
default:
console.log('Invalid option. Please try again.');
// 可选：递归重试当前菜单，而非退出
getWeatherData(city, apiKey);
break;
}
})
.catch((error) => {
console.error('Failed to fetch weather data:', error.message);
console.log('Please check the city name and try again.');
cityname(); // 网络失败时也允许重新输入城市
});
}⚠️ 注意事项：
- 切勿在函数参数、let/const 声明中复用已有函数名，这是 JS 作用域规则的常见陷阱；
- switch 语句中每个 case 后务必显式添加 break（原代码 case 1 缺失 break，会导致穿透执行 case 0）；
- 异步操作（如 axios.get）中调用递归函数是安全的，但需确保错误处理到位，避免未捕获异常中断流程；
- 为提升用户体验，建议在 catch 块中也触发 cityname()，使网络错误或无效城市名后仍可重试。
总结：命名一致性固然重要，但语义明确性与作用域安全性更为关键。将参数命名为 city 而非 cityname，既消除遮蔽风险，又更准确表达其数据类型（字符串），是兼顾可维护性与健壮性的最佳实践。
