Witam, w skrypcie rejestracji mam taki kawałek kodu:

[PHP] pobierz, plaintext 
      }
        if (!preg_match($email_chars, $email)) {
            error_s("Email ma zły format..");
            $isValid = false;
[PHP] pobierz, plaintext 

Dlaczego gdy przy rejestracji wpisuje adres w takiej formie: test@test-strona.pl to wyświetla mi błąd ,, Email ma zły format" ?

Nie przechodzi przez preg_match. Wklej większy kawałek kodu, w szczególności ze zmienną $email_chars.

[PHP] pobierz, plaintext 
function error_s($text) {
    echo("<p style=\"color:yellow;font-family:verdana;\">".$text);
    echo("<br /><br /><a style=\"color:orange;\" href=\"".$_SERVER["SCRIPT_NAME"]."\">Wróć...</a></p>");
}
 
$page = '
<form method="post" action="' . $_SERVER["SCRIPT_NAME"] . '">
<p style="text-align:center;">
<strong>' . $title2 . ' - ' . $title . '</strong>
<br /><br /><br />
Login:
<br /><input name="username" type="text" maxlength="14" /><br />
Hasło:
<br /><input name="password" type="password" maxlength="12" /><br />
Email:
<br /><input name="email" type="text" maxlength="50" />
<br /><input name="tbc" type="checkbox" checked="checked" /> TBC<br /><br /><br />
<button type="submit">Rejestruj</button>
</p>
</form>
';
 
$user_chars = "#[^a-zA-Z0-9_\-]#";
$email_chars = "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";
 
$con = @mysql_connect($ip, $user, $pass);
if (!$con) {
    error_s("Nie można połączyć się z bazą: ".mysql_error());
}
 
if (! empty($_POST)) {
    // dodajemy zmienną, która mówi, czy formularz został prawidłowo wypełniony
    // na początku przyjmujemy, że tak
    $isValid = true;
    if (( empty($_POST["username"])) || ( empty($_POST["password"])) || ( empty($_POST["email"])) || ( empty($_POST["tbc"]))) {
        error_s("You did not enter all the required information.");
        // jeśli jednak któreś pole jest puste, to formularz nie został prawidłowo wypełniony
        $isValid = false;
    } else {
        $username = strtoupper($_POST["username"]);
        $password = strtoupper($_POST["password"]);
        $email = strtoupper($_POST["email"]);
        $blocked = "1";
        $active = "0";
 $t = md5(uniqid(time())); //generujemy 32-bitowy token
        if (strlen($username) < 5) {
            error_s("Login jest za krótki.");
            // formularz nie został dobrze wypełniony również wtedy, gdy nazwa użytkownika jest zbyt krótka
            $isValid = false;
        } elseif (strlen($username) > 14) {
            error_s("Login jest za długi.");
 
            // lub zbyt długa
            $isValid = false;
 
        }
 
        if (strlen($password) < 5) {
            error_s("Hasło jest za krótkie.");
            $isValid = false;
 
        } elseif (strlen($password) > 14) {
            error_s("Hasło jest za długie.");
            $isValid = false;
 
        }
        if (strlen($email) < 5) {
            error_s("Email jest za krótki.");
            $isValid = false;
 
        } elseif (strlen($email) > 50) {
            error_s("Email jest za długi.");
            $isValid = false;
        }
 
        if (preg_match($user_chars, $username)) {
            error_s("Nazwa użytkownika zawiera niedozwolone znaki.");
            $isValid = false;
 
        } elseif (preg_match($user_chars, $password)) {
            error_s("Hasło zawiera niedozwolone znaki.");
            $isValid = false;
 
        }
        if (!preg_match($email_chars, $email)) {
            error_s("Email ma zły format..");
            $isValid = false;
 
 
        }
        if ($_POST['tbc'] != "on") {
            $tbc = "0";
        } else {
            $tbc = "1";
        }
        $username = mysql_real_escape_string($username);
        $password = mysql_real_escape_string($password);
        $email = mysql_real_escape_string($email);
        $qry = @mysql_query("select username from ".mysql_real_escape_string($r_db).".account where username = '".$username."'", $con);
        if (!$qry) {
            error_s("Error querying database: ".mysql_error());
            $isValid = false;
 
 
        } elseif ($existing_username = mysql_fetch_assoc($qry)) {
            foreach ($existing_username as $key=>$value) {
                $existing_username = $value;
            }
        }
        $existing_username = strtoupper($existing_username);
        if ($existing_username == strtoupper($_POST['username'])) {
            error_s("Ta nazwa użytkownika (login) jest już zajęta..");
            $isValid = false;
        }
        unset($qry);
        $qry = @mysql_query("select email from ".mysql_real_escape_string($r_db).".account where email = '".$email."'", $con);
    }
    if (!$qry) {
        error_s("Error querying database: ".mysql_error());
        $isValid = false;
    }
    if ($existing_email = mysql_fetch_assoc($qry)) {
        foreach ($existing_email as $key=>$value) {
            $existing_email = $value;
        }
 
    }
    if ($existing_email == $_POST['email']) {
        error_s("Ten adres email jest już zajęty.");
        $isValid = false;
 
    }
    if ($isValid) {
        $sha_pass_hash = sha1(strtoupper($username).":".strtoupper($password));
        $register_sql = "insert into ".mysql_real_escape_string($r_db).".account (username, sha_pass_hash, email, expansion, locked, token) values (upper('".$username."'),'".$sha_pass_hash."','".$email."','".$tbc."','".$blocked."','".$t."')";
        $qry = @mysql_query($register_sql, $con);
 
 $nadawca = "From: The Ocolus <gall@compzone.org>rn"; //ustawiamy nadawcę listu
 $temat = "Aktywuj swoje konto !"; //ustawiamy temat listu
 mail($email, $temat, "Dziekujemy $username za zalozenie konta w naszym serwisie. Zeby aktywowac Twoje nowo utworzone konto kliknij w ponizszy link <a href="http://www.theocolus.pl/activate.php?t=$t&n=$username" target="_blank">http://www.theocolus.pl/activate.php?t=&#0...=$username</a>  Wiadomosc ta zostala wyslana do Ciebie przez osobnika, ktory posiadal IP: $REMOTE_ADDR", $nadawca); //wysyłamy list na podany przez użytkownika adres e-mail znajdujący się w zmiennej $email
 $wynikmail = "Dziękujemy za założenie konta w naszym serwisie ! Na podany przed chwilą przez Ciebie adres e-mail została wysłana wiadomość, którą należy odebrać i aktywować swoje konto klikając w link aktywacyjny znajdujący się w wiadomości"; //jeżeli się to uda wyświetlamy stosowną informację
 
 
        if (!$qry) {
            error_s("Error creating account: ".mysql_error());
            $isValid = false;
        }
        echo("Konto zostało utworzone pomyślnie... <br /> <br />  $wynikmail <br /> <a href=\"".$_SERVER["SCRIPT_NAME"]."\">Wróć...</a></p>");
    } else {
        echo $page;
    }
 
} else {
    echo($page);
}
?>
[PHP] pobierz, plaintext 

Tak jak myślałem, dodaj po prostu do $email_chars myślniki i Ci wtedy takie adres przepuści.

tzn?

$email_chars -= "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";

albo

$email_chars - "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";


albo

$email_chars = "-";

Chyba ci chodzi o to że mam dodać myślnik do zbioru znaków:D ale jak

[PHP] pobierz, plaintext 
$email_chars = '/^[^0-9][A-z0-9_-]+([.][A-z0-9_-]+)*[@][A-z0-9_-]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/';
[PHP] pobierz, plaintext 

Tak. Zarówno w domenie, jak i nazwie użytkownika może myślnik występować, więc musisz go puszczać w obu częściach stringu.

PS. Sprawdzone, działa

A co zrobić gdy nie przepuszcza takich adresów: r.tomiczek@gmail.com z kropką.

Zrobić myk, aby przepuszczał

[PHP] pobierz, plaintext 
$email_chars = '/^[^0-9][A-z0-9_.-]+([.][A-z0-9_-]+)*[@][A-z0-9_-]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/';
[PHP] pobierz, plaintext 

PS Staraj się analizować kod, wtedy sam sobie poradzisz

I po co koło wymyślać? Zobacz filter_var i opcję walidacji email.

Może inaczej, co trzeba zrobić żeby przepuszczało wszystkie adresy?
Bo kolejny email z złym formatem to: 89tfet@gmail.com

Zobacz filter_var