В чем проблема добавления в BD?

Question

[Tizi]

Столкнулся с проблемой
Имею вот такую структуру формы

клик

<div class="glav-form-reg">
						<!-- <span class="close"></span> -->
						<div class="cont">

							<script type="text/javascript">
								$( ".button-reg" ).click(function() {
									var mail = $('.form-mail').val();
									$.post('add_user.php', {mail: mail}, function(data){
										$('.cont').html(data);
									});
								});
							</script>

							<form id="login-register" method="post">
								<div class="divError" id="error"></div>
								<h3>Регистрация.</h3>
								<input class="form-mail" value="E-MAIL.RU" type="email" name="email">
								<p>
									Регистрация не обязательная. она служит для рассылки оповещений о новых обновления.
								</p>

								<button type="submit" class="button-reg"  name="add">Login / Register</button>
							</form>
						</div>
					</div>

И файл Add_user.php

клик

if(isset($_POST['button-reg']))
{
	
	$email = strip_tags(trim($_POST['email']));

	mysql_query("
		INSERT INTO user_brain(email) VALUES ('$email')
		");

	mysql_close();


    echo 'Вы успешно зарегестрированы'; //если регистрация дейсвительно успешна

 }

При нажатии на кнопку регистрации должно отправятся в БД почта которая введена.
Не понимаю в чем ошибка. Сам запрос MySQL Работает ( проверял в PHPMyAdmin )
Но при нажатии на кнопку не какого эффекта нету.

P.s: Палками не кидайтесь )) только стал вникать в PHP.

Answer 1

[Иван Корюков]

А вот и буду кидаться палками, ловите сразу 4 =)
1) Вы хотите отправить ajax запрос, но вы делаете форму и работающую (button type="submit") кнопку отправки. Не надо так. Либо форма, либо ajax.
2) JS код, который вешает обработчик на кнопку располагается выше самой кнопки и не обёрнут в $(document).ready. Т.е. в тот момент когда исполняется этот код, самой кнопки ещё нет, и обработчик нажатия не вешается.
3) обработчик отправки формы реагирует на isset($_POST['button-reg']), однако когда вы делаете ajax запрос, вы не передаёте параметр "button-reg".
4) расширение php, которое позволяет вызывать функции mysql_*** устарело. Используйте mysqli или PDO

Comments

[Tizi]

А как мне решить эту проблему используя только AJAX ?

[Иван Корюков]

Tizi: достаточно просто исправить все замечания, которые описаны в моём ответе и в ответе D3lphi.
Т.е. - уберите из разметки форму. Опустите js ниже кнопки, или оберните его в $(document).ready,
при отправке ajax Запроса добавьте параметр button-reg. Ну и по хорошему бы начать использовать не mysql_***, а mysqli или PDO.

[Tizi]

Так ??

<div class="cont">
							<div class="divError" id="error"></div>
							<h3>Регистрация.</h3>
							<input class="form-mail" value="E-MAIL.RU" type="email" name="email">
							<p>
								Регистрация не обязательная. она служит для рассылки оповещений о новых обновления.
							</p>

							<button type="submit" class="button-reg"  name="add">Login / Register</button>
							<script type="text/javascript">
								$(document).ready(function() {		
									$( ".button-reg" ).click(function() {
										var mail = $('.form-mail').val();
										$.post('add_user.php', {mail: mail}, function(data){
											$('.cont').html(data);
										});
									});

								});

							</script>
						</div>

$email = strip_tags(trim($_POST['email']));

	mysqli_query("INSERT INTO user_brain(email) VALUES ('$email')");

    echo 'Вы успешно зарегестрированы'; //если регистрация дейсвительно успешна

[Иван Корюков]

Tizi: Не понимаю почему вы спрашиваете.

[Tizi]

Иван Корюков: Это спойлер ))) Там на скриншотах 2 ошибки когда поправил код.

[Иван Корюков]

Tizi: Да, я видел. Вы не сказали что вас беспокоят эти ошибки. Они крайне тривиальны чтобы заострять на них внимание и я подумал, что вы просто хотите чтобы я проверил - "так" ли вы написали код.
По поводу ошибок - это не просто набор символов - это осмысленные предложения на английском языке)
Первая говорит о том, что на 4 строке вы пытаетесь взять из массива элемент по индексу, которого в массиве нет. Обратите внимание - на js вы ajax запросом отправляете параметр mail, а на php пытаетесь из массива взять email. "mail" != "email"
Вторая ошибка - функция mysqli_query принимает как минимум два аргумента, а вы передали один.

[Tizi]

Иван Корюков: У меня в таблице 2 столбца. 1 - id а 2- это email. в принципе не нужно спрашивать id у пользователя что бы его передать ? тогда откуда я его должен получить раз он автоматически проставляется при появлении в BD новой строки?

[Иван Корюков]

Tizi: А при чём тут количество столбцов в таблице? Вы определение функции mysqli_query вообще смотрели? Функция принимает два аргумента. Первый - соединение с БД. Второй - строка запроса.
Вы передаёте только один - строку запроса, но передаёте в первую позицию - вместо первого аргумента - соединение с БД.

Answer 2

[D3lphi]

Уберите условный оператор:
if(isset($_POST['button-reg'])) // <-- Этот

Вы не передаете в форму "значение" кнопки. Следовательно, это условие все время ложно. Из параметров у вас передается только mail. Непонятно, зачем вам тут форма нужна, если вы ajax'ом передаете данные, не сериализуя их?

Вы путаете class и name. Даже, если бы вы использовали форму, то проверять нужно было так:
if(isset($_POST['add']))
Потому что, ключами массива являются ИМЕНА инпутов, а не их классы.

Пожалуйста, хватит использовать mysql_*! Это устаревшее расширение! mysqli_* и pdo существуют.