Дата последнего изменения: 09.07.2024
Всегда минимизируйте запросы. Например, если в цикле идет запрос к элементу ИБ, то уже необходимо задуматься над минимизацией. Да, это займет больше времени, но вам скажут спасибо клиенты.
Нельзя:
foreach($arResult["ORDERS"] as $key => $val)
{
foreach($val["BASKET_ITEMS"] as $vvval)
{
$rsEls = CIBlockElement::GetByID();
}
}
Следует:
$arIDs = array();
foreach($arResult["ORDERS"] as $key => $val)
{
foreach($val["BASKET_ITEMS"] as $vvval)
{
$arIDs[] = $vvval["PRODUCT_ID"];
}
}
if(!empty($arIDs))
{
$rsEls = CIBlockElement::GetList(array(), array("ID" => $arIDs));
...
}
foreach($arResult["ORDERS"] as $key => $val)
{
foreach($val["BASKET_ITEMS"] as $vvval)
{
//наполняем данные, налаживая соответствие ID-ков
}
}
Фактически, вы сводите порой десятки, если не сотни, запросов к одному.
Если для какого-либо изменения в БД предусмотрен специальный метод, то следует использовать именно его, а не более общий метод изменения БД.
Хороший пример - модуль интернет-магазина и работа с заказом: можно изменить флаг оплаты заказа путем CSaleOrder::Update, а можно путем CSaleOrder::PayOrder. Так вот, следует применять именно PayOrder, потому что в нем произойдет вызов соответствующих обработчиков.
Даже если вам надо изменить множество полей (того же заказа) и флаг оплаты, то произведите изменение через PayOrder, а затем уже апдейт остальных полей.