Почему после 6ти итераций падает php с переполнением памяти?

Question

[dez64ru]

Modx 2.7.1
Модернизировал таблицу modResource, добавив extId и parentExtId
после импорта категорий чиню зависимости родителей у ресурсов таким скриптом.

foreach ($this->modx->getIterator('modResource', ['extId:!=' => null]) as $obj) {//Берем все р-сы где не пусто extID
            $parentObj = $this->modx->getObject('modResource', ['extId' => $obj->parentExtId]); //Ищем родителя по extId родителя и parentExtId текущего рес-са

            $parentId = null !== $parentObj ? $parentObj->id : $this->getConfig('root_catalog_id'); //Если такого нет, то ставим дефолного родителя.

            $obj->set('parent', $parentId); //Установка родителя

            if ($obj->save()) { //сохраняем
                $this->log("Succes set parent {$parentId} for {$obj->id}");
            } else {
                $this->log("Error set parent {$parentId} for {$obj->id}");
            }

            unset($obj, $parentObj, $parentId);
        }

Comments

[Одиночка Айс]

потому что в каждой итерации вы делаете новый запрос... сделайте выборку заранее

[dez64ru]

Одиночка Айс, А вы точно программист?)
А если серьезно, то вы предлагаете сохранить всю выборку в память, вместо того, чтоб проходить по каждой итерации? Как-то слабо похоже на оптимизацию потребления озу)

[Одиночка Айс]

dez64ru, если я не программист, тогда почему в офф. доках делают выборку заранее?
https://docs.modx.com/xpdo/2.x/class-reference/xpd...

modX ковырял еще лет 10 назад (ево, рево, вечно путаю их), да и с "итерациями" не знаком, но, тем не менее

[dez64ru]

Одиночка Айс, Не в обиду. Но вы точно не программист.

$var = [1,2];
foreach($var as $i) {}

Абсолютно равнозначно
foreach([1,2] as $i) {}

[Одиночка Айс]

dez64ru, угу, только тут нет постоянного обращения к БД

[dez64ru]

Одиночка Айс, Сходил бы и протестировал, прежде чес вводить людей в заблуждение и самому себе лайки ставить.

[Одиночка Айс]

dez64ru, ты бы прежде чем массивами кидаться из функций в циклы, проверил что тебе возвращает $this->modx->getIterator('modResource', ['extId:!=' => null])

[dez64ru]

Одиночка Айс, Да, он отдает объект, который при каждой итерации выполняет запрос. И? Все еще нет никакой разницы, как ты этот объект передашь в foreach - он все так же на каждой итерации будет делать запрос.

[Одиночка Айс]

dez64ru, тогда зачем нужен целый $parentObj, если используется только id? Объекты полностью не уничтожаются unset'ом (в сети можно найти статьи про это), как вариант, можно после использования приравнивать их к null.

[dez64ru]

Одиночка Айс, Потмоу что мне лень писать голые запросы. Скрипт итак отлично работает. 30мб всего на 100.000к товаров и 3к категорий.

Answer 1

[dez64ru]

Не учел, что $obj->parentExtId может быть null.