Valider e-mail adresse

if (email.indexOf("@")==0||email.lastIndexOf(".")==0||email.indexOf("@")==-1||email.lastIndexOf(".")==-1||email.lastIndexOf(".")<email.indexOf("@")||email.lastIndexOf(".")-email.indexOf("@"))

^Tjekker om emailadressen er gyldig.

der skal kun indtastes id på det input felt hvor email indtastes i
document.getElementById('emailinputfeltid');
så skal du bare gemme disse 2 functioner

function ValidateEmail()
    {
        var emailID= document.getElementById('emailinputfeltid');
       
        if ((emailID.value==null)||(emailID.value=="")){
            alert('Ugyldig email');   
        }
        if (echeck(emailID.value)==false){
            alert('Ugyldig email');   
        }
    }

og denne

function echeck(str) {

        var at="@";
        var dot=".";
        var lat=str.indexOf(at);
        var lstr=str.length;
        var ldot=str.indexOf(dot);
        if (str.indexOf(at)==-1){
alert('Ugyldig email');
            return false;
        }

        if (str.indexOf(at)==-1 || str.indexOf(at)==0 || str.indexOf(at)==lstr){
    alert('Ugyldig email');
            return false;
        }

        if (str.indexOf(dot)==-1 || str.indexOf(dot)==0 || str.indexOf(dot)==lstr){
alert('Ugyldig email');
            return false;
        }

        if (str.indexOf(at,(lat+1))!=-1){
alert('Ugyldig email');
            return false;
        }

        if (str.substring(lat-1,lat)==dot || str.substring(lat+1,lat+2)==dot){
    alert('Ugyldig email');
            return false;
        }

        if (str.indexOf(dot,(lat+2))==-1){
alert('Ugyldig email');
            return false;
        }
       
        if (str.indexOf(" ")!=-1){
alert('Ugyldig email');
            return false;
        }

        return true;                   
    }

kalp>> if-sætningerne burde måske slåes sammen til én, hvis man ikke vil risikere at få smidt 7 alerts med "Ugyldig email" op på skærmen :)

hehe jeg bruger den ikke med alerts rigtig men på en label:P

Ah, ok :)

w13 >> nu hvor jeg kigger på koden så vil den faktisk ikke smide 7 alerts op på skærmen.


der er jo return false hvis den havner i en af dem:)

Ja, det er vist korrekt.

<ole>

En ny linje til din egen funktion (i spørgsmålet):
    if (!email.value.match(/^.*@\w[\w\.-]*\.[a-z]{2,6}$/)) alertMessage += "\nE-mail-adressen er ikke valid!";

/mvh
</bole>

Hmmm, tak for de gode forslag. Dog kan jeg rigtig godt li' oleboles forslag da den er så dejlig simpel på en enkelt linie. Dog fatter jeg ikke helt at den ikke virker. Jeg kan stadig bare skrive fx: a, og stadig behandler den formen? Kan ikke lige selv se hvad der skulle være galt med den.

Ahh, damn. Der var jeg for hurtig. Jeg havde lagt linien for langt nede og nu virker det korrekt.

<ole>
Rocks endnu engang!
</bole>

Den er ikke 100% skudsikker, men den fanger laaaaangt de fleste fejl  ;o)

jeg fatter ikke hvorfor der ikke skal ' ' rundt om den regexp :S

Det er fordi, at regulære udtryk escapes med '/' i stedet:
    --> /^.*@\w[\w\.-]*\.[a-z]{2,6}$/ <--

Læs evt. lidt om RegExp her:
    http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Global_Objects:RegExp

- og spørg gerne igen, men læs først. Det får du mest ud af  ;o)

tror lige det skal læses:D
i andre sprog er jeg jo vant til "" rundt om:)

Ja, hvilket ikke ligefrem er logisk, da der jo ikke er tale om en streng  ;o)

På den anden side kan du i JS bruge RegExp på flere måder. Instantierer du RegExp-objektet med new RegExp(), skal du også bruge gåseøjne. Det står der en masse om bag det link, jeg skrev  =)

Takker for points  ;o)