Optionen einer Select Box hinzufügen

[Struppi]

Den Select Kontainer lasse ich mittels DOM erstellen, hier das Skript:

du machst viel zuviel.

}else if(art == "Select"){
   var newSelect = document.createElement("select");

Das ist ok, hier erzeugst du ein select Element, aber warum dann das:

var select_name = document.createAttribute("name");
   var select_id = document.createAttribute("id");
   var select_size = document.createAttribute("size");

Ein Select Element besitzt diese Attribute, du musst sie nicht erzeugen.

select_name.nodeValue = document.getElementById("text_name").value+i;
   select_id.nodeValue = document.getElementById("text_name").value+i;
   select_size.nodeValue = document.getElementById("text_size").value;

und das könntest du dann wesentlich übersichtlicher schreiben.

newSelect.name = document.getElementById("text_name").value + i;
newSelect.id = document.getElementById("text_name").value + i ;
newSelect.size = document.getElementById("text_size").value;

newSelect.setAttributeNode(select_name);
   newSelect.setAttributeNode(select_id);
   newSelect.setAttributeNode(select_size);
   //newSelect.setAttributeNode(select_option);

Und das fällt komplett weg.

Struppi.