[js] logishe fout ergens in deze code?

[killercow]

ik ben al geruime tijd met deze code aan het stoeien, hij geeft zelf geen error, wij werkt ook niet maar als ik hem in de code plak tussen wat andere functies werken die ineens ook niet meer. (wazige zin)

volgens mij is deze code goed maarjah, ik denk dat k ergens over heen lees.

code:

function checkempty(objects){
    var object=stringSplit(objects,'+');
    var missing='';
    var missingcount=0;
    for (var i=0; i<object.length; i++){
        if((document.getElementById(object[i]).value=='') || (document.getElementById(object[i]).value==NULL)){
             missing=missing+ " " + object[i];
             missingcount++;
        }
    }
    if(missingcount==1){
        alert(missing +'is een verplicht veld');
        return false;
    } elseid(missingcount>1){
        alert(missing +'zijn verplichte velden');
        return false;
    } else {
        return true;
    }
}
function checkrange(objects, low, high){
    var object=stringSplit(objects,'+');
    var missing='';
    var missingcount=0;
    for (var i=0; i<object.length; i++){
        if((document.getElementById(object[i]).value>high) || (document.getElementById(object[i]).value<low)){
            missing=missing+ " " + object[i];
            missingcount++;
        }
    }
    if(missingcount>0){
        alert('de waarde van' +missing+ 'moet zich tussen' +low+ 'en' +high+ 'bevinden');
        return false;
    } else {
        return true;
    }
}

is er iemand die de fout ziet? want ik zie hem namenlijk niet, de functie stringsplit, geeft een array terug met alle delen van de gesplitte string (logish toch.) en die werkt perfect.

alvast bedankt.

[oh,when?]

code:

[..]
} elseid(missingcount>1){

moet dan niet elseif zijn?

[Verwijderd]

code:

function checkempty(objects){
    var object=stringSplit(objects,'+');
    var missing='';
    var missingcount=0;
    for (var i=0; i<object.length; i++){
        if( (document.getElementById(object[i]).value=='') || (document.getElementById(object[i]).value==NULL) ){
             missing=missing+ " " + object[i];
             missingcount++;
        }
    }
    if(missingcount==1){
        alert(missing +'is een verplicht veld');
        return false;
DEZE REGEL IS FOUT!:
    } elseid(missingcount>1)
        alert(missing +'zijn verplichte velden');
        return false;
    } else {
        return true;
    }
}

//edit damn, te laat

[killercow]

Op dinsdag 14 mei 2002 14:00 schreef oh,when? het volgende:

code:

[..]
} elseid(missingcount>1){

moet dan niet elseif zijn?

uhm jah dat ook idd
maar t werkt nog steeds niet
hij geeft dus een error als ik een functie op die pagina uit een ander frame wil aanroepen, dit werkt wel perfect als bovenstaande code er niet in zit, maar geeft de error : deze eigenschap op methode wordt niet ondersteund door dit object.. als ik de code erin zet.

[Bosmonster]

Notepad-diehard zeker?

Je roept hem aan vanuit een ander frame? document.getElementById() moet je dan ook wel over het document in het andere frame laten gelden ofnie dan?

[killercow]

uiteraard, alles van php to js (naja eigenlijk textpad)

[Verwijderd]

Op dinsdag 14 mei 2002 14:00 schreef oh,when? het volgende:

code:

[..]
} elseid(missingcount>1){

moet dan niet elseif zijn?

Nee, dat moet

code:

else if

zijn.... toch...

[Bosmonster]

Op dinsdag 14 mei 2002 14:04 schreef killercow het volgende:
uiteraard, alles van php to js (naja eigenlijk textpad)

Aha.. zonde, want die typ-fout had je met syntax highlighting zelf direct gezien...

[Verwijderd]

Op dinsdag 14 mei 2002 14:06 schreef Bosmonster het volgende:

[..]

Aha.. zonde, want die typ-fout had je met syntax highlighting zelf direct gezien...

Precies zo zag ik je vout ook

[killercow]

jah idd, t werkt (spatie vergeten programmeer talen verschillen soms net iets meer dan ik denk) anyway nu nog ff uitvinden waarom NULL niet gedefineerd is aldus js parser, maar daar kom ik hopenlij zelf wel uit

naja thanx
(tis ook wel erg dat m'n laatste 5 topics dit soort foutjes waren )

[Verwijderd]

NULL is nooit gedefinieerd

[Bosmonster]

Op dinsdag 14 mei 2002 14:10 schreef Zoepnek het volgende:
NULL is nooit gedefinieerd

var variabele = null

zo, null gedefinieerd null is de gedefineerde niksheid..

[Verwijderd]

je hebt een varabele gedefinieerd niet null

[Bosmonster]

Op dinsdag 14 mei 2002 14:08 schreef killercow het volgende:
jah idd, t werkt (spatie vergeten programmeer talen verschillen soms net iets meer dan ik denk) anyway nu nog ff uitvinden waarom NULL niet gedefineerd is aldus js parser, maar daar kom ik hopenlij zelf wel uit

naja thanx
(tis ook wel erg dat m'n laatste 5 topics dit soort foutjes waren )

Misschien tijd om dat toch eens een echte editor te gaan gebruiken

[Bosmonster]

Op dinsdag 14 mei 2002 14:13 schreef Zoepnek het volgende:
je hebt een varabele gedefinieerd niet null

er is in de meeste talen juist een variabele met de naam "null" voorgedefinieerd met een null-waarde.. oftewel niks. Niet gedefinieerd is niet per definitie null.

maar goed.. daar valt over te discussieren.. of je het hebt over de variabele null of een null-waarde.

[crisp]

Gewoon een tip: test je JS uit in Mozilla; die heeft veel duidelijkere error-reporting.