Ошибки и оптимизация кода

@TestColor · Регистрация: 08.02.2020

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

Здравствуйте. Имеется код, необходимо найти ошибки и если есть возможность оптимизировать. Помогите плз.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
<?php
 
function sendJson($array)
{
if (isset($_POST['order_id'])) {
    $orderId = $_POST['order_id'];
    if ($order = DB::query('SELECT o1.`order_id`, o2.`order_status`
        FROM `orders` o1
                 INNER JOIN `orders` o2 ON o2.`order_id` = o1.`order_id`
        WHERE o1.`order_id` = {$orderId}')) {
        sendJson([
            'status'       => 'success',
            'order_id'     => $order['order_id'],
            'order_status' => $order['status'],
        ]);
    }
 
    sendJson([
            'status'  => 'fail',
            'message' => sprintf('Заказ с ID `%x` не найден', $orderId)
        ]);
}
 
 
   print(json_decode($array));
}

@ft4l · 08.02.2020, 15:46

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

PHP
1
' .... WHERE o1.`order_id` = {$orderId}'

В строках заключённых в апострофы переменные не интерполируются

Для чего передаётся $array .... только для распечатывания?
Может имеется ввиду elseпосле блока if ($order .... {}
если бы этот блок заканчивался раньше второй рекурсии?
но тогда как избежать бесконечного цикла если isset($_POST['order_id']) хоть тру хоть нет
когда если не else только при тру?

Это такие задвчки дают студентам? ))
Скажите профессору, что назначение этой функции в исходном TЗ не представляечтся возможным угадать )

@TestColor · 08.02.2020, 15:55 **[ТС]**

x_lab, Извиняюсь, изначально код такой

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
<?php
 
if (isset($_POST['order_id'])) {
    $orderId = $_POST['order_id'];
    if ($order = DB::query('SELECT o1.`order_id`, o2.`order_status`
        FROM `orders` o1
                 INNER JOIN `orders` o2 ON o2.`order_id` = o1.`order_id`
        WHERE o1.`order_id` = {$orderId}')) {
        sendJson([
            'status'       => 'success';
            'order_id'     => $order['order_id'],
            'order_status' => $order['status'],
        ]);
    }
 
    sendJson([
            'status'  => 'fail',
            'message' => sprintf('Заказ с ID `%x` не найден', $orderId);
        ]
    );
}
 
function sendJson($array)
{
    print(json_decode($array));
}

Но я вроде ничего не напутал, сделав так, как в первом посте?)

@ft4l · 08.02.2020, 16:43

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

если isset($_POST['order_id']) хоть тру хоть нет

не точно.... там два вложенных условных блока

Добавлено через 3 минуты
принцип тот-же, делается запрос в двойных кавычках, и решается вызывать-ли sendJson() два раза, в случае успешного запроса

Добавлено через 2 минуты

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

json_decode

json_encode вероятно

Добавлено через 31 минуту

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

$orderId = $_POST['order_id'];

возможно

PHP
1
$orderId = (int) $_POST['order_id'];

Новые блоги и статьи Все статьи Все блоги /
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .