[JS] Formulier check werkt wel in IE, niet in FF/Moz

[1st_Ro]

Okay, ik heb een klein probleempje met mijn Javascript en ik kom er niet 1 2 3 uit.

Onderstaand scriptje is een checkje voor een formulier. Qua code niet echt bijzonder, maar het gekke is dat dit scriptje wel werkt in Internet Explorer en niet in Firefox (en da's wel zo prettig ).

code:

function FormCheck(formName) {
    var msg = "De volgende velden zijn niet of onjuist ingevoerd:\n\n";
    var OK = 0;

    for (i=0; i < document[formName].length; i++) {
        switch (document[formName].elements[i].name) {

            case "naam": 
                if (document[formName].elements[i].value == "") {
                    msg += "- Naam\n";
                    OK = 1;
                }
                break;
        
            case "message": 
                if (document[formName].elements[i].value == "") {
                    msg += "- Uw vraag\n";
                    OK = 1;
                }
                break;
        
            case "email":
                var regexp = /[0-9a-zA-Z]{2,}@[0-9a-zA-Z/.-]{2,}.[a-z]{2,4}/i;
                if (!regexp.exec(document[formName].elements[i].value)) {
                    msg += "- Emailadres\n";
                    OK = 1;
                }
                break;
        }
    }

    if (OK == 1) {
        alert (msg);
    }
    else if (OK == 0) {
         document[formName].submit();
    }
}

Ik ben dus aan het alerten enzo geweest ("naam" en "message" werken correct) en volgens de Javascript monitor van Firefox gaat het hierom :

Error: unterminated character class 0
Source File: http://devserver/~devserver/czjobs/js/contactformcheck.js
Line: 23, Column: 17
Source Code:
var regexp = /[0-9a-zA-Z]{2,}@[0-9a-zA-Z/.-]{2,}.[a-z]{2,4}/i;

Afijn, het ligt dus blijkbaar aan mijn regexp, maar wat is er dan mis mee?

Iemand die ziet wat ik niet zie ?

[ Voor 6% gewijzigd door 1st_Ro op 04-08-2004 16:22 ]

[1st_Ro]

ARGH, te vroeg gepost hier !

Blijkbaar is regexp een gereserveerd woord (nooit geweten!) ...

Ik heb de variabele regexp vervangen door rule et voila, het werkt...

Dit is echt zo'n Homer Simpson moment : D'OH !

[André]

Opgelost

[ Voor 94% gewijzigd door André op 04-08-2004 16:26 ]

[1st_Ro]

Andre, ging per ongeluk

[drm]

Even langs de zijlijn:

code:

if (OK == 1) {
        alert (msg);
    }
    else if (OK == 0) {
         document[formName].submit();
    }

is niet erg logisch. Dan kun je beter OK op true zetten en als er een fout optreedt op false zetten, want als je deze functie in de onsubmit van je form zet, kun je OK als returnwaarde geven. Wanneer de returnwaarde van de functie in de onsubmit false is, wordt het formulier namelijk niet gesubmit, en behoudt OK ook zijn semantische betekenis:

code:

if ( !OK ) {
   alert ( msg );
}
return OK;

Maar dit terzijde

[crisp]

regexp is helemaal geen reserved word: http://devedge.netscape.c...5/reference/keywords.html
het punt is dat je de / in je characterclass bent vergeten te escapen (en de foutmelding zegt dat ook duidelijk imho):

JavaScript:

var regexp = /[0-9a-zA-Z]{2,}@[0-9a-zA-Z\/.-]{2,}.[a-z]{2,4}/i;

en deze regexp kan trouwens nog korter:

JavaScript:

var regexp = /[0-9a-z]{2,}@[\w\/.]{2,}.[a-z]{2,4}/i;

[ Voor 6% gewijzigd door crisp op 04-08-2004 20:56 ]

[1st_Ro]

crisp,

Dan vind ik het wel typisch dat ik het oplos door die naam te veranderen ?
Ik ben nog niet goed met regular expressions, dus het zal vast wel korter kunnen ...

iig bedankt voor de extra info !