Question

$("#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

Answer 1

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(''));
});