Нерациональная работа с базой

Правила раздела: faq.php?mode=okay
Модератор: Модераторы

korshunov
korshunov
Репутация: 146
Сообщения: 1854
Зарегистрирован: 03.12.2015
С нами: 8 лет 3 месяца
Skype

Сообщение #1 korshunov » 03.06.2017, 18:59

В api/Features.php есть такой код:

Код: Выделить всё

            $this->db->query('select product_id from __options where feature_id=?', intval($id));
            $p_ids = $this->db->results('product_id');
            if (!empty($p_ids)) {
                $this->db->query('update __products set last_modify=now() where id in(?@)', $p_ids);
            }


Если хоть капельку над ним задуматься, то видно, что участвующие в нем два запроса очень легко заменить на один. Если это сделать, то
1. код станет почти вдвое короче,
2. код станет более читабельным, более простым и понятным,
3. код будет выполняться быстрее.

И это далеко не единственное место, где встречается подобный нерациональный код, подобное присутствует как в указанном файле, так и во многих других...

Название раздела: Баг-репорт OkayCMS (Сообщения об ошибках)
Правила раздела: faq.php?mode=okay

Быстрый ответ


Введите код в точности так, как вы его видите. Регистр символов не имеет значения.
Код подтверждения

   

Вернуться в «Баг-репорт OkayCMS (Сообщения об ошибках)»

Кто сейчас на форуме (по активности за 5 минут)

Сейчас этот раздел просматривают: 16 гостей