[Javascript] script vertoont fouten na 1x uitgevoerd

[RainerSGF]

Ik heb voor een administratie-gedeelte van een website code geprogrammeerd die toelaat geposte berichten op de site na te zien en door de administrator aan te vinken indien deze mogen gepubliceerd worden. Dit heb ik laten realiseren via Ajax, echter nu de code/script éénmaal met succes uitgevoerd is...blijkt het script een fout te vertonen wanneer het een volgende keer moet uitvoeren.

Even hier mijn code laten zien:

JavaScript:

function getSelectedMessages(){
        
        
         xmlhttp = GetXmlHttpObject();
                
        if(xmlhttp == null){
            alert("Uw browser ondersteunt geen HTTP-aanvragen!");
            return;
        }
        
    
        for(var i=0; i < document.berichten.check.length; i++){         
            if(document.berichten.check[i].checked){
                alert("" + i);  //alert-test, maar draait niet bij 2de script-uitvoering
                n = document.getElementsByName("nr")[i].innerHTML;
                nr = Number(n);
                
                
                url = "functions.php";
                url = url + "?nr=" + nr;
                //url = url + "&sid=" + Math.random();      
                
                xmlhttp.onreadystatechange = state_change;
                xmlhttp.open("GET",url,true);
                xmlhttp.send(null);
            }
        }
    }

Tot op heden heb ik al eens geprobeerd te debuggen met 'Firebug' (via afbreekpunten). Hierbij zie ik dat gestopt wordt bij 'xmlhttp' (zegt dat deze 'undefined' is) en bij de for-loop. Ik heb 'xmlhttp' wel als variabele gedeclareerd helemaal in het begin van dit externe javascript-bestand.

K'zie verder niet waar ik mis ben. Iemand die me kan helpen?

[Sponz]

Zet xmlhttp = GetXmlHttpObject(); binnen de loop, ik denk dat ie dan wel meer dan 1 keer loopt.

[pieturp]

Niet dat 't waarschijnlijk veel uitmaakt voor je probleem, maar 't is wel veel netter om al je variabelen bij initialisatie te voorzien van het statement 'var'. Dus

code:

var xmlhttp = getXmlHttpObject();
var n = "";
var nr = ""; //etc.

Zodoende worden 't geen globals die in andere functies mogelijk ook worden gebruikt!

Ook kun je i.p.v. alert(); console.log(); gebruiken. Je geeft immers aan firebug te gebruiken :-)

edit:

Ook hoef je geen variabele nr te maken, via een omslachtige Number(n). Je maakt er namelijk gelijk weer een string mee (var) url. Sowieso is JavaScript loosely typed, dus strings worden automatisch numbers als je er mee zou willen rekenen :-)

[ Voor 22% gewijzigd door pieturp op 11-01-2010 13:42 . Reden: aanvulling ]