javascript - 对并不总是发生的事件执行 jQuery 脚本是否不好？

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 8 年前。

Improve this question

我的页面可能有也可能没有以下警报:

<div class="alert">Some alert</div>

5 秒后我将它们隐藏起来:

<script type="text/javascript">
$(document).ready(function () {

window.setTimeout(function() {
    $(".alert").fadeTo(1500, 0).slideUp(1500, function(){
        $(this).remove(); 
    });
}, 5000);

});
</script>

所以我想知道，我是否应该将警报隐藏功能放入我的常规函数​​中，这些函数总是在页面加载时执行，即使没有要隐藏的警报？或者将其放在实际警报 div 代码正下方的 HTML 代码中，以便仅在警报显示时发生？

第一个选项的好处是我的代码更有组织性。缺点是即使没有警报它也可能执行。

第二个选项的优点是它仅在警报显示时执行，但缺点是代码比较困惑。

哪个更好？

Answer 1

没有什么不好，但是通过删除 setTimeout 可以提高效率:

$(".alert").delay(5000).fadeTo(1500, 0).slideUp(1500, function(){
    $(this).remove(); 
});

这样，什么都不做，如果没有警报，也不会注册任何回调。

至于代码的位置，您应该避免将其放在 HTML 的中间。交错 HTML 和 Javascript 是一种不好的做法，会阻碍可维护性。一个干净的模块化 Javascript 应该是解决方案。