Форум

да последний вариант уже первоначальный код не в 17 раз памяти больше использует, в 2
но и время возросло до 5-8 секунд

[QUOTE]Я дал вам ссылку на документацию. Там четко написано - при передаче 4-го параметра меняется только значение свойства с этим кодом.[/QUOTE]
сейчас увидел
[QUOTE]Используйте CIBlockElement::GetPropertyValuesArray для получения свойств набора элементов.[/QUOTE]
как раз 2 вариант
[QUOTE]Если второй версии, то почему бы сразу их не перечислить те свойства, которые необходимо получить?[/QUOTE]
в первом варианте нужны дополнительные данные по свойствам, такие заначения и описания свойства поэтому использовался
[CODE] while (($product = $products->GetNextElement()) !== false) {
           $productFields = $product->GetFields();
           $productProps = $product->GetProperties();[/CODE]

[QUOTE]Зачем в том же цикле сразу делать CFile::GetFileArray и изменение элемента? Можно собрать массив из этих данных, за 1 запрос получить данные и дальше уже изменить их в БД.
Выйдет быстрее.[/QUOTE]
так как мне нужны описания всех картинок, то и собираю их в цикле и чтобы в дальнейшем не плодить циклов прямо тут же произвожу изменния
Если можете подсказать более правильный вариант, я буду только за

и еще момент - я привел 2 решения, но складывается ощущение, что видите только одно

первый вариант был написан давно, я его и изменяю из-за его скорости - 2 вариант(работает быстро, но памяти потребляет очень много, превышает лимит установленный в настройках сервера ).
[QUOTE]если достаточно цикла по $products->Fetch() без получения свойств?[/QUOTE]
свойства нужны и если пользоваться в первом варианте fetch то их не получишь

[QUOTE]Кроме того, вы используете \CIBlockElement::SetPropertyValuesEx, хотя меняете только одно свойство. В этом случае быстрее использовать SetPropertyValues с указанием символьного кода свойства. [/QUOTE]
да изменяется 1 свойство, и насколько я знаю как раз для этого и нужно SetPropertyValuesEx, а SetPropertyValues требует перечисления всех свойств не зависимо нужно их менять или нет

Всем здравствуйте.
Есть 2 варианта одного и того же - заполнение свойства по дескрипшину картинки.

1 вариант - работает очень долго(20-28 секунд), но потребяет памяти 94980808

[CODE] $products = \CIBlockElement::GetList(
           [],
           [
               'IBLOCK_ID' => 26,
               'ACTIVE' => 'Y',
           ],
           false,
           false,
           [
               'ID',
               'IBLOCK_ID',
'DETAIL_PICTURE',
           ]
       );

       while (($product = $products->GetNextElement()) !== false) {
           $productFields = $product->GetFields();
           $productProps = $product->GetProperties();

$arFile = \CFile::GetFileArray($productFields["DETAIL_PICTURE"]);
if ($arFile['DESCRIPTION'] == 'color') {
                   if ($productProps['COLOR_IMG']['VALUE'] != $productFields["DETAIL_PICTURE"]) {
                       \CIBlockElement::SetPropertyValuesEx(
                           $productFields['ID'],
                           $productFields['IBLOCK_ID'],
                           [
                               'COLOR_IMG' => $productFields["DETAIL_PICTURE"],
                           ]
                       );
                   }
           }else{
foreach ($productProps['MORE_PHOTO']['VALUE'] as $key => $value) {
if ($productProps['MORE_PHOTO']['DESCRIPTION'][$key] == 'color') {
if ($productProps['COLOR_IMG']['VALUE'] != $value) {
\CIBlockElement::SetPropertyValuesEx(
$productFields['ID'],
$productFields['IBLOCK_ID'],
[
'COLOR_IMG' => $value,
]
);
}
}
}
}
       }[/CODE]

2 вариант(2-4 секунды), но памяти потребляет 1651298448(что в 15-17 раз больше предыдущего варианта)

[CODE]
$products = \CIBlockElement::GetList(
           [],
           [
               'IBLOCK_ID' => 26,
               'ACTIVE' => 'Y',
           ],
           false,
           false,
           [
               'ID',
               'IBLOCK_ID',
'DETAIL_PICTURE',
           ]
       );
$rprArr=[];
       while ($fields = $products->Fetch()) {
           $items[$fields['ID']] = [
               'ID' => $fields['ID'],
               'IBLOCK_ID' => $fields['IBLOCK_ID'],
               'DETAIL_PICTURE' => \CFile::GetFileArray($fields["DETAIL_PICTURE"])
           ];
           
           $rprArr[$fields['ID']] = $fields['ID'];
       }
\CIBlockElement::GetPropertyValuesArray($rprArr, 26, array(
           'ID' => $rprArr,
           'IBLOCK_ID' => 26,
       ));
foreach($rprArr as $kl=>$vl){
           if ($items[$kl]['DETAIL_PICTURE']['DESCRIPTION'] == 'color') {
                   if ($vl['COLOR_IMG']['VALUE'] != $items[$kl]['DETAIL_PICTURE']["ID"]) {
                       \CIBlockElement::SetPropertyValuesEx(
                           $kl,
                           $items[$kl]['IBLOCK_ID'],
                           [
                               'COLOR_IMG' => $items[$kl]['DETAIL_PICTURE']["ID"],
                           ]
                       );
                   }
           }else{
foreach ($vl['MORE_PHOTO']['VALUE'] as $key => $value) {
if ($vl['MORE_PHOTO']['DESCRIPTION'][$key] == 'color') {
if ($vl['COLOR_IMG']['VALUE'] != $value) {
\CIBlockElement::SetPropertyValuesEx(
$kl,
$items[$kl]['IBLOCK_ID'],
[
'COLOR_IMG' => $value,
]
);
}
}
}
}
       }
[/CODE]

Получается хочешь уложиться во время выполнения скрипта 2 вариант лучше, но упираешься в нехватку памяти
пытаешься уложиться в память упираешься в предел времени выполнения

как быть? и тот вариант плох и другой

После переезда на другой сервер перестал работать обработчик каталога(был сентос+виртмашина+php7.1, стал чистый дебиан+php7.1).
Я начал разбираться что и как. дошел до момента пересчета количества и цен.
создаю файл с содержимым

[QUOTE]require($_SERVER["DOCUMENT_ROOT"]."/bitrix/modules/main/include/prolog_before.php");
use \Bitrix\Main\SystemException;
array_map('CModule::IncludeModule', ['iblock', 'catalog', 'sale']);

$obProduct = new CCatalogProduct();
$obProduct->Update(369109, ['QUANTITY' => 10]);[/QUOTE]

больше в нем ничего нет. при попытке вызвать этот файл получаю ошибку указанную выше.

по похожей ошибке нашел тему [URL=https://dev.1c-bitrix.ru/support/forum/forum32/topic97193/]https://dev.1c-bitrix.ru/support/forum/forum32/topic97193/[/URL]

но пока никак не пойму как с этим разобраться установил ulimit -n 10240.
Если в консоле из под рута с таким лимитом запустить, скрипт отрабатывает, без лимита нет

Максим, вы только ulimit изменяли?

я пытаюсь обновить количество товара
[QUOTE]$obProduct = new CCatalogProduct();
$obProduct->Update(369109, ['QUANTITY' => 90]);[/QUOTE]
а в ответ ошибка как у вас

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

[QUOTE]
require($_SERVER["DOCUMENT_ROOT"]."/bitrix/modules/main/include/prolog_before.php");
use \Bitrix\Main\SystemException;
array_map('CModule::IncludeModule', ['iblock', 'catalog', 'sale']);
/* ... */
$obProduct = new CCatalogProduct();
$obProduct->Update(369109, ['QUANTITY' => $items[$id]['QUANTITY']]);[/QUOTE]
но в ответ получаю
[QUOTE]Fatal error: Uncaught Error: Class 'Bitrix\Main\Diag\ExceptionHandlerLog' not found in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhandler.php:266 Stack trace: #0 [internal function]: Bitrix\Main\Diag\ExceptionHandler->handleException(Object(Error)) #1 {main} thrown in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhandler.php on line 266

Fatal error: Uncaught Error: Class 'Bitrix\Main\Diag\ExceptionHandlerLog' not found in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhand­ler.php:350 Stack trace: #0 [internal function]: Bitrix\Main\Diag\ExceptionHandler->handleFatalError() #1 {main} thrown in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhandler.php on line 350[/QUOTE]

Как-то я пропустил наличие GetPropertyValuesArray, и в документации нет. Внесите в документацию пожалуйста, реально Fetch+GetPropertyValuesArray время выполнения сводит с 30 до 3-4 секунд

Спасибо.

Спасибо.
Но часть функционала на сайте завязана на вкладку "Торговый каталог для товаров".

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

в обработчике используется CIBlockElement::GetList для сбора общего массива.
 while ($row = $res->GetNextElement()) {
           $fields = $row->GetFields();
$items[$fields['ID']] = [...];
$props = $row->GetProperties();
           $items[$fields['ID']]['PROPERTIES'] = $props;

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

вопрос есть ли возможность ускорить сбор массива?
использования параметра селект не выход, так как нужны все свойства их коды и все ид вплоть до VALUE_ENUM_ID
если использовать
while ($fields = $res->Fetch()) { для полей, в этом случае можно и селект задействовать для них
и затем
$db_props = CIBlockElement::GetProperties(26, $fields['ID'], array("sort" => "asc"), Array());
while ($ob = $db_props->Fetch())

то половина идентификаторов в частности VALUE_ENUM_ID будет отсутствовать и получается их нужно формировать ручками - но время сокращается в 2-2,5 раза

есть ли третий вариант? может я что-то упускаю из вида

Подскажите пожалуйста, когда на сайт приходит товар, имеющий SKU, из 1С, то цена и количество находятся в SKU.
Но в карточке товара тоже закладка торговый каталог и в ней есть цена и количество. Они должны автоматом заполняться или нужен отдельный обработчик?

К чему вопрос: в карточке эти поля заполнены, но содержат некорректную информацию(sku =100р, а в карточке 200р - старая цена, и количество суммарное отличается). Не соображу что-то базовое сломалось или обработчик нужно искать который прописывает значения.