Как сделать валидацию инпутов?

Question

[Никита]

Сейчас при заполнении формы приходит два письма, одно на почту указанную в форме, другое на нашу почту.
Подскажите как сделать валидацию, хотя бы от пустых значений? (в html не получается)

в index.php код такой:

function sendmail(t){
 var name,phone,mail,type,price,item, catalog;
 type=t;
 if (type==1)
 {
 name = $('.popupcatalog .name').val();
 phone = $('.popupcatalog .phone').val();
 mail = $('.popupcatalog .mail').val();
 catalog = $('.popupcatalog .catalog').val();
 }
 if (type==2) {
 name = $('.pform .name').val();
 phone = $('.pform .phn1').val()+$('.pform .phn2').val()+$('.pform .phn3').val();
 price = $('#productpage .right .top .pprice').html();
 item = $('#productpage .right .top .ptitle').html();
 }
 if (type==3) {
 name = $('.popupcallback .name').val();
 phone = $('.popupcallback .phone').val();
 }
 
 $.post(
			"<?php echo get_template_directory_uri(); ?>/send.php", 
			{
			name: name,
      		phone: phone,
			mail: mail,
			price: price,
			item: item,
			type: type,
			catalog: catalog,
			},
			function(data) {
			if (data==1) {
                $('.popupout').fadeOut();
				$('#message').fadeIn();	
			}
			
			});
};

в send.php такой :

<?php
$sendto = "[e-mail]";

$name = $_POST['name'];
$phone = $_POST['phone'];
$mail = $_POST['mail'];
$price = $_POST['price'];
$item = $_POST['item'];
$catalog = $_POST['catalog'];
$tp = $_POST['type'];



if ($tp==1) {
$subject  = "Каталог";
}
if ($tp==2) {
$subject  = "Консультация";
}
if ($tp==3) {
$subject  = "Обратный звонок";
}

$msg  = " ";
$msg .= "Ваше имя: ".$name."<br/>";
$msg .= "Ваш телефон: ".$phone."<br/>";
if ($tp==1) {
$msg .= "Ваша ссылка:" . $catalog;
}
if ($tp==2) {
$msg .= "Товар: ".$item."<br/>";
$msg .= "Цена: ".$price."<br/>";
}

$headers  = "Content-type: text/html; charset=utf-8 \r\n"; 
$headers .= "From: элитмебель24 <[e-mail]>\r\n"; 
$headers .= "Bcc: [e-mail]\r\n";

if ($tp==1) {
mail($mail, $subject, $msg, $headers);
}
if (@mail($sendto, $subject, $msg, $headers)) {echo 1;}

?>

Comments

[Ilya Korablev]

Вообще, я бы сначала подумал над своим кодом. Это вообще не код, а говнокод.
Не юзайте @ чтобы не отображались ошибки
Почему вы в двойных кавычках не вставляете переменную, а выносите ее?

[Никита]

Ilya Korablev: Код писал не я, но разбираться пришлось мне. Если у вас есть идеи, куда тут можно написать пару строк для валидации пустых значений, то я вас внимательно слушаю.

Answer 1

[arab789]

Проверяй перед отправкой поста полученные данные. Допустим if( name.length>0 && phone.length>0 ... и т.д. ){ отправка поста }else{ jq пиши в блок error что не все поля заполнены }. Можно так же отдельно каждое проверять и по нему ошибки выводить, еще не было бы проверить, что ввел пользователь, если не в js, то на сервере обязательно, а то вводят много чего хорошего...

Comments

[Никита]

Уточните, еще как этот момент прописать? { jq пиши в блок error что не все поля заполнены }.

[arab789]

делаете див c id error. Затем в проверке типо такого: $("#error").html('ERROR login or password!');

[Сергей Бобров]

arab789: чувак, ну ты гонишь, для чего делать то, что должен делать браузер, ты бы с начала разобрался, почему не работает валидация, а потом начал делать костили, а так и живем

[arab789]

tachini: смотря какая валидация нужна. Более тонко ее настроить можно с помощью js. И это не мне нужно разбираться, почему не работает. Я человеку предложил вариант, как выход из его ситуации.

[Сергей Бобров]

arab789: короче вот решение:

<form action = 'javascript:void(null)' onsubmit="call()">
            ...
            <input name = 'name' type = 'text' required>
            ...
            <input type = 'submit'>
</form>
<script>
       function call() {
                 $.post(
      "<?php echo get_template_directory_uri(); ?>/send.php", 
      {
      name: name,
      phone: phone,
      mail: mail,
      price: price,
      item: item,
      type: type,
      catalog: catalog,
      },
      function(data) {
      if (data==1) {
                $('.popupout').fadeOut();
        $('#message').fadeIn();	
      }
      
      });
};
        }
</script>

И не надо доставать из формы нечего вручную, все подставиться на свое места, и валидация будет работать

[arab789]

tachini: еще раз повторяю, решение не мне нужно! я предложил вариант, а кто чем пользуется, это личное дело каждого.

Answer 2

[Сергей Бобров]

<input type = 'text' required>
required - указывает что поле обязательное для ввода

Comments

[Никита]

В вопросе я написал, что путем html не получается. Пишет, что сообщение отправлено, не смотря на required.

[Сергей Бобров]

Никита Терещенко: Добавьте вашу форму в описание, должно получаться

[Сергей Бобров]

Никита Терещенко: как у тебя происходит ajax запрос? пришей его к кнопке submit