Нерациональная работа с базой
Добавлено: 03.06.2017, 18:59
В api/Features.php есть такой код:
Если хоть капельку над ним задуматься, то видно, что участвующие в нем два запроса очень легко заменить на один. Если это сделать, то
1. код станет почти вдвое короче,
2. код станет более читабельным, более простым и понятным,
3. код будет выполняться быстрее.
И это далеко не единственное место, где встречается подобный нерациональный код, подобное присутствует как в указанном файле, так и во многих других...
Код: Выделить всё
$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. код будет выполняться быстрее.
И это далеко не единственное место, где встречается подобный нерациональный код, подобное присутствует как в указанном файле, так и во многих других...