JQuery .each循环未正确跳过指定元素及href判断失效求助

问题概述 #

需实现：点击非特定类（slideout-trigger/register-now-btn）且href不为#的链接时，自动追加当前页面URL的查询参数。但现有代码存在两个异常：

1. 带上述特殊类的链接未被跳过，仍执行参数追加逻辑
2. href为#的链接也会执行参数追加操作

核心排查方向 #

1. 分支逻辑绑定范围错误 #

原代码中else仅与第二个if（判断register-now-btn）绑定，而非与前两个特殊类判断的整体逻辑绑定。这会导致：

• 若第一个if的return未生效（如代码执行异常），非register-now-btn的链接会直接进入else分支
• 正确逻辑应为：只要是两个特殊类之一，就跳过处理；否则才执行参数追加

2. jQuery.each循环的return语义误解 #

在jQuery .each()中：

• return true 等价于continue，跳过当前元素继续下一个循环
• 若特殊类链接仍进入后续逻辑，需排查：
  • DOM元素类名是否与代码拼写完全一致（如大小写、连字符/下划线差异）
  • 循环内是否有前置代码抛出异常，导致return未执行（可在循环开头加console.log($(this).attr('class'))验证）

3. href判断失效的原因 #

• 实际获取的href可能存在空格（如DOM中写为href=" # "），需用trim()去除前后空白后再判断
• 部分场景下$(this).attr('href')可能返回undefined（如链接未设置href属性），需先判断href是否存在
• 可在判断前打印console.log('当前href:', href)，确认实际获取值

修正后的代码示例 #

$('a').each(function(index, item) {
    // 统一判断特殊类，直接跳过
    if ($(this).hasClass("slideout-trigger") || $(this).hasClass("register-now-btn")) {
        console.log("跳过特殊链接:", $(this).text());
        return true;
    }

    var querystring = window.location.search.substring(1);
    var href = $(this).attr('href');

    // 处理空白和空值，严格判断是否为#
    if (href && href.trim() !== "#") {
        // 用includes替代match，更简洁可靠
        href += (href.includes('?') ? '&' : '?') + querystring;
        $(this).attr('href', href);
    }
})

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