0

Делаю через XMLHttpRequest POST запрос к PHP файлу. Файл в свою очередь возвращает после выполнения массив, который показывает, все ли в порядке:

$response = [
    'message' => 'Something goes wrong.',
    'success' => false,
];

if ($_SERVER['REQUEST_METHOD'] != 'POST') {
    $response['message'] = "POST request was not detected.";
    echo json_encode($response);
}

if (!( isset($_POST['app-email']) && isset($_POST['app']) )) {
    $response['message'] = "Some POST variables were not detected.";
    echo json_encode($response);
}

//Некоторые действия с данными

if($success){ // $success - результат успешности операций с данными
    $request['message'] = "Successfully delivered."; //45 строка
    $request['success'] = true; //46 строка
    echo json_encode($request);
} else {
    $response['message'] = "Can't be delivered.";
}

echo json_encode($response);

PHP, как видно из кода, должен возвращать массив в формате JSON. И он возвращает, но почему-то сразу два, причем склеенные друг с другом:

{message:"Successfully delivered", success: true}{message:"Something goes wrong", success: false}

Из-за того, что они просто "склеены" в строке, JS при попытке распарсить эту строку выдает ошибку синтаксиса в месте стыка "}{".

Скорее всего это из-за echo, ведь оно не прерывает выполнение PHP. Отсюда два вопроса:

  1. Почему возвращается два разных массива, ведь в 45 строке (указал) мы меняем содержимое массива. А возвращается измененный и массив с данными "по умолчанию".
  2. Как грамотно сделать подобную логику? Чтобы в разных ситуациях возвращал массив с состоянием выполнения, но только один раз. (Когда использую exit вместо Echo, то возникает 500 ошибка сервера)

1 ответ 1

2

Ну все верно, вам нужно останавливать программу если распечатан готовый респонс, то-есть на каждом этапе echo json_encode($request); нужно делать die; для возвращения респонса джейсоном, или оформить как-то так, для избежания дублирования кода и операций:


try {
    if ($_SERVER['REQUEST_METHOD'] != 'POST') {
        throw new \DomainException("POST request was not detected.");
    }

    if (!( isset($_POST['app-email']) && isset($_POST['app']) )) {
        throw new \DomainException('Some POST variables were not detected.');
    }

    //Некоторые действия с данными

    if ($success) { // $success - результат успешности операций с данными
        echo json_encode(['success' => true, 'message' => 'Successfully delivered.']);
        die;
    }

    throw new \DomainException('Can\'t be delivered.');
} catch (\DomainException $exception) {
    echo json_encode(['success' => false, 'message' => $exception->getMessage()]);
    die;
}
6
  • Спасибо за ответ. А какой способ предпочтительнее? Через исключения или через echo+die? Способ с исключениями кажется мне более профитным. Не нужно беспокоиться о die и постоянно менять элементы массива в тех случаях, когда мы возвращаем ошибку.
    – ComeInRage
    Commented 26 мая 2021 в 16:55
  • И также вопрос, чисто из любопытства. Вот я допустил ошибку с die, но почему возвращалось не 2 одинаковых массива, а 2 разных? Мы ведь его в блоке if изменили.
    – ComeInRage
    Commented 26 мая 2021 в 17:00
  • 1
    Способ с исключениями, та как в таком случае можно минимизировать количество echo+die, и в программе будет 2 выхода на респонс, плохой в кетче и хороший в трай, естественно также зависит и от ситуации, но в даном случае если только 2 параметра message и success - кетч вполне подойдёт + клас ексепшена можно расширить и передавать дополнительные параметры и естественно также кетчей может быть несколько на несколько случаев, для ошибок по форме например и для обычного сообщения.
    – UKRman
    Commented 26 мая 2021 в 17:01
  • 1
    Потому что ниже используется переменная $response а для саксес почему-то $request, а так как программа не стопнулась роспечатало 2 переменные
    – UKRman
    Commented 26 мая 2021 в 17:04
  • Действительно. Спасибо большое. Если не ошибаюсь, то в PHP если указать стейтмент arr['key'] = 'something', то массив создастся, если его нет, что и привело к ошибке с разными массивами?
    – ComeInRage
    Commented 26 мая 2021 в 17:11

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.