[JS] Childnodes opvragen, daarna via for loop verwijderen

[BikkelZ]

Ik heb dit bewuste stukje code:

JavaScript:

    var optionsProduct = productPulldown.childNodes;
    alert(optionsProduct.length);
    h = optionsProduct.length;
    for (i=0;h>i;i+=1) {
        alert(i);
        productPulldown.removeChild(optionsProduct[i]);
    }

Bij alert(optionsProduct.length); roept hij dat hij er 8 heeft. Dit komt overeen met het aantal options in de select waar het hier over gaat.
alert(i) loopt door tot 4.

Daarna krijg ik de error (helaas geen FireFox om te debuggen ) "Type komt niet overeen" op de regel

JavaScript:

productPulldown.removeChild(optionsProduct[i]);

Het is dus net alsof hij de helft doet, en dan niet meer snapt wat hij precies heeft. Hij loopt tenslotte net zo vaak als hij childs heeft.

[ Voor 7% gewijzigd door BikkelZ op 09-06-2005 12:24 ]

[BikkelZ]

Eigenlijk wil ik gewoon alle children verwijderen, maar blijkbaar is daar geen directe functie voor?

[André]

En als je het omgekeerd doet dan?

code:

var optionsProduct = productPulldown.childNodes;
alert(optionsProduct.length);
h = optionsProduct.length;
for (i = h - 1; i > -1; i--)
{
  alert(i);
  productPulldown.removeChild(optionsProduct[i]);
}

[BikkelZ]

Ik had het inderdaad al zelf gevonden.

Op mijn manier maak ik het stapeltje steeds lager, en vraag ik toch steeds een hogere aan in het stapeltje, dus haalt hij de helft maar weg.

Ik heb nu:

JavaScript:

productPulldown.removeChild(optionsProduct[0]);

En dat werkt dus perfect.

[Clay]

Vind het wel logisch dat het na 4 fout gaat. Je hebt er 8, en dan ga je met een for deleten; 0, 1, 2, 3 ... Op het moment dat je option[4] (de 5e) delete zijn er nog maar 4 over, die 5e bestaat dan al niet meer. Je zou dus ook net zo goed n keer option[0] kunnen deleten...

edit:
te laat

[ Voor 6% gewijzigd door Clay op 09-06-2005 12:33 ]

[BikkelZ]

André schreef op donderdag 09 juni 2005 @ 12:26:
En als je het omgekeerd doet dan?

code:

var optionsProduct = productPulldown.childNodes;
alert(optionsProduct.length);
h = optionsProduct.length;
for (i = h - 1; i > -1; i--)
{
  alert(i);
  productPulldown.removeChild(optionsProduct[i]);
}

Omgekeerd werkt natuurlijk ook goed. Ik had gewoon even in de gaten moeten houden dat bij iedere removeChild de Array zichzelf aanpaste.

[Clay]

Options van een select kan je trouwens ook deleten door de options.length property op 0 te zetten. 't lijkt me dat dat soort properties alleen getters zouden moeten zijn en het is dus imo ueberranzig, maarja.

[MTWZZ]

Toevoeging op de for-loop:

JavaScript:

while(productPulldown.hasChildNodes()) {
        productPulldown.removeChild(productPulldown.firstChild);
}

[crisp]

Clay schreef op donderdag 09 juni 2005 @ 12:35:
Options van een select kan je trouwens ook deleten door de options.length property op 0 te zetten. 't lijkt me dat dat soort properties alleen getters zouden moeten zijn en het is dus imo ueberranzig, maarja.

Volgens devedge zou het ook read-only moeten zijn, maar feit is dat het in alle mij bekende browsers gewoon read/write geimplementeerd is...

[BikkelZ]

Clay schreef op donderdag 09 juni 2005 @ 12:35:
Options van een select kan je trouwens ook deleten door de options.length property op 0 te zetten. 't lijkt me dat dat soort properties alleen getters zouden moeten zijn en het is dus imo ueberranzig, maarja.

...maar dat pikt 'ie weer niet altijd. Nee, zelfs niet in IE

MTWZZ schreef op donderdag 09 juni 2005 @ 13:07:
Toevoeging op de for-loop:

JavaScript:

while(productPulldown.hasChildNodes()) {
        productPulldown.removeChild(productPulldown.firstChild);
}

Dit is dus dé nette methode om dit te doen volgens mij!

[crisp]

BikkelZ schreef op donderdag 09 juni 2005 @ 14:51:
[...]


...maar dat pikt 'ie weer niet altijd. Nee, zelfs niet in IE

mwa, ik ben nog niet tegengekomen dat dat niet werkte...

[...]
Dit is dus dé nette methode om dit te doen volgens mij!

true...