О лишних запросах
Совершенство достигается не тогда, когда уже нечего прибавить, но когда уже ничего нельзя отнять
Антуан де Сент-Экзюпери
Антуан де Сент-Экзюпери
Оптимизация процесса интеграции 1С Предприятия и 1С Битрикс – это неисчерпаемая тема. Чего только не увидишь порой в обработчиках событий добавления и обновления элемента инфоблока.
Больше всего меня убивает, когда я вижу, что разработчики тянут из базы то, что у них и так уже есть. Пример из жизни (нет, серьезно – это реальный пример):
А то, что нужно – уже передано в обработчик в $arFields['PROPERTY_VALUES'][92]
Зачем так неэкономно разбрасываться запросами к базе? Одна строка – и 20 000 лишних обращений к базе при импорте 20 000 элементов.
Больше всего меня убивает, когда я вижу, что разработчики тянут из базы то, что у них и так уже есть. Пример из жизни (нет, серьезно – это реальный пример):
AddEventHandler("iblock", "OnAfterIBlockElementAdd", "BXMUpdateElement_FIELDS");
function BXMUpdateElement_FIELDS(&$arFields)
{
$ibe = new CIBlockElement();
$dbr = $ibe->GetList(array(), array('ID'=>$arFields['ID']) );
while($oe = $dbr->GetNextElement())
{
$arP = $oe->GetProperty('CML2_TRAITS');
}
…
}
А то, что нужно – уже передано в обработчик в $arFields['PROPERTY_VALUES'][92]
Зачем так неэкономно разбрасываться запросами к базе? Одна строка – и 20 000 лишних обращений к базе при импорте 20 000 элементов.
Комментарии
Оно(значение) уже в PROPERTY с ID =92? А завтра случится война, пожар, кто-нибудь грохнет весь инфоблок,или просто это свойство - и создаст заново, какой тогда будет ID? Обработчик "сломается"... Впрочем, как и в случае, если изменится символьный код свойства.
Должен быть какой-то более простой путь решения.