Помогите доработать форму обратной связи.

<?php  



if (!isset($_POST['name']) || strlen($_POST['name'])<20)  

{  

    echo "Сообщение об ошибке";  

} 

elseif (!isset($_POST['surname']) ||  strlen($_POST['surname'])<20)  

{  

     echo "Сообщение об ошибке";      

} 

else 

{ 

    $adds = "Content-Type: text/plain; charset=windows-1251"; 

    $to = "******@mail.ru";  

    $subject = "Тема сообщения";  

    $message = "Фамилия: {$_POST['surname']}\nИмя: {$_POST['name']}";  

    mail ($to,$subject,$message,$adds) or die( "Не могу отправить письмо !!!" );  

    echo "Спасибо за отправку вашего сообщения<br /><a href='/'>Вернуться на главную</a>";  

} 

?>

Сейчас не работает только проверка на количество символов!

Количество символов в полях должно 20 или больше. По логике для фамилии и имени это много, поставьте 4.

В какой версии и почему?

Блин, это тут стоит значит, min кол символов, я как то не подумал...

Теперь осталось только заточить под мои нужды!

Всем спасибо!!!

Вы хотите вывести все ошибки сразу? Это правильный подход. Но код надо изменить, например

$error=array();
if(!isset($_POST['name']) || strlen($_POST['name'])>20 || strlen($_POST['name'])<3)
	$error[]='error name';
if(!isset($_POST['surname']) || strlen($_POST['surname'])>20 || strlen($_POST['surname'])<3)
	$error[]='error surname';
if(count($error))
{
	foreach($error as $k=>$v)
		echo $v."<br>";
}
else
{
    $to = "******@mail.ru";
    $subject = "Тема сообщения";
    $message = "Фамилия: {$_POST['surname']}\nИмя: {$_POST['name']}";
    mail ($to,$subject,$message) or die( "Не могу отправить письмо !!!" );
}

Длина полей от 3 до 20 символов.

ошибки не будет

if(TRUE || дальше нет смысла проверять, так как уже TRUE)

Shtogrin, а как работать будет?

$error=array();

if(!isset($_POST['name']) || strlen($_POST['name'])>20 || strlen($_POST['name'])<3)

    $error[]='error name';

if(!isset($_POST['surname']) || strlen($_POST['surname'])>20 || strlen($_POST['surname'])<3)

    $error[]='error surname';

if(count($error))

{

    foreach($error as $k=>$v)

        echo $v."<br>";

}

else

{

    $to = "******@mail.ru";

    $subject = "Тема сообщения";

    $message = "Фамилия: {$_POST['surname']}\nИмя: {$_POST['name']}";

    mail ($to,$subject,$message) or die( "Не могу отправить письмо !!!" );

} 

Мне надо что бы выводилось типа, не заполнено поле имя и поле фамилия - если они не заполнены.

$k=>$v - что то не понимаю зачем.

А как вы рисуете саму форму? Этот html код надо вставить в скрипт.

Это мы собрали все ошибки в массив error и в цикле foreach выводим

ошибка 1<br>

ошибка 2<br>

Можете выслать html код формы или если есть данные какого-то временного хостинга для этого скрипта на roman [at] shtogrin.com.

Кстати, в каком-то из языков в отличие от php и других нормальных языков :) логика извращённая: сначала проверяется правое условие, а затем левое.. точно, не помню, в ASP чтоли.