复选框取消勾选未触发函数，如何移除ID生成目标URL？

首先咱们得先搞明白为啥取消勾选时alert没触发——你的代码逻辑分支走偏了！当前代码里，只有当destination不是null/local或者market没有dataid时，才会进入else if判断复选框状态。但你的HTML里a标签明确设置了data-destination="local"且带有dataid，所以这个else if分支永远不会被执行，alert自然弹不出来。

接下来针对你的需求（勾选带ID跳转，取消不带ID跳转），咱们一步步调整：

1. 修正JavaScript逻辑 #

调整判断顺序，先获取复选框状态，再根据状态决定是否携带ID，同时补上实际跳转的代码：

function Allchecks1(event) {
  // 阻止事件冒泡到a标签，避免触发a的默认跳转行为
  event.stopPropagation();
  
  const market = $(event.target).closest('a');
  const destination = market.attr('data-destination');
  const id = market.attr('dataid');
  
  // 先获取复选框当前状态
  const isChecked = $('#cb').is(':checked');
  
  // 仅在符合destination条件时处理跳转逻辑
  if ((destination === 'null' || destination === 'local') && id) {
    if (isChecked) {
      // 勾选时传入ID，跳转带ID的地址
      navigateToEvent(id);
    } else {
      // 取消勾选时不传ID，跳转基础地址
      navigateToEvent();
    }
  } else {
    // 其他情况的提示（可选）
    alert("不符合跳转条件");
  }
}

function navigateToEvent(id) {
  let url = '/List/events/';
  if (id) url += id;
  // 补上实际跳转的代码！之前只生成了URL，没执行跳转
  window.location.href = url;
}

2. 优化HTML代码 #

你的HTML里<li>标签多了个多余的双引号，同时给a标签加上href="javascript:void(0)"避免误触默认跳转：

<li><a href="javascript:void(0)" data-destination="local" dataid="' + this.MarketID + '" data-nodeid="' + this.NodeId + '">
  <span><input type="checkbox" id="cb" onclick="Allchecks1(event);"></span>
</a></li>

关键注意点 #

• 逻辑顺序：把复选框状态判断放在符合跳转条件的分支内部，才能正确响应勾选/取消的操作。
• 实际跳转：之前的navigateToEvent只生成了URL，必须加上window.location.href = url才能实现页面跳转。
• 事件控制：阻止事件冒泡到a标签，避免a标签的默认行为干扰复选框的逻辑执行。

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