С
Size: a a a
2020 March 03
С
тусил в питере?
да какое. хожу иногда выпить один ночью и по тихой пробелы в знаниях закрываю.
AK
в барах закрываешь?
K
<?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));
}
скажи - я переписал бы это говно на фреймворке
С
в барах закрываешь?
одно другому не мешает
AK
типа, так, вот тыкилу не пил, закрываем пробельчик...
С
скажи - я переписал бы это говно на фреймворке
даже если на голом пыхе я бы это говно на psr-интерфейсы какие-то перевёл бы хоть
С
типа, так, вот тыкилу не пил, закрываем пробельчик...
Да!
Белый русский выпил недавно
Белый русский выпил недавно
AK
а вообще так достаточно часто делают, кидают кусок кода и нужно описать какие там ошибки, как можно было бы лучше сделать
K
даже если на голом пыхе я бы это говно на psr-интерфейсы какие-то перевёл бы хоть
ну или так - на какой из предложенных уровней вам поднять этот код
https://medium.com/@DonnaInsolita/the-evolution-of-php-developer-4d3c2fdfa1ae
https://medium.com/@DonnaInsolita/the-evolution-of-php-developer-4d3c2fdfa1ae
АМ
АМ
ох уж этот всратый медиум
AK
Александр Морозов
там куку вроде говорили можно удалить
АМ
там куку вроде говорили можно удалить
да постоянно делаю так или инкогнито, но сервис все равно всратый(
AL
выглядит говенно
+
AK
@alexliberty устроился в итоге куда?
AL
@alexliberty устроился в итоге куда?
Пока ещё нет. Мыловцы молчат в пизде торчат, в Тануки разраб заболел и сказал сегодня-завтра скажет.
Вчера ещё собес был, но там вроде всё нормально, сделают офер на неделе
Вчера ещё собес был, но там вроде всё нормально, сделают офер на неделе
AK
Пока ещё нет. Мыловцы молчат в пизде торчат, в Тануки разраб заболел и сказал сегодня-завтра скажет.
Вчера ещё собес был, но там вроде всё нормально, сделают офер на неделе
Вчера ещё собес был, но там вроде всё нормально, сделают офер на неделе
в инструменты не стал?
NT
<?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));
}
Ну смотри, у тебя идет $orderId = $_Post['order_id']. Без чистки. Потенциальная инъекция. Дальше у тебя query в if. Нет проверки что order object of Order. Отправка статуса - гавно, тебе два раза нужно было инициализировать один и тот же массив ответа.