stop ved ingen @

Jeg bruger selv følgend regulærre udtryk til at validerer email adresser, men jeg vil på ingen måde stå inde for at den er 100% rigtig!

if (!preg_match_all('/^([a-zA-Z0-9]+)([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $email, $matches)) {
    // Email adressen er ikke gyldig
} else {
    // Email adressen er gyldig
}

hmm hvordan sætter jeg lige det ind

jeg har sat det ind sådan

<?php
// Vis alle kommentarer for en nyhed
include_once("../mysql_funktioner.php");
aabn_forbindelse_og_vaelgdb();
if ($_POST["id"]) {
    $nyhedsid = $_POST["id"];
} else {
    $nyhedsid = $_GET["id"];
}

if (!$nyhedsid) {
    echo "Intet nyheds-id angivet.";
} else {

if (!preg_match_all('/^([a-zA-Z0-9]+)([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $email, $matches)) {
echo"Email adressen er ikke gyldig";// Email adressen er ikke gyldig
} else {
    // Email adressen er gyldig

    if ($_POST[add_comment]) {
        $navn = $_POST["navn"];
        $email = $_POST["mail"];
        $kommentar = addslashes($_POST[kommentar]);
        $addcomment = mysql_query("insert into kommentarer (nyhedsid, forfatternavn, forfattermail, kommentar, dato) VALUES ('".$nyhedsid."','".$navn."','".$mail."','".$kommentar."',NOW())") or die(mysql_error());
        echo "Din kommentar er tilføjet. Mange tak.<br /><br />";
    }
    }
    $getcomments = mysql_query("select id, forfatternavn, forfattermail, dato, kommentar from kommentarer where nyhedsid='".$nyhedsid."'");
    if (mysql_num_rows($getcomments) > 0) {
        while ($r = mysql_fetch_array($getcomments)) {
            echo "<a href='mailto:'.$r[forfattermail].''>".$r[forfatternavn]."</a> - ".$r[dato]."<br />";
            echo stripslashes($r[kommentar])."<br /><br />";
        }
    } else {
        echo "Ingen kommentarer tilføjet til denne nyhed endnu.";
    }
   
?>
<hr>
<form action="<? $_SERVER[PHP_SELF];?>" method="post">
<input type="hidden" name="id" value="<? $nyhedsid;?>">
Dit navn:<br>
<input type="text" name="navn" class="input"><br><br>
Din e-mail adresse:<br>
<input type="text" name="email" class="input"><br><br>
Kommentar:<br>
<textarea cols="0" rows="5" name="kommentar" class="input"></textarea><br><br>
<div align="center"><input type="submit" name="add_comment" value=" - Tilføj - " class="button">
</form>
<?
}
?>

men nu udskriver den fejlen hele tiden

<?php
// Vis alle kommentarer for en nyhed
include_once("../mysql_funktioner.php");
aabn_forbindelse_og_vaelgdb();
if ($_POST["id"]) {
    $nyhedsid = $_POST["id"];
} else {
    $nyhedsid = $_GET["id"];
}

if (!$nyhedsid) {
    echo "Intet nyheds-id angivet.";
} else {

    if ($_POST[add_comment]) {
        $navn = $_POST["navn"];
        $email = $_POST["mail"];
        if (!preg_match_all('/^([a-zA-Z0-9]+)([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $email, $matches)) {
            echo"Email adressen er ikke gyldig";
        } else {
            $kommentar = addslashes($_POST[kommentar]);
            $addcomment = mysql_query("insert into kommentarer (nyhedsid, forfatternavn, forfattermail, kommentar, dato) VALUES ('".$nyhedsid."','".$navn."','".$mail."','".$kommentar."',NOW())") or die(mysql_error());
            echo "Din kommentar er tilføjet. Mange tak.<br /><br />";
        }
    }
    $getcomments = mysql_query("select id, forfatternavn, forfattermail, dato, kommentar from kommentarer where nyhedsid='".$nyhedsid."'");
    if (mysql_num_rows($getcomments) > 0) {
        while ($r = mysql_fetch_array($getcomments)) {
            echo "<a href='mailto:'.$r[forfattermail].''>".$r[forfatternavn]."</a> - ".$r[dato]."<br />";
            echo stripslashes($r[kommentar])."<br /><br />";
        }
    } else {
        echo "Ingen kommentarer tilføjet til denne nyhed endnu.";
    }
?>
<hr>
<form action="<? $_SERVER[PHP_SELF];?>" method="post">
<input type="hidden" name="id" value="<? $nyhedsid;?>">
Dit navn:<br>
<input type="text" name="navn" class="input"><br><br>
Din e-mail adresse:<br>
<input type="text" name="email" class="input"><br><br>
Kommentar:<br>
<textarea cols="0" rows="5" name="kommentar" class="input"></textarea><br><br>
<div align="center"><input type="submit" name="add_comment" value=" - Tilføj - " class="button">
</form>
<?
}
?>

Der er også lige nogen fejl i din kode:
<? $_SERVER[PHP_SELF];?>  -->  <? echo $_SERVER[PHP_SELF];?>
<? $nyhedsid;?>  -->  <? echo $nyhedsid;?>

okay tak

men der er et problem nu den vil ikke tilføje det i min db heller ikke selv om der er @ med

Hvad skriver den ud til skærmen?

du kan se det her http://codemanager.1go.dk
og så kommetar

Prøv den her, rettet nogen ting som så lidt suspekte ud :)
<?php
// Vis alle kommentarer for en nyhed
include_once("../mysql_funktioner.php");
aabn_forbindelse_og_vaelgdb();
if ($_POST["id"]) {
    $nyhedsid = $_POST["id"];
} else {
    $nyhedsid = $_GET["id"];
}

if (!$nyhedsid) {
    echo "Intet nyheds-id angivet.";
} else {

    if ($_POST[add_comment]) {
        $navn = $_POST["navn"];
        $email = $_POST["mail"];
        if (!preg_match_all('/^([a-zA-Z0-9]+)([a-zA-Z0-9._-]*|[a-zA-Z0-9]*)@[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]*([a-zA-Z0-9]))+$/', $email, $matches)) {
            echo"Email adressen er ikke gyldig";
        } else {
            $kommentar = addslashes($_POST[kommentar]);
            $addcomment = mysql_query("insert into kommentarer (nyhedsid, forfatternavn, forfattermail, kommentar, dato) VALUES ('$nyhedsid','$navn','$email','$kommentar',NOW())") or die(mysql_error());
            echo "Din kommentar er tilføjet. Mange tak.<br /><br />";
        }
    }
    $getcomments = mysql_query("select id, forfatternavn, forfattermail, dato, kommentar from kommentarer where nyhedsid='$nyhedsid'");
    if (mysql_num_rows($getcomments) > 0) {
        while ($r = mysql_fetch_array($getcomments)) {
            echo "<a href='mailto:{$r[forfattermail]}'>{$r[forfatternavn]}</a> - {$r[dato]}<br />";
            echo stripslashes($r[kommentar])."<br /><br />";
        }
    } else {
        echo "Ingen kommentarer tilføjet til denne nyhed endnu.";
    }
?>
<hr>
<form action="<?php echo $_SERVER[PHP_SELF]; ?>" method="post">
<input type="hidden" name="id" value="<?php echo $nyhedsid; ?>">
Dit navn:<br>
<input type="text" name="navn" class="input"><br><br>
Din e-mail adresse:<br>
<input type="text" name="email" class="input"><br><br>
Kommentar:<br>
<textarea cols="0" rows="5" name="kommentar" class="input"></textarea><br><br>
<div align="center"><input type="submit" name="add_comment" value=" - Tilføj - " class="button">
</form>
<?
}
?>

den siger stadigvæk at email ikke er gyldig selb om der er @ og det hele

Prøv efter
echo"Email adressen er ikke gyldig";

at indsætte disse debug linjer:
echo"<pre>",
var_dump($matches);
echo"</pre>";

Så kan man se hvad der blev matched, og hvad der ikke gjorde.

array(5) {
  [0]=>
  array(0) {
  }
  [1]=>
  array(0) {
  }
  [2]=>
  array(0) {
  }
  [3]=>
  array(0) {
  }
  [4]=>
  array(0) {
  }
}

Ahhh, har fundet din fejl ... i din fom kalder du email feltet email. men du bruger $_POST['mail'] i din kode.

Ændre:
$_POST['mail']
til
$_POST['email']

så skulle det virke

hmm det forstod jeg ikke meget af

hehe ja kan se det

jeps så virker det tak for hjælpen

kan du ikke lige smide et svar

jo lige en ting mere hvordan læste du det ud fra den kode

HeHe, mange års erfaring hjælper ;-)
Så at arrayet af matches fra detregulære udtryk var rimlig tomt. Det betyder at den variabel der blev tjekket i sikkert også var tom, og så backtrackede jeg variablen $email baglæns gennem koden til der var noget som ikke passede.

okay på den måde så forstår jeg det men du skal havde tak for hjælpem

Nu er der givet svar, men jeg vil lige bemærke, at + også er et gyldigt tegn ihvertfald i brugerdelen af en email-adresse. Der er måske også andre gyldige tegn, tefckes regex ignorerer.

"men jeg vil på ingen måde stå inde for at den er 100% rigtig!"
Jeg er godt klar over den ikke er fuldstændig, men jeg havde i sin tid ikke lyst til at sidde og glo en RFC igennem for at få den 100% korrekt :)

Helt iorden, jeg fangede den også bare, fordi jeg netop har et par emailadresser med +.

Fidusen er nemlig, at hvis der bliver sendt til en bruger sandbox+exp på min server, så bliver mailen afleveret i brugeren sandbox's mailbox, og mit filter kan derefter bruge modtageradressen til en hel masse sjov.

Jeg kan derfor ofte se på spam, hvor de har fået min adresse fra.

Men du skal ikke være ked af, at du glemte det lille +, for de såkaldt professionelle, der har lavet jubiis brugersystem kan heller ikke finde ud af det. De har også glemt den.

hehe, det er jeg da glad for at høre ;o)

Her på experten har de dog styr på sagerne. Endnu et plus til E ;-)