Почему в консоли вылетатет ошибка unexpected character at line 1 column 1 of the JSON data, когда поля формы не заполнены?

Question

[Влад]

Когда все поля заполнены, то на клиент после отправки прилетает

Object { error: "", success: "Сообщение отправлено." }

. Всё как надо.
Но вот когда поля пустые, то в консоле вылетает ошибка:

Uncaught (in promise) SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

Error: Promised response from onMessage listener went out of scope

А должно прилетать

Object { error: "Сообщение не отправлено. Форма заполнена некорректно.", success: "" }

Вот код:

// ОБРАБОТКА ОШИБОК, AJAX
  const form  = document.querySelector(".order-form");
  const formInputs = document.querySelectorAll(".order-form input");
  const url = "/php/send-order-form.php";

  // функция очистки
  function cleanForm() {
    //циклом проходимся по ним и обнуляем значение
    for (let item of formInputs) {
      item.value = "";
    }
  }

  // Функция отправки формы fetch
  async function postData(url, data = {}) {
    const response = await fetch(url, {
      method: "POST",
      body: data,
    });
    return await response.json();
  }

  // отправка
  // при отправке формы любым способом
  form.addEventListener("submit", function (e) {
    // запрещаем стандартное действие
    e.preventDefault();
    // создаем объект новый
    let data = new FormData(form);

    // передаем в фукцию fetch данные и получаем результат
    postData("/php/send-order-form.php", data).then((data) => {
      // обработка ответа от сервера
      console.log(data);
      if (data.error == "") {
        alert(data.success);
        cleanForm();
      } else {
        alert(data.error);
      }
    });
  });
};

<?php
/***   Получаем данные из формы отправленные скриптом ***/
// перед присвоением в переменную, проверяем есть ли данные
if (!empty($_POST["name"])) $name = $_POST['name'];
if (!empty($_POST["phone"])) $phone = $_POST['phone'];
if (!empty($_POST["message"])) $message = $_POST['message'];

/***   Проверка данных ***/
$OK = false;
if ($name and $phone and $message) $OK = true;

/***   Отправка данных ***/
if ($OK) {
    // отправка

    // если отправка успешна
    $result["error"] = "";
    $result["success"] = "Сообщение отправлено.";
} else {
    $result["error"] = "Сообщение не отправлено. Форма заполнена некорректно.";
}

/***   Возврат результата отправки ***/
header("Content-Type: application/json");
echo json_encode($result);
?>

С чем это связано?

Comments

[Everything_is_bad]

ну так посмотри что именно тебе приходит, перед тем как это будет парситься в json

[Дмитрий]

Нет такой переменной $phon
Букву e на конце забыли. Но по идее этот ворнинг должен быть и при успешном выполнении….

[ProjectSoft]

Влад, с переменными аккуратней. Они должны быть объявлены.
Проверяйте через isset.

$name = isset($_POST["name"]) ? $_POST["name"] : false;
$phone = isset($_POST["phone"]) ? $_POST["phone"] : false;
$message = isset($_POST["message"]) ? $_POST["message"] : false;

// Дефолт - сообщение не отправлено
$result = new stdClass;
$result->error = 1;
$result->message = "Сообщение не отправлено. Форма заполнена некорректно.";

if($name && $phone && $message) {
	// Данные заполнены. Отправляем
	$result->error = 0;
	$result->message = "Сообщение отправлено.";
}

header("Content-Type: application/json");
echo json_encode($result);

[Ипатьев]

ProjectSoft, Пишите чётче. при таком построении фразы рекомендация использовать isset выглядит как рекомендация по решению проблемы.
С нормальным объяснением это стоит написать в виде ответа

[ProjectSoft]

Ипатьев, вообще то есть справочный материал https://www.php.net/manual/ru/ .
Зачем и почему именно isset узнать можно именно там. И бежать за разъяснениями нужно именно туда. Топикстартера Влад нужно заставить туда сбегать ))) Просто, чтобы знал куда, где, как, что искать.
Ответ и не дан идеальным. Он дан только как рекомендация. Ибо проверок ещё целая куча, валидация от имени до сообщения и почему стандартной валидации на php всё-таки мало.

[Влад]

Дмитрий, я уже в другой ветке ответил:

Ребята, сорян, это у меня на хабре случайно так вышло. Ввёл вас в заблуждение. Конечно же у меня phone!
Иначе бы с сервера не прилетало Object { error: "", success: "Сообщение отправлено." }.
Я подправил вопрос.
Извиняюсь.

Answer 1

[Олег]

С чем это связано?
С грязным стилем написания кода.



