重構Javascript程式碼範例(重構前後對比)_javascript技巧

今天有做了幾個asp.net結合Javascript的教學文章。現回顧頭來看那些Javascript腳本，有寫得不太理想，過於複雜。現抽取出來，重建它們。
之前一：

複製程式碼 代碼如下:

代碼如下:



代碼如下:



代碼如下:



代碼如下:



代碼如下:cb.checked = cb.checked ? false : true;
var gv = document.getElementById('');
var rc = gv .rows.length;
for (var i = 1; 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 = "#66ffff33;"; }

else {

row.style.backgroundColor = "";
}
}



經過重構之後的Javascript腳本
:




複製程式碼


程式碼如下:

function SelectedAll(cb) {
gv = document.getElementById(''); var rc = gv.rows.length; for (var i = 1; 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(""); var input = cbl.getElementsByTagName("input") var input = cbl.getElementsByTagName("input") if ; cb.checked) { for (var i = 0; i input[i].checked = true;

}

}
else {
for (var i = 0; i input[i].checked = false;
}
}
}

} } } } } } } } } } 重構之後的Javascript腳本： 複製代碼 代碼如下: function Check_Uncheck_All(cb) { var cbl = document.getElementById(""); var input = cbl.getElementsByTagName("input"); var i = 0; i input[i].checked = cb.checked; } }