如何修复因参数名冲突导致的递归函数调用失败问题

当函数参数与外部同名函数重名时，会形成变量遮蔽（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）而非强制退出，增强交互友好性。

通过命名隔离与错误防御，即可安全实现“输入城市 → 查天气 → 重选城市”的闭环递归逻辑。