本文详解因函数参数名与同名函数发生作用域遮蔽（shadowing），导致 `typeerror: cityname is not a function` 的根本原因，并提供可立即生效的修复方案与健壮性增强建议。

在 Node.js 命令行交互应用中，使用 readline-sync 实现用户反复输入城市名并查询天气的流程时，一个常见但隐蔽的错误是：将异步回调函数的形参命名为与外部函数同名（如 cityname），从而在函数作用域内覆盖（shadow）了同名函数声明。这会导致在 switch 分支中调用 cityname() 时，JavaScript 引擎误将该标识符解析为字符串参数而非函数，最终抛出 TypeError: cityname is not a function。

问题核心在于以下代码片段：

function getWeatherData(cityname, apiKey) { // ❌ 参数名 "cityname" 遮蔽了外部函数 cityname()
  axios.get(`...${cityname}&appid=${apiKey}`)
    .then((res) => {
      // ...
      switch (options) {
        case 1: cityname(); // ⚠️ 此处 cityname 指的是字符串参数，不是函数！
      }
    });
}

由于 JavaScript 的函数作用域规则，形参 cityname 在 getWeatherData 函数体内具有最高优先级，外部同名函数 cityname() 在此作用域中不可访问——这就是典型的 标识符遮蔽（Identifier Shadowing）。

✅ 正确做法是：为参数选用语义清晰且不与函数名冲突的名称，例如 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(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.');
          break;
      }
    })
    .catch((err) => {
      console.error('Failed to fetch weather data:', err.message);
      // 可选：失败后仍允许重试
      cityname();
    });
}

? 额外建议（提升健壮性）：

• 添加 .catch() 处理网络请求失败（如城市不存在、API 错误），避免程序静默终止；
• 使用 const 替代 var 声明变量，避免变量提升与作用域混淆；
• 考虑将递归调用包装为 process.nextTick(cityname) 或 setTimeout(cityname, 0)，防止深层递归引发栈溢出（虽本例深度可控，但属良好实践）；
• 对用户输入做基础校验（如非空、长度限制），提升交互体验。

通过规避命名冲突，不仅解决了当前错误，更强化了代码的可读性与可维护性——这是编写清晰、可靠 CLI 工具的重要基石。