jQuery / javascript优化(jQuery/javascript optimization)

jQuery / javascript优化(jQuery/javascript optimization) $("#mySelect").change(function() { var myVar = $('#MyDiv'); for (var i=0;i<this.value;i++) { myVar.clone().appendTo('#formContainer'); } });

对可读性或性能的任何改进都欢迎

$("#mySelect").change(function() { var myVar = $('#MyDiv'); for (var i=0;i<this.value;i++) { myVar.clone().appendTo('#formContainer'); } });

Any improvements to readability or performance welcome

最满意答案

我同意@Diodeus，你应该创建html字符串，然后一次性追加它。 尝试这个。

$("#mySelect").change(function() { var myVar = $('#MyDiv'), html = [], count = parseInt(this.value, 10), myDivHTML = $('#MyDiv').wrap('<div />').parent().html(); for (var i = 0; i < count; i++){ myVar.push(myDivHTML); } $('#formContainer').append(html.join('')); });

I agree with @Diodeus, you should go with creating html string and then append it in one go. Try this.

$("#mySelect").change(function() { var myVar = $('#MyDiv'), html = [], count = parseInt(this.value, 10), myDivHTML = $('#MyDiv').wrap('<div />').parent().html(); for (var i = 0; i < count; i++){ myVar.push(myDivHTML); } $('#formContainer').append(html.join('')); });