Size: a a a

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

2020 March 03

AL

Alex Libertovsky in Генератор непрочитанных сообщений
Faust
Ларавель какой-то
Там в конце почти по уму сделано, жаль что не die() 🌚
источник

С

Санжар in Генератор непрочитанных сообщений
Andrey Bakharev
как минимум sql-инъекции, когда json отправляешь - заголовки корректные указывай
По моему вообще нельзя так отправлять прямо, не?
источник

AB

Andrey Bakharev in Генератор непрочитанных сообщений
я не знаю что DB::query(), но если на остальное смотреть - простой скриптик, а значит можно прямо так
источник

AB

Andrey Bakharev in Генератор непрочитанных сообщений
ну и еще там непонятно зачем второй раз таблица джойнится, если можно сразу данные достать
источник

С

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

AB

Andrey Bakharev in Генератор непрочитанных сообщений
ну т.е. select order_id, order_status from orders where order_id = ? - результат тот же самый будет
источник

С

Санжар in Генератор непрочитанных сообщений
Судя по DB:: это фасад из лары вроде
Дичь короче
источник

С

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

С

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

P

Panda🤔 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));
}
источник

AL

Alex Libertovsky in Генератор непрочитанных сообщений
Санжар
Хз, какое-то странное задание
Кароч будешь говнокод разбирать чей-то
источник

AB

Andrey Bakharev in Генератор непрочитанных сообщений
лару не знаю,
если какой-то отдельный пакет можно оттуда для бд ставить - фиг с ним,
если в ларе идет, то нафига отдельным файлом? все через лару и делать,
в том числе и доступ к посту
+ там уже должно быть для отправки json'а
источник

С

Санжар in Генератор непрочитанных сообщений
Alex Libertovsky
Кароч будешь говнокод разбирать чей-то
походу да
источник

С

Санжар in Генератор непрочитанных сообщений
Andrey Bakharev
лару не знаю,
если какой-то отдельный пакет можно оттуда для бд ставить - фиг с ним,
если в ларе идет, то нафига отдельным файлом? все через лару и делать,
в том числе и доступ к посту
+ там уже должно быть для отправки json'а
если лара, то да, могли бы Eloquent юзать, а не голые запросы так писать. странно короче, надо узнать, вдруг там дикое легаси
источник

AB

Andrey Bakharev in Генератор непрочитанных сообщений
зато прокачаешься ))) потом при переходе на другую работу это как "достижение" опишешь,
и будешь вечно на легаси сидеть и переписывать его по нормальному ))
станешь мастером в этом деле и сможешь по 300К требовать
источник

АМ

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

AK

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

AB

Andrey Bakharev in Генератор непрочитанных сообщений
кстати, да - там если условие сработало, то оно второй раз fail напечатает, т.к. в sendJson нет die, и в условии нет ретурна
источник

AK

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

AK

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