Как правильно формировать запросы к базе на php?

Question

[ReactiveSnaile]

Знающие люди, гляньте пожалуйста, как такое правильно делать, подскажите:
На страничке выводятся записи, так же есть три инпута select с вариантами сортировки записей, сначала новые/старые, стадия выполнения(7 вариантов), пользователи, сортировать нужно учитывая значения всех инпутов, получилось вот такое:

spoiler

/**
     * Возвращает сортированный список задач с указанными идентификаторами
     * @param array $optionsArray <p>Массив с параметрами сортировки</p>
     * @return array <p>Массив со списком задач</p>
     */
    public static function getSortTasks($optionsArray)
    {
        // Соединение с БД
        $db = Db::getConnection();

        // Получаем опции для запроса
        $direction = $optionsArray['direction'];
        $work_status = $optionsArray['work_status'];
        $contractor_id = $optionsArray['contractor_id'];

        $work_status_sql = 'WHERE status = :status';
        $contractor_id_sql = ' AND contractor_id = :contractor_id';

        if($direction == 'new') $direction = 'DESC';
        if($direction == 'old') $direction = 'ASC';
        if($work_status == 'allstatus') $work_status_sql = '';
        if($contractor_id == 'allcontractor') $contractor_id_sql = '';
        if($work_status == 'allstatus' && $contractor_id != 'allcontractor') $contractor_id_sql = 'WHERE contractor_id = :contractor_id';

        // Получение и возврат результатов
        //$sql = "SELECT id, title, contractor_id, description, price, status, public FROM task WHERE status = :status AND contractor_id = :contractor_id ORDER BY id $direction";
        $sql = 'SELECT id, title, contractor_id, description, price, status, public FROM task '.$work_status_sql. $contractor_id_sql.' ORDER BY id '.$direction;
    
        // Используется подготовленный запрос
        $result = $db->prepare($sql);
        
        if($work_status != 'allstatus'){
            $result->bindParam(':status', $work_status, PDO::PARAM_STR);
        }
        if($contractor_id != 'allcontractor'){
            $result->bindParam(':contractor_id', $contractor_id, PDO::PARAM_INT);
        }
        

        // Выполнение коменды
        $result->execute();

        $tasksList = array();

но что то кажется мне что есть варианты по адекватнее, что скажете?

Comments

[ThunderCat]

местами можно подсократить, но в целом рабочее решение. Если не гнаться за кодом ради кода - можно приступать к следующей задаче.

[ReactiveSnaile]

ThunderCat, Спасиб! У меня тоже такая ава была!:)))

[Sergey Romanov]

А Db:: что из себя представляет? Это ПДО? Или другая ОРМ?

[ReactiveSnaile]

Sergey Romanov, PDO

[FanatPHP]

Sergey Romanov, ПДО это не ОРМ

[FanatPHP]

Я прям пришел написать ровно то же самое что и ThunderCat: в целом рабочее решение. Но код я бы подсократил. От всех этих повторяющихся по 5 раз одих и тех же условий в глазах рябит

[ReactiveSnaile]

FanatPHP, про рябит согласен, но сейчас для меня так нагляднее, переписываю, сейчас по сути макет, такую штуку первый раз делаю

[Sergey Romanov]

Так может сойти. Но это не самое элегантное решение. Я бы поставил например Элоквент и использовал его. Там можно было бы чейном добавлять условия, сортировку, ....

И выглядит это будет красиво, и читаться и будет гибче.

[ReactiveSnaile]

Sergey Romanov, Спасибо за наводку, но это скорее учебный проект, вероятность того что он будет использоваться 50/50, скорее всего будет переписан на Ларавел, и тогда.... :)

[Андрей]

DBSimple весьма удобен чтобы ходить в базу из пыхи.