Работа с капчей. Что не так?

@terri · Регистрация: 12.05.2014

Author24 — интернет-сервис помощи студентам

Сделал капчу при добавлении комментариев и не могу соединить это все в 1. Если капча введена правильно - отсылаем комментарий, иначе - не отсылаем(форма не должна отсылаться, когда капча неверна).
Сейчас ошибку мне выдает на 10 строке. - скрин ошибки
Файл comments

PHP

if (isset($_POST["captcha"])) {
    if ($_POST["captcha"] == $_SESSION["code"]) {   
        echo "Комментарий добавлен";
            $name     = $_POST['com_name'];
            $message  = $_POST['com_mess'];
            $hotel     = $_GET['id'];
              
            $query = 'INSERT INTO comments (hotel_id, text, name) VALUES (:hotel, :message, :name)';
            $stmt = $pdo->prepare($query);
            $stmt->execute(array(':hotel'=>$hotel, ':message'=>$message, ':name'=>$name));    
 
    } else {
    die ("Неверно введена капча");
    }
 }

js файл

Javascript

$('#send_com').click(function() {
  
    var name = $('#com_name').val();
    var email = $('#com_email').val();
    var message = $('#com_mess').val();
    var captcha = $("#captcha").val();
 
          var dataString = 'captcha=' + captcha;
          $.ajax({
          type: "POST",
          url: "check_captcha.php",
          data: dataString,
          success: function(html) {
          alert(html);
          },
          })
  
  });

@pav1uxa · 08.11.2015, 11:13

terri, а где тут передаются данные? Я вижу POST запрос на скрипт check_captcha.php, в запросе передается всего одна переменная - captcha. А скрипт пытается засунуть данные $_POST['com_name'], $_POST['com_mess'], $_GET['id'] в БД. Вот только откуда они возьмутся? SQL Вам так и пишет - пытаетесь засунуть пустое значение в ячейку таблицы, которая не может содержать пустого значения.

Советую включить отображение ВСЕХ ошибок. Тогда бы на эти строчки у Вас появились варнинги и Вы сразу бы все поняли:

PHP

1
2
3

            $name     = $_POST['com_name'];
            $message  = $_POST['com_mess'];
            $hotel     = $_GET['id'];

Включать хотя бы на время отладки.

@terri · 08.11.2015, 13:35 **[ТС]**

pav1uxa, добавил в условие на 2-й строке
(isset($_POST['com_name'] !='') и так далее - не работает

@Jewbacabra · 08.11.2015, 14:43

Сообщение от terri

добавил в условие на 2-й строке
(isset($_POST['com_name'] !='') и так далее - не работает

Естественно. От добавления проверки был ли передан параметр передаваться он не станет.

@terri · 08.11.2015, 15:23 **[ТС]**

Jewbacabra, вы предлагаете поставить условие в 1 строку? Как бы переменные я определил, что-то не хочет

@Jewbacabra · 08.11.2015, 15:45

terri, нет. Еще раз:
передается только

Javascript

1
2
3

var dataString = 'captcha=' + captcha;
// ...
data: dataString,

А в запросе используется еще и

PHP

$name     = $_POST['com_name'];
            $message  = $_POST['com_mess'];
            $hotel     = $_GET['id'];
              
            $query = 'INSERT INTO comments (hotel_id, text, name) VALUES (:hotel, :message, :name)';

И что ему вставлять в бд, если ничего не передано?

@terri · 08.11.2015, 17:30 **[ТС]**

Jewbacabra, передавать в таком виде?

Javascript

1	data: "captcha="+captcha+"&com_name="+name+"&com_email="+email ???

@Jewbacabra · 08.11.2015, 17:38

terri, да, но лучше в виде объекта

Javascript

data: {
    captcha: captcha,
    com_name: name,
    com_email: email
}

@terri · 08.11.2015, 17:44 **[ТС]**

Jewbacabra, а в файле comments.php( в первом посте) все верно?

@terri 8 / 8 / 4 Регистрация: 12.05.2014 Сообщений: 233
	08.11.2015, 13:35 [ТС]	3
	pav1uxa, добавил в условие на 2-й строке (isset($_POST['com_name'] !='') и так далее - не работает 0

@Jewbacabra 4925 / 3920 / 1620 Регистрация: 24.04.2014 Сообщений: 11,441
	08.11.2015, 14:43	4
	Сообщение от terri добавил в условие на 2-й строке (isset($_POST['com_name'] !='') и так далее - не работает Естественно. От добавления проверки был ли передан параметр передаваться он не станет. 0

@terri 8 / 8 / 4 Регистрация: 12.05.2014 Сообщений: 233
	08.11.2015, 15:23 [ТС]	5
	Jewbacabra, вы предлагаете поставить условие в 1 строку? Как бы переменные я определил, что-то не хочет 0

@terri 8 / 8 / 4 Регистрация: 12.05.2014 Сообщений: 233
	08.11.2015, 17:44 [ТС]	9
	Jewbacabra, а в файле comments.php( в первом посте) все верно? 0