接手项目中setInterval交替执行逻辑的技术疑问

我来帮你拆解这段看起来有点绕的定时器逻辑——其实它的核心想法是在每次点击li.mention-individuals时，切换使用两个定时器变量来管理消息刷新的定时任务，但这个设计存在不少冗余甚至逻辑问题，咱们一步步看：

先梳理这段逻辑的核心行为 #

每次用户点击这个联系人列表项时，代码会尝试：

• 先清除旧的定时器（如果有的话）
• 切换使用intervalId或intervalIdtwo来创建新的1秒刷新一次的消息拉取任务

逐分支解析if/else逻辑 #

咱们把每个分支拆开看：

第一个分支：if (!intervalId) #

• 当intervalId是null（也就是还没创建过第一个定时器）的时候：
  1. 创建第一个定时器，赋值给intervalId，每隔1秒调用loadMessageFromSearch拉取消息
  2. 清除intervalIdtwo（如果之前有创建过），并把它设为null
• 这一步的意图是：第一次点击时启用第一个定时器，同时确保第二个定时器是关闭状态

第二个分支：else if (!intervalIdtwo) #

• 当intervalId已经存在（第一个定时器在运行），但intervalIdtwo是null的时候：
  1. 清除intervalId，把它设为null
  2. 创建第二个定时器，赋值给intervalIdtwo，同样每隔1秒拉取消息
• 这一步的意图是：第二次点击时，关掉第一个定时器，切换到第二个定时器运行

第三个分支：else #

• 当两个定时器变量都不为null的时候，弹出alert('Nothing found')
• 但实际上这个分支几乎不会触发，因为前面两个分支都会把另一个定时器置为null

这段逻辑的问题与不合理之处 #

其实这段代码的设计完全是冗余的，因为：

1. 两个定时器做的是完全一样的事情：不管用intervalId还是intervalIdtwo，都是每隔1秒调用同一个loadMessageFromSearch函数，没有任何差异
2. 切换定时器没有任何意义：每次点击只是换了个变量存定时器ID，但实际功能完全重复，反而增加了维护复杂度
3. 存在内存泄漏风险：虽然每次切换会清除另一个定时器，但如果代码出现异常，可能导致某个定时器没被清除，一直在后台运行
4. 逻辑可以大幅简化：根本不需要两个定时器变量，只用一个就足够了

优化后的简化版本 #

你可以把这段定时器逻辑改成更清晰的写法，完全不需要两个变量：

// 只需要一个定时器变量
var messageRefreshTimer;

$(document).on('click', 'li.mention-individuals', function() { 
    // ... 其他代码保持不变 ...

    // 先清除之前的定时器（如果存在）
    if (messageRefreshTimer) {
        clearInterval(messageRefreshTimer);
    }
    // 创建新的定时器，每隔1秒拉取消息
    messageRefreshTimer = setInterval(function() { 
        loadMessageFromSearch(useridForAjax, otheridFromSearch); 
    }, 1000);
})

这样既保留了“每次点击都重新启动消息刷新定时器”的核心需求，又去掉了冗余的切换逻辑，代码更易读也更可靠。

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