Kode der ikke virker

Den tidligere tråd er her: http://www.eksperten.dk/spm/507723

Prøv evt at kigge på verify_email scriptet herfra:

http://www.ljweb.biz/usefulscripts/

Det har jeg gjort, men det virker heller ikke =(

Han skriver følgende:
if (preg_match("/\b[A-Z0-9._%+-]+@[A-Z0-9._%-]+\.[A-Z0-9._%-]{2,4}\b/i", $mail))
{
    //godkendt
}
else
{
  // ikke godkent
}

SÅ vidt jeg kan se er det lidt omvendt af den kode du skriver her, så prøve lige med følgende:

if (!preg_match("/\b[A-Z0-9._%+-]+@[A-Z0-9._%-]+\.[A-Z0-9._%-]{2,4}\b/i", $mail)) {

hvorvidt det er den rigtige vej eller ej virker koden stadigvæk ikke..

jeg har prøvet at skrive en okay email, og fik et svar (det er lige meget hvilket). Derpå prøvede jeg en helt forkertformateret e-mail og fik nøjagtigt samme svar...

jeg plejer at bruge den her:

if (!preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $_POST['turn_emai'])) {
  $error[] = "Kontakt emailen er ugyldig.";
}

Thomas

jeg tror altså der er forskellige karakter der skal escapes.

fx
. -> \.
+ -> \+
- -> \-

prøv:
function Validate_E_mail($email) {
if (preg_match("/\b[A-Z0-9._%+-]+@[A-Z0-9._%-]+\.[A-Z0-9._%-]{2,4}\b/i", $email)) {
  return false;
  } else {
  return true;
  }
}

$mail = "me@me.com";

if( Validate_E_mail($mail) ) {
  echo "ok";
}

Okay jeg prøver lige din THB

", $mail))

->
", $email))

+ den returner rigtig false og true

THB den returnere "false" uanset om e-mail er rigtig eller forkert

så kan du skrive:

function Validate_E_mail($email){
return (preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $email)) ? TRUE : FALSE;
}

$mail = "thomas@thomas.dk";
if(!Validate_E_mail($mail)) print("Fejl");

stois >> noget kunne tyde på at DU laver en fejl et eller andet sted - kan du ikke prøve at komme med noget mere af din kode.

Hvad har du ændret?

ikke noget... :D
Jeg lavede det bare til en hel funktion ligesom chries..

Den virker altså helt fint på mit site..

Thomas

function Validate_E_mail($email) {
if (!preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $mail))  {
  return "false";
  exit;
} else {
  return "true";
}
}

det er alt der er i min funktion

Der er altså noget som jeg gør galt og det er ikke den der sætning

Kani se hvad det evt. kunne være det er jeg gør galt ???

Du skal ikke have gåseøjne omkring "true" og "false"

funktion:
E_mail($email) {

preg:
+$/', $mail))  {
ændres til
+$/', $email))  {

%&¤#%&¤& .... og noget det er da også rigtigt *klasker sig selv høøjlydt i panden*

thb:

bør den anden * ikke være et plus? Ellers tror jeg den accepterer "@email. dk" :)

Og evt ændre den sidste del fra a-zA-Z0-9 til a-zA-Z (jeg tror ikke der findes nogle tld med tal i :))

return (preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $email)) ? TRUE : FALSE;

detox> lige i mit tilfælde vil jeg gerne have at der bliver returneret en tekst streng som jeg så kan gøre med hvad jeg vil

Jeg vil også lige sige, at jeg ikke havde gang i at lave det til en funktion, det er dit eget projekt :)

THB>

Mener du det skal være:

return (preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z_-]+(\.[a-zA-Z_-]*([a-zA-Z]))+$/', $email)) ? TRUE : FALSE;

????

Sorry Jaw, det var ikke for at sige du var dårlig ovehovedet ikke....

Jeg er bare typen der kan tage god programmering og %¤/&¤%e det fuldstændigt op

Jeg kommer da lige i tanke om at TLD sagtens kan være tal... hvad med f.eks. 909.dk

hehe, tog det skam heller ikke op... meeen, vil ikke have skyld for noget jeg ikke var skyld i (så vidt jeg kan se!)

he he selvfølgelig ikke =)

function Validate_E_mail($email) { // Thanks goes to Lars B. Jensen for this function (http://ww.ljweb.com)
if (preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z]))+$/', $email))  {
  return "false";
  exit;
} else {
  return "true";
}
}

>-----[ X ]-----<

Jeg har lige prøvet det her, men det virker ikke endnu =(

er det her noget der kun virker på en Unix computer eller hvad?

function Validate_E_mail($email) { // Thanks goes to Lars B. Jensen for this function (http://ww.ljweb.com)
if (preg_match('/^([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z]))+$/', $email))  {
  return false;
} else {
  return true;
}
}

fra php.net

Read RFC 2822 regarding email addresses to find that the following is a more correct way of validating email addresses:

preg_match( "/^[-^!#$%&'*+\/=?`{|}~.\w]+@[-a-zA-Z0-9]+(\.[-a-zA-Z0-9]+)+$/", $email );

Parse error: parse error, unexpected ';', expecting T_OLD_FUNCTION or T_FUNCTION or T_VAR or '}' in c:\appserv\www\test\class.php on line 36

if (preg_match( "/^[-^!#$%&'*+\/=?`{|}~.\w]+@[-a-zA-Z0-9]+(\.[-a-zA-Z0-9]+)+$/", $email))....

function Validate_E_mail($email) {
        if (preg_match( "/^[-^!#$%&'*+\/=?`{|}~.\w]+@[-a-zA-Z0-9]+(\.[-a-zA-Z0-9]+)+$/", $email))  {
            return "false";
            exit;
        } else {
            return "true";
        }
    }
?>  <--- Der er fejlen... men hvad betyder det?

NOOOOOOO!!!!!

Fejlen lå et helt andet sted =(

argh!!! jeg hader når jeg laver sådan noget.. Fejlen lå i at jeg ikke lagde den rigtige variable ind i $email, så er der joh ikke noget at sige til at den ikke får rigtigt fat i den....

Dumme slag til mig

En af jer, der har hjulpet mig, send mig et svar så i kan få point!

TLD kan da ikke være et tal. I 909.dk er det dk, der er TLD.

Snydt!