Если задавить вывод ошибок, то код начнет работать.
!!!! НО !!!!!

Как быстро вы отловите ошибку с тем, что подчеркнуто красным ?

Comments

[Ипатьев]

Стоило написать, что именно в этом коде "грязно". иначе это просто бессмысленное сотрясение воздуха
Плюс если бы в реальном коде была опечатка с $phon, то тогда бы и заполненная форма не отправлялась

[Олег]

Ипатьев, фанат это вы ?

[Влад]

Ребята, сорян, это у меня на хабре случайно так вышло. Ввёл вас в заблуждение. Конечно же у меня phone!
Иначе бы с сервера не прилетало Object { error: "", success: "Сообщение отправлено." }.
Я подправил вопрос.
Извиняюсь.

[Влад]

Парни, у меня теперь другая вечеринка.
Я поднапрягся, почитал и написал вроде бы правильный код, потому что локально под Open Server код работает, а когда я закачал его к хостеру на сервак, то сообщение почему-то не отправляется и выскакиевает "Сообщение не отправлено. Форма заполнена некорректно."
WTF? Хостер beget.com
Вот код:

<?php
/////////////////// Send message  ///////////////////////

if ($_SERVER['REQUEST_METHOD']=='POST') {

  foreach($_POST as $key => $value) {
    $value=trim($value);
    $_POST[$key]=$value;
  }

  $err=0;
  if (empty($_POST["name"])) $err=1;
  if (strlen($_POST["name"]) > 100) $err=1;
  if (strlen($_POST["phone"]) > 18  || strlen($_POST["phone"]) < 18) $err=1;
  $phone_ok = (preg_match("/^\+7 \(\d{3}\) \d{3}-\d{2}-\d{2}$/", $_POST["phone"]));
  if (!$phone_ok && $_POST["phone"]) $err=1;
  if (empty($_POST["message"])) $err=1;

  if ($err) {
    exit(json_encode(["error" => "Сообщение не отправлено. Форма заполнена некорректно."]));
  }

  if (!$err) {

    $mail="Сообщение с http://".$_SERVER['HTTP_HOST']."\n";
    $mail.="Имя: ".$_POST['name']."\n";
    $mail.="Тел.: ".$_POST['phone']."\n";
    $mail.="Страница запроса: ".$_SERVER['HTTP_REFERER']."\n";
    $mail.="Сообщение: ".$_POST['message']."\n";

    $to="[e-mail]";
    $subject="Сообщение с http://".$_SERVER['HTTP_HOST'];
    $headers="Content-type: text/plain; charset=UTF-8";

    if(mail($to, $subject, $mail, $headers)) {
      exit(json_encode(["success" => "Сообщение отправлено.", "error" => ""]));
    } else {
      exit(json_encode(["error" => "Сообщение не отправлено. Форма заполнена некорректно."]));
    }
  }
}

[Олег]

два разных случая выводят один и тот же текст ошибки. Точку чтоли лишнюю поставили, чтобы не гадать где сработало :)
Читайте
https://www.php.net/manual/ru/function.mail.php

Параметр additional_params задают, чтобы передать дополнительные флаги в виде аргументов командной строки для программы, настроенной в директиве sendmail_path для отправки писем. Например, этим пользуются при отправке письма агентом sendmail с аргументом -f, чтобы установить адрес отправителя конверта.

Делаю ставку, что поможет процитированное

[Влад]

Олег, Вот так что ли нужно было сделать if(mail($to, $subject, $mail, $headers, "-f"))?
Если так, то не помогает.

[Олег]

Влад, внимательно не читали. -f емайл отправителя
так как эксплуатация функции mail() основная цель взломов, то по дефолту ее вызов запрещен
или как минимум просто не донастроен.
ключ -f - это чтобы не лазить по конфигам
К начальному вопросу уже не относиться
https://qna.habr.com/search?q=php+mail%28%29+error

Answer 2

[Ипатьев]

1. Ошибки взаимодействия с сервером надо смотреть не в консольке, а во вкладке Сеть. Там в подвкладке Response или как она там в переводе называется, вы увидите ответ сервера.

2. Как правильно написал в комментариях ProjectSoft, проблема в том, что вы определяете переменные только при успешном прохождении проверки. Но если данных нет, то ни переменной $name, ни $phone, ни $message в коде не существует. И в итоге РНР вам законно выводит ошибку при попытке обратиться к этим переменным.
По итогам проверки переменная должна быть объявлена в любом случае. Это можно сделать либо с помощью конструкции else, либо как показано в примере ProjectSoft, с помощью тернарного оператора.
Переменную $OK можно определить как результат логического выражения, без всяких условий
$OK = $name and $phon and $message;
Но сама по себе она не имеет смысла. У вас результатом кода $name and $phon and $message УЖЕ является логическое выражение, которое вы УЖЕ можете подставить в условие. И зачем это делать с пересадкой - непонятно.

