email validation in php

[ff123]

hey im trying to put email validation in php, i tried searching and the code i found does not work it shows 500 internal error here is my code

<?php 

$to = ""; 
$subject = "subject";
$mail = $_POST['mail']; 

if ($mail = $_POST['mail']) { 

$body = "E-Mail: $mail"; 

echo "success";

mail($to, $subject, $body, "From: $mail"); 
}  else { 
echo "error";
} 
?>

and here is the code im trying to add
http://www.go4expert.com/forums/showthread.php?t=1128

[forum_amnesiac]

you can use this code to validate an email address

$result = preg_split('/\b[A-Z0-9._%+-]+@(?:[A-Z0-9-]+\.)+[A-Z]{2,4}\b/i', $email); 

If $result is TRUE then proceed if FALSE then error

[ff123]

and where should i use this $result?

[forum_amnesiac]

This is where to use it, note the comment saying it can replace your 'if' statement.

<?php  

$to = "";  
$subject = "subject"; 
$mail = $_POST['mail'];  
$result = preg_split('/\b[A-Z0-9._%+-]+@(?:[A-Z0-9-]+\.)+[A-Z]{2,4}\b/i', $mail);  

if ($result){
    
if ($mail = $_POST['mail']) {  /*Why this line it will always be true, you can replace it with the 'if' above*/

$body = "E-Mail: $mail";  

echo "success"; 

mail($to, $subject, $body, "From: $mail");  
}  else {  
echo "error"; 
}  
?>

If you are submitting the email from an HTML form then personally I prefer to do any validation at the input stage.

It is fairly easy to use javascript to field validate a form

[forum_amnesiac]

This test is probably better, it stops consecutive '.'

$result = preg_split('/[a-z0-9!#$%&\'*+\/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&\'*+\/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?/i', $mail);

[techietim]

When possible, you should use the filter extension over regex.
http://php.net/filter

[ff123]

This is where to use it, note the comment saying it can replace your 'if' statement.

<?php  

$to = "";  
$subject = "subject"; 
$mail = $_POST['mail'];  
$result = preg_split('/\b[A-Z0-9._%+-]+@(?:[A-Z0-9-]+\.)+[A-Z]{2,4}\b/i', $mail);  

if ($result){
    
if ($mail = $_POST['mail']) {  /*Why this line it will always be true, you can replace it with the 'if' above*/

$body = "E-Mail: $mail";  

echo "success"; 

mail($to, $subject, $body, "From: $mail");  
}  else {  
echo "error"; 
}  
?>

If you are submitting the email from an HTML form then personally I prefer to do any validation at the input stage.

It is fairly easy to use javascript to field validate a form

thanks alot... well i was using javascript validate thingy, but i wanted with php.. thanks anyway

[ff123]

hey again.. one more thing im using this for my contact, where should i put that email validation thing so it can even validate in here..

if ( $message = $message && $name = $name && $fone = $fone && $mail = $mail ){

$body = "From: $name\n E-Mail: $mail\n Phone: $fone\n Message:\n $message";
} else {
echo ""
} 

[forum_amnesiac]

Try this

$result = preg_split('/[a-z0-9!#$%&\'*+\/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&\'*+\/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?/i', $mail); 

if ($result){
if ( $message = $message && $name = $name && $fone = $fone && $mail = $mail ){ 

$body = "From: $name\n E-Mail: $mail\n Phone: $fone\n Message:\n $message"; 
} else { 
echo "" 
}  
} else {
echo "" 
} 

It may easily be possible to write this neater but it should work

[ff123]

thats cool mate.. it works but please tell me what is the main difference between

preg_match("/^[\.A-z0-9_\-]+[@][A-z0-9_\-]+([.][A-z0-9_\-]+)+[A-z]{1,4}$/", $mail 

and

$result = preg_split('/[a-z0-9!#$%&\'*+\/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&\'*+\/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?/i', $mail);