Почему при новой отправке ajax запроса вызываются предыдущие?

Question

[Programmir]

Идет отправка формы на почту через ajax. После каждой отправки делаю reset формы. Потом не обновляю страницу и посылаю новые данные и в done повторяется отправка предыдущих заполненных полей. Например, если я пять раз отправил письмо, то при пятой попытке этот done выполняется пять раз, отправляя пять раз письмо с предыдущими полями. Как это исправить?

Comments

[Алексей Скляров]

код свой прикрепите для начала

[Programmir]

$("#sendzajav").on('click', function(e){
	

	var emailval  = $("#email").val();
	var nameval    = $("#name").val();
	var phoneval    = $("#phone").val();
	var namelen    = nameval.length;
  	var phonevalid = ValidPhone(phoneval);

 	if(phonevalid == false) {
    	$("#phone").addClass("error");
  	}
  	else if(phonevalid == true){
   		$("#phone").removeClass("error");
 	}
 	var testname = inputlen(namelen,"name");

 	if (testname == true && phonevalid == true){
		 $("#sendzajav").val("Идет Отправка...");
		 $(this).parent('form').submit(function() {
			var th = $(this);
			$.ajax({
				type: "POST",
				url: "mail.php", 
				data: { "Имя": nameval, "Телефон": phoneval, "Почта": emailval  }
			}).done(function() {
				alert("Спасибо за Вашу заявку!");
				setTimeout(function() {				
					$('form').trigger("reset");
				}, 1000);
			});
			return false;
		});  		
 	}
});

Answer 1

[Андрей Ежгуров]

Потому, что по событию click создаётся новый обработчик события submit - который добавляется к уже существующим обработчикам и в котором фиксируются значение полей в момент срабатывания click.
Обработчик click вообще не нужен.

$("#sendzajav").submit(function (e) {
    e.preventDefault();

    var emailval = $("#email").val();
    var nameval = $("#name").val();
    var phoneval = $("#phone").val();

    if (ValidPhone(phoneval)) {
        $("#phone").removeClass("error");
    } else {
        $("#phone").addClass("error");
        return;
    }

    if (!inputlen(nameval.length, "name")) {
        return;
    }

    $("#sendzajav").val("Идет Отправка...");
    $.ajax({
        type: "POST",
        url: "mail.php",
        data: {"Имя": nameval, "Телефон": phoneval, "Почта": emailval}
    }).done(function () {
        alert("Спасибо за Вашу заявку!");
        setTimeout(function () {
            $('form').trigger("reset");
        }, 1000);
    });
});

Comments

[Programmir]

ясно, спасибо