简化onclick jquery/JavaScript函数
我试图简化网站上的所有.js功能,以提高页面加载速度和网站的整体性能。我自己设法完成了大部分工作,但当我开始最小化两个函数时,这些函数主要是从页面标题中的functions.js调用的回调函数和onclick函数 所以这里是我的问题,我如何从下面的2个函数中选择一个函数来缩短脚本,而不丢失任何功能,或者如果有其他方法使它比当前更简单简化onclick jquery/JavaScript函数,javascript,jquery,Javascript,Jquery,我试图简化网站上的所有.js功能,以提高页面加载速度和网站的整体性能。我自己设法完成了大部分工作,但当我开始最小化两个函数时,这些函数主要是从页面标题中的functions.js调用的回调函数和onclick函数 所以这里是我的问题,我如何从下面的2个函数中选择一个函数来缩短脚本,而不丢失任何功能,或者如果有其他方法使它比当前更简单 function user_dialog(dialog_content, download_button) { if (typeof jQuery.ui !
function user_dialog(dialog_content, download_button) {
if (typeof jQuery.ui != 'undefined') {
$("#dialog").attr("title", "Lets share it").html(dialog_content);
$("#dialog").dialog({
modal: false,
width: 400,
buttons: {
Cancel: function() {
$(this).dialog("close");
},
"Download": function () {
$(this).dialog("close");
window.location = download_button;
}
}
});
} else {
window.location = download_button;
}
}
function user_notice(link_source) {
download_link = $(link_source).attr("href");
$.ajax({
type: "POST",
url: "/includes/json.php",
data: "action=reminder&thepath=" + download_link,
dataType: "json",
error: function() {
window.location = download_link;
},
success: function(resp_dialog) {
if (resp_dialog.status == 1) {
user_dialog(resp_dialog.html, download_link);
}
}
});
}
提前感谢您的帮助 在函数变量中添加一个操作变量,如us\u action和un\u action,然后使用if语句说明if是us\u acion,否则执行此操作,如果un\u action执行此操作,则执行此操作。如果要提高页面加载速度,则应使用。这不是什么优化。。。如果您希望获得性能提升,则需要首先考虑其他问题。缩短这段代码几乎没有什么好处。结合这些函数可能会产生更多的代码。@adamb谢谢,这是一个非常明智的建议,我刚刚完成了这项工作,并成功地将function.js的大小从1883kb减少到892kb,而没有丢失任何功能。太好了你有什么事吗?我猜不是。没有人知道。主动缓存将是您可能获得的最大性能提升。@CoryDanielson,主动缓存是我做的第一件事情之一,因此这是一种将负载降至最低的额外事情。另外,对于其他人来说,他们可能也对减少和改进感兴趣,使用好的CDN进行静态内容分发,这有很大的帮助。在你看来,PageSpeed 94和yslow 93是好的,不是很好,不好,真的很差吗?哈哈,CoryDanielson已经告诉您,缩短此代码不会带来任何好处。我只是想给你一个如何缩短它的想法,但除非你处理重复函数的负载,否则缩短它们不会带来太多优化。如果您希望优化代码,那么除此之外,还有许多其他的事情需要首先考虑。