Size: a a a

Генератор непрочитанных сообщений

2020 March 03

С

Санжар in Генератор непрочитанных сообщений
источник

С

Санжар in Генератор непрочитанных сообщений
Alexander Kladov
тусил в питере?
да какое. хожу иногда выпить один ночью и по тихой пробелы в знаниях закрываю.
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
в барах закрываешь?
источник

K

Konstantin in Генератор непрочитанных сообщений
Санжар
<?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));
}
скажи - я переписал бы это говно на фреймворке
источник

С

Санжар in Генератор непрочитанных сообщений
Alexander Kladov
в барах закрываешь?
одно другому не мешает
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
типа, так, вот тыкилу не пил, закрываем пробельчик...
источник

С

Санжар in Генератор непрочитанных сообщений
Konstantin
скажи - я переписал бы это говно на фреймворке
даже если на голом пыхе я бы это говно на psr-интерфейсы какие-то перевёл бы хоть
источник

С

Санжар in Генератор непрочитанных сообщений
Alexander Kladov
типа, так, вот тыкилу не пил, закрываем пробельчик...
Да!
Белый русский выпил недавно
источник

С

Санжар in Генератор непрочитанных сообщений
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
а вообще так достаточно часто делают, кидают кусок кода и нужно описать какие там ошибки, как можно было бы лучше сделать
источник

K

Konstantin in Генератор непрочитанных сообщений
Санжар
даже если на голом пыхе я бы это говно на psr-интерфейсы какие-то перевёл бы хоть
ну или так - на какой из предложенных уровней вам поднять этот код

https://medium.com/@DonnaInsolita/the-evolution-of-php-developer-4d3c2fdfa1ae
источник

АМ

Александр Морозов in Генератор непрочитанных сообщений
источник

АМ

Александр Морозов in Генератор непрочитанных сообщений
ох уж этот всратый медиум
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
Александр Морозов
там куку вроде говорили можно удалить
источник

АМ

Александр Морозов in Генератор непрочитанных сообщений
Alexander Kladov
там куку вроде говорили можно удалить
да постоянно делаю так или инкогнито, но сервис все равно всратый(
источник

AL

Alex Libertovsky in Генератор непрочитанных сообщений
Alexander Kladov
выглядит говенно
+
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
@alexliberty устроился в итоге куда?
источник

AL

Alex Libertovsky in Генератор непрочитанных сообщений
Alexander Kladov
@alexliberty устроился в итоге куда?
Пока ещё нет. Мыловцы молчат в пизде торчат, в Тануки разраб заболел и сказал сегодня-завтра скажет.
Вчера ещё собес был, но там вроде всё нормально, сделают офер на неделе
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
Alex Libertovsky
Пока ещё нет. Мыловцы молчат в пизде торчат, в Тануки разраб заболел и сказал сегодня-завтра скажет.
Вчера ещё собес был, но там вроде всё нормально, сделают офер на неделе
в инструменты не стал?
источник

NT

Nikolay Turskyi in Генератор непрочитанных сообщений
Санжар
<?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. Отправка статуса - гавно, тебе два раза нужно было инициализировать один и тот же массив ответа.
источник