Странное поведение QTreeWidgetItem::removeChild Опции

[chereppiter]

Есть функция построения дерева (QTreeWidget), в которой для каждого элемента верхнего уровня рекурсивно вызывается функция добавления отпрысков:

void addChilds(QTreeWidgetItem* item, int id)
{

QTreeWidgetItem* child;
for (unsigned int i = 0; i < nodes.size(); ++i)
{

if (nodes[i].parentid == id)
{

child = new QTreeWidgetItem(item);
child->setText(0, QString::fromUtf8(nodes[i].name.c_str()));
addChilds(child, nodes[i].id);
if (!child->childCount())
{

item->removeChild(child);
delete child;

}

}

}

}

По возвращению из рекурсивной addChilds, если у текущего чаилда детей нет, то он удаляется. Ситуация такая: на одной из итераций цикла к item был добавлен child (и не был удалён, т.к. у него тоже есть дети). На одной из последующих итераций создаётся child, для которого потом childCount оказывается 0, поэтому он удаляется (item->removeChild(child)). При этом почему-то вместе с этим отпрыском удаляется и тот, который был добавлен на одной из предыдущих итераций. Проблема решается исключением из вышеприведённого кода строчки:

item->removeChild(child);

Т.е. если удаление выглядит так:

if (!child->childCount())
{

delete child;

}

, то всё работает нормально.
Кто-нибудь знает причину этого загадочного явления?

[Алексей1153]

в общем, вскрытие показало, что чукча умер от вскрытия действительно есть косяк у метода removeChild : он удаляет не тот элемент, причём в контейнере виджета тот, который хотели удалить, занимает место на самом деле удалённого

При этом вызов деструктора чайлда (с помощью delete) приводит к правильному удалению правильного элемента контейнера дерева (чайлд знает родителя, поэтому тут всё корректно у них завязано)

дигноз:не пользоваться removeChild



на скрине три этапа: до вызова removeChild , затем до вызова delete, и после вызова.
Видно, что при вызове removeChild - чайлд (0xac77f30) затёр собой нулевой элемент
После вызова delete он пропал и сам

При моём подходе, кстати, вызов removeChild не нужен

Сообщение отредактировал Алексей1153 - 21.12.2012, 0:31