Почему скрипт работает через раз?

Question

[Денис Плюс]

Вроде бы элементарный скрипт, но почему-то срабатывает через раз. В чем может быть дело?

index.php
<form class="form-inline" action="" method="post" name="form">
	 <input type="text" class="form-control" name="name" id="name" placeholder="Введите название фильма...">
	<button class="btn btn-primary" name="add" id="add">Добавить</button>
 </form>

$(document).ready(function(){
      $('button').on('click',function(e){
         var $name = $('#name').val();
         $.post("connect.php", {data: $name});
        });
        });

connect.php
 if(isset($_POST['data']))
     {
         $file = fopen('xsx.txt', 'w+');
         fwrite($file, $_POST['data']);
         fclose($file);
         header('Location:http://films/');
}

Comments

[lloydbanks]

Так вы не запретили дефолтное поведение нажатия на кнопку, вот у вас самбит и срабатывает быстрее, чем произойдет аякс.
Перед $.post(...
добавьте e.preventDefault()

Answer 1

[Талян]

Тут вроде перед name доллар лишний был

$(document).ready(function(){
      $('button').on('click',function(e){
         var name = $('#name').val();
         $.post("connect.php", {data: name});
        });
        });

Comments

[Денис Плюс]

$ в имени переменной тут никакой рояли не играет. jQuery она не мешает. Такое ощущение, что лагает серверная часть, обнаружил, что если подождать ~60сек. и повторить попытку, все работает. У меня стоит обновленный Denwer с PHP7, раньше его не использовал. Попробую другой сервак поставить.

Answer 2

[Георгий Баручян]

1) добавьте в конце обработчика return false; иначе у вас будет происходить submit формы.
2) сделайте редирект не в php скрипте, а в обработке овета ajax

Comments

[Денис Плюс]

1) e.preventDefault() - это то же самое?
2) Как редирект сделать в данном случае?

[Георгий Баручян]

Иван Абрек,
1) тоже самое, но вы это не используете в коде
2) нужно передать success, подробнее тут

$(document).ready(function(){
            $('button').on('click',function(e){
                e.preventDefault();
                var $name = $('#name').val();
                $.post("connect.php", {data: $name}, function(){
                    window.location.href = '/';
                });
            });
        });

[Денис Плюс]

Георгий Баручян, спасибо. Только начал разбираться, как-то напрягает обилие разных скобок, сложно понять, что где.

[edward_freedom]

Иван Абрек, поставь например phpstorm и там будет видно, где какая скобка

Answer 3

[Vitsliputsli]

А что хотите получить? Если нужно дописывать в файл введенные пользователем строки, то не нужно никакой form, допиливайте ajax. Если нужно после ввода пользователя уходить на http://films, то не нужен никакой ajax, поправьте action и сделайте submit в form.