重构Javascript代码示例(重构前后对比)

最新推荐文章于 2024-07-21 14:17:52 发布

转载最新推荐文章于 2024-07-21 14:17:52 发布 · 362 阅读

1 ·

CC 4.0 BY-SA版权

原文链接：https://blog.youkuaiyun.com/WuLex

Html+Javascript 同时被 2 个专栏收录

260 篇文章

订阅专栏

重构

28 篇文章

订阅专栏

本文分享了ASP.NET结合JavaScript的教程中，原有脚本的重构过程，通过简化和优化，提高了代码的效率和可读性。展示了如何改进全选/取消全选功能和检查/取消检查所有复选框的实现。

今天有做了几个asp.net结合Javascript的教程文章。现回顾头来看看那些Javascript脚本，有写得不太理想，过于复杂。现抽取出来，重构它们。
之前一：

代码如下:

function SelectedAll(cb) {
    cb.checked = cb.checked ? false : true;
    var gv = document.getElementById('<%=GridViewCouplets.ClientID %>');
    var rc = gv.rows.length;
    for (var i = 1; i < rc; i++) {
        var input = gv.rows[i].cells[0].getElementsByTagName("input");
        if (input[0].type == "checkbox" && input[0].checked) {
            input[0].checked = false;
            gv.rows[i].style.backgroundColor = "";
        } else {
            input[0].checked = true;
            gv.rows[i].style.backgroundColor = "#66ff33;";
        }
    }
}

function SelectedSingle(cb) {
    var row = cb.parentNode.parentNode;
    if (cb.checked) {
        row.style.backgroundColor = "#66ff33;";
    } else {
        row.style.backgroundColor = "";
    }
}

经过重构之后的Javascript脚本:

代码如下:

function SelectedAll(cb) {
    var gv = document.getElementById('<%=GridViewCouplets.ClientID %>');
    var rc = gv.rows.length;
    for (var i = 1; i < rc; i++) {
        var input = gv.rows[i].cells[0].getElementsByTagName("input");
        if (input[0].type == "checkbox") {
            input[0].checked = cb.checked;
            gv.rows[i].style.backgroundColor = input[0].checked ? "#66ff33;" : "";
        }
    }
}

function SelectedSingle(cb) {
    var row = cb.parentNode.parentNode;
    row.style.backgroundColor = cb.checked ? "#66ff33;" : "";
}

之前二：

代码如下:

function Check_Uncheck_All(cb) {
    var cbl = document.getElementById("<%=CheckBoxListMusicType.ClientID%>");
    var input = cbl.getElementsByTagName("input");
    if (cb.checked) {
        for (var i = 0; i < input.length; i++) {
            input[i].checked = true;
        }
    } else {
        for (var i = 0; i < input.length; i++) {
            input[i].checked = false;
        }
    }
}

重构之后的Javascript脚本：
代码如下:

function Check_Uncheck_All(cb) {
    var cbl = document.getElementById("<%=CheckBoxListMusicType.ClientID%>");
    var input = cbl.getElementsByTagName("input");
    for (var i = 0; i < input.length; i++) {
        input[i].checked = cb.checked;
    }
}