Warning: file_get_contents(/data/phpspider/zhask/data//catemap/9/javascript/387.json): failed to open stream: No such file or directory in /data/phpspider/zhask/libs/function.php on line 167

Warning: Invalid argument supplied for foreach() in /data/phpspider/zhask/libs/tag.function.php on line 1116

Notice: Undefined index: in /data/phpspider/zhask/libs/function.php on line 180

Warning: array_chunk() expects parameter 1 to be array, null given in /data/phpspider/zhask/libs/function.php on line 181
简化onclick jquery/JavaScript函数_Javascript_Jquery - Fatal编程技术网

简化onclick jquery/JavaScript函数

简化onclick jquery/JavaScript函数,javascript,jquery,Javascript,Jquery,我试图简化网站上的所有.js功能,以提高页面加载速度和网站的整体性能。我自己设法完成了大部分工作,但当我开始最小化两个函数时,这些函数主要是从页面标题中的functions.js调用的回调函数和onclick函数 所以这里是我的问题,我如何从下面的2个函数中选择一个函数来缩短脚本,而不丢失任何功能,或者如果有其他方法使它比当前更简单 function user_dialog(dialog_content, download_button) { if (typeof jQuery.ui !

我试图简化网站上的所有.js功能,以提高页面加载速度和网站的整体性能。我自己设法完成了大部分工作,但当我开始最小化两个函数时,这些函数主要是从页面标题中的functions.js调用的回调函数和onclick函数

所以这里是我的问题,我如何从下面的2个函数中选择一个函数来缩短脚本,而不丢失任何功能,或者如果有其他方法使它比当前更简单

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已经告诉您,缩短此代码不会带来任何好处。我只是想给你一个如何缩短它的想法,但除非你处理重复函数的负载,否则缩短它们不会带来太多优化。如果您希望优化代码,那么除此之外,还有许多其他的事情需要首先考虑。