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

[oip]

Цитата:

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

Олег, там используется перебор коллекции

while (mi.....more())

Это немного другое дело, здесь очевидно, надо было использовать запросную часть.

Ок. BOMDesignerCtrl\canMoveNode

    while (true)
    {
        if (bomTableMove.RecId == bomTableDrop.RecId)
        {
            dropParent          = tree.getParent(curParent);
            bomTableDropParent  = node2BOMTable.lookup(dropParent);
            return bomTableMoveParent.RecId != bomTableDropParent.RecId || dropParent == rootId;
        }
        else
        {
            curParent = tree.getParent(curParent);
            if (curParent == rootId)
                return true;
            bomTableDrop = node2BOMTable.lookup(curParent);
        }
    }

[MikeR]

Цитата:

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

Ок. BOMDesignerCtrl\canMoveNode

Отыскал таки
По существу - ты бы сам рекомендовал при однозначной связи потомком и родителя делать через while (true), а не через рекурсию?

[db]

Спор ни о чем.
Например с помощью топора можно построить дом или проломить кому нибудь голову. Из этого можно сделать однозначное заключение о применимости топора в любых условиях?

Цитата:

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

Отыскал таки
По существу - ты бы сам рекомендовал при однозначной связи потомком и родителя делать через while (true), а не через рекурсию?

Рекурсия может легко переполнить стек. Но это утверждение опять же ничего не значит в отрыве от конкретной задачи

[MikeR]

Цитата:

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

Спор ни о чем.

Какие споры?

Цитата:

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

Рекурсия может легко переполнить стек.

А это здесь зачем?
А что

while(true)

не может?!

[egorych]

Цитата:

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

А что

while(true)

не может?!

Нет, не может!
Может получиться бесконечный цикл, но переполнения не будет!

[MikeR]

Цитата:

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

Нет, не может!
Может получиться бесконечный цикл, но переполнения не будет!

Смотря, что положить внутрь цикла.