JavaScript闭包结合事件使用遇问题，请求代码排查与修正

你的代码存在三个核心问题，导致无法实现问候语切换功能：

错误点与修复说明 #

• DOM内容更新方式错误：document.getElementById("demo")返回的是DOM元素对象，不能直接赋值字符串，必须通过textContent（纯文本场景推荐）或innerHTML属性修改显示内容。
• 状态未切换：闭包中的on_or_off变量没有在点击后翻转状态，导致每次点击都只会执行if分支，永远显示"Hello"。
• 变量名冲突风险：全局变量click可能和浏览器原生的click事件名冲突，建议改名规避。

修正后的完整代码 #

JavaScript部分

function change_greeting() {
  let on_or_off = true; // 用布尔值比数字1更符合语义
  return function button() {
    // 缓存DOM元素，避免重复查找提升性能
    const demoElement = document.getElementById("demo");
    if (on_or_off) {
      demoElement.textContent = "Hello";
    } else {
      demoElement.textContent = "Goodbye";
    }
    // 每次点击后切换状态
    on_or_off = !on_or_off;
  }
}
// 改名避免全局变量冲突
const clickHandler = change_greeting();

HTML部分

<button onclick="clickHandler()">Click Me</button>
<p id="demo"></p>

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