3. При этом надо понимать, что и isset, и empty в данном случае это подавление ошибок. И без нужды их лучше не применять. А просто написать

$name = $_POST['name'];
$phone = $_POST['phone'];
$message = $_POST['message'];

Тогда РНР сможет сообщить и об ошибках при составлении формы.

Но чтобы эти ошибки зря не лезли при запросе файла, лучше весь код целиком заключить в проверку типа
if ($_SERVER['REQUEST_METHOD'] === 'POST')

Как вариант - проверять каждую переменную на существование и пустоту отдельно, и выдавать соответствующие ошибки, типа "Поле name не отправлено", "Поле name не заполнено", и т.д.

Comments

[Олег]

Совсем выше согласен, но нет ответа на вопрос почему.

В PHP за вывод ошибок отвечают две переменных. Показывать их вообще и какие именно.
На публичном сервере (проде) вывод ошибок в ответ пользователю должен быть запрещен.
Складывайте их в приватные логи.

При разработке error_reporting в идеале E_ALL. Но жизнь не идеал, приходиться мириться и отключать по необходимости E_STRICT и/или E_DEPRECATED.

Грязным стилем программирования я назвал, код выводящий при выполнении сообщения типа WARNING.

В вашем случае предупреждение, что переменная не была инициализирована.

Почему не корректный json? - вывод сообщений об ошибках до тела JSON

[ProjectSoft]

При этом надо понимать, что и isset, и empty в данном случае это подавление ошибок.

Ипатьев Серьёзно?
Опустим empty ибо оно в вопросе рано указано... почему же isset вдруг стал подавителем ошибок? Абсолютно не согласен. В данной ситуации просто составляющая логической цепочки.

[Влад]

ProjectSoft, что-то я не понял по-поводу isset().
Это вообще к ошибкам не имеет никакого отношения.
Переменные с пустым содержимым в любом случае будут true.
Какой вообще смысл применятьisset()?
То ли я дурак? То ли лыжи не едут?

[Ипатьев]

Влад, ещё как имеет.
И что значит "какой смысл применять isset"? Вы сами его и применяете. Только по-дурацки.
И кроме этого проверяете свои переменные на пустоту два раза.

if (!empty($_POST["name"])) - здесь вы проверяете переменную на существование (isset)и не-пустоту.
А потом еще раз на не-пустоту, вот здесь if ($name and $phone and $message)

А ProjectSoft в своем примере предлагает вам сделать по-другому: сначала проверить на существование, и присвоить пустое значение, если переменная не существует.
А потом проверять на пустоту.
Это более логичное поведение в общем случае

Но в данном случае проверка на isset не нужна. В вашем случае элемента массива $_POST с ключом 'name' может не существовать только если запрос был сделан не методом POST или если в форме нет поля ввода с таким именем. Оба этих случая являются форс-мажорными, и вам как раз полезно будет видеть ошибку, которую напишет в этом случае РНР.

[Влад]

Я всё переделал, смотрите соседнюю ветку.

[Ипатьев]

Зачем вы задаете вопросы, если вас не интересуют ответы?

[ProjectSoft]

Влад,

Переменные с пустым содержимым в любом случае будут true.

Глубоко ошибаешься. Хотя бы попробуй.
Если строка не имеет ни одного символа условие вернёт false. Если есть хоть один символ, к примеру пробел, уже вернёт true.

[Влад]

ProjectSoft, а чего тут пробовать?

[Влад]

Тему нужно вообще грохнуть. Она лишена смысла. Код был изначально кривой, в всё давно переписал + проблема была в том, что заказчик не привязал свой номер телефона к аккаунту на timeweb, поэтому почта не отправлялась.

Тему уничтожаю.

[ProjectSoft]

Влад,

а чего тут пробовать?

Ты не понял для чего isset и как проверить истинность.

[Ипатьев]

ProjectSoft, он конечно туговат, но в данном случае он имел в виду, что "При проверке переменных с пустым содержимым через isset в любом случае будут true, а мне надо проверять и на пустоту тоже" что вполне логично.
А вы теряете ход мысли и начинаете уже вещать не про конкретную задачу, а про отдельные слова, вырванные из контекста. Вы конечно будете правы, но рассказывать это автору вопроса бессмысленно - он не оценит всю глубину ваших познаний.