Коллеги, что вы думаете о данном коде?

[mazzy]

Цитата:

Сообщение от gl00mie

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

Цитата:

Сообщение от mazzy

Особенно плохо с универсальным деревом parent/child с произвольной глубиной.

Да, и полностью согласен с gl00mie. Похоже на закрытие склада.
И надо помнить, что стандартное закрытие склада ограничено глубиной 50 уровней на чисто программистском уровне. Тупо чтобы не было бесконечных циклов. Но они все равно возникают время от времени

может быть и здесь нужно было ограничить?

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