Как правильно выводить данные php?

Question

[Ян Соколов]

Приветствую всех, кто заглянул.

Есть файл ../api/index.php, его вызываю с js по средствам XMLHttpRequest.
В файле api я вызываю http запрос стороннего сервиса и возвращаю тело ответа. Если запрос вылетает по времени, то возвращаю строку '{"header": {"status": 503,"error": "service unavailable"}}'

Для вывода ответа я использую print_r($result,true);

Только вот проблема. Когда со стороннего сервиса приходит ответ вовремя, то print_r работает хорошо, а когда я хочу вернуть вышеуказанную строку JSON с ошибкой 503, то на стороне клиента (JS) приходит пустая строка.
А если пользоваться не print_r, а echo, тогда вывод просто строки работает корректно, а JSON ответ стороннего сервиса выводится с "1" на конце.
Я с этим боролся как раз подставив true во второй параметр функции print_r.

Сторонним сервисом я называю сервер БД, php у нас просто как шлюз для прямого взаимодействия с клиентом. В общем для безопасности и чтобы не нагружать сервер БД огромным кол-вом сеансов и т п.
Версия PHP 8.1

Вот упрощенный код в файле api:

$ch = curl_init('serverDB.com/ping');
curl_setopt($ch, CURLOPT_TIMEOUT, 2);//TODO: delete after debug

$result = curl_errno($ch) ? '{"header": {"status": 503,"error": "service unavailable"}}' : curl_exec($ch);
curl_close($ch);
print_r($result,true);

Answer 1

[AUser0]

Вообще-то curl_errno() показывает РЕЗУЛЬТАТ запуска функции curl_exec(). Поэтому вызывать её раньше - просто глупо.
Разбирайтесь с результатом curl_exec(), если там должен быть JSON - значит просто отрезайте лишний "1" в конце строки.
И используйте именно echo(), именно им и нужно делать вывод в stdout. Использовать print_r(), даже с true - это одевать штаны через голову.

Comments

[Ян Соколов]

По поводу curl_erno перед curl_exec я тоже сначала так думал, не знаю почему, но прокатило)

И кстати не понимаю все равно почему curl_exec() сам по себе выдает в конец "1", но просто отрубил на уровне js'а вот так xhr.responseText.slice(0, -1) если кому будет нужно

[AUser0]

Lostcay, curl_errno() всегда вернёт 0, потому что самого запроса ещё не делалось, ошибки ЕЩЁ НЕТ! Соответственно у вас ВСЕГДА $result = curl_exec(), что бы вы там не рисовали в ручном JSON с 503.

[AUser0]

Lostcay, и ищите где у вас выводится эта лишняя 1. Дебажте, выясняйте, она действительно приходит снаружи, или нет? Если да - сразу вырезайте её из $result, нельзя передавать ошибку дальше! Правильнее даже полностью json_decode() пришедшего ответа, и давать реакцию в зависимости от результата.

[Ян Соколов]

AUser0, нашел, на стороне php все хорошо выводит, это js дурит.

Может сталкивались?

let xhr = new XMLHttpRequest();
xhr.open('GET', location.origin+'/api?method=ping',false);
xhr.send();
xhr.responseText

Выдает

{"header":{"status":200}}1

А просто запрос в адресную строку браузера или через postman без единички.
Вот откуда она тут может появляться?

[Ян Соколов]

Lostcay, В общем кажется нашел в чем была проблема. Все таки с php. Я когда брал $result = ...; оно вроде как уже выводило, а echo $result еще и "1" в конец выводило

[AUser0]

Lostcay, ну да, всё правильно. Вы ведь не просили CURLOPT_RETURNTRANSFER=true, поэтому вывод скачанного сразу в stdout. А в $result у вас хранится boolean результат curl_exec(), который true, потому что функция благополучно запросила и получила данные. А echo(true) даёт 1, потому что приведение типов.

[Ян Соколов]

AUser0, Теперь все понятно. Спасибо большое)

Answer 2

[Ипатьев]

Специально для тех, кого интересует этот животрепещущий вопрос и персонально для Максим Ткачев.

print_r($result,true); - это бред обкуренного жирафа. Эта строчка не выводит вообще ничего.
Про функцию print_r надо забыть, она была нужна только в 4 версии РНР.

Чтобы вывести любые данные, полученные курлом, надо использовать обычное echo
Предварительно задав CURLOPT_RETURNTRANSFER
А если что-то ещё не выводится, или выводится не так, то надо смотреть, где ещё в коде написан бред.