Problem inserting items inside TreeWidget

[hakermania]

Hello guys, I have a problem inserting items to a tree widget!

Some info about the (attached) screenshot now, 'list' is declared as
QList<QTreeWidgetItem*> list;
Also,

qDebug() << parnt->text(i-2);
list.insert(i-2,parnt);

qDebug() << parnt->text(i-2);
list.insert(i-2,parnt);

To copy to clipboard, switch view to plain text mode 

are inside a for loop, while

ui->treeWidget->addTopLevelItems(list)

ui->treeWidget->addTopLevelItems(list)

To copy to clipboard, switch view to plain text mode 

is outside it.
When I output the parnt at 'i-2' as you can see, it does echo

2011 May
2011 October

2011 May
2011 October

To copy to clipboard, switch view to plain text mode 

At the exact next line I insert the parnt to the 'list' at 'i-2' (the exact point I echoed parnt to the previous line)

But as you can see in the window besides it, it inserts '2011 May' 2 times, and I don't know why!

Also, mention that

insert.at(i-2)->addChild/addChilds(parnt);

insert.at(i-2)->addChild/addChilds(parnt);

To copy to clipboard, switch view to plain text mode 

crashes my application, so I cannot use them (I don't know why).

Thx in advance for any answer/help!error.png

[wysota]

It's not possible to determine what's wrong here with such incomplete snippets of code. I can only advise you to print the contents of "list" before you add it to the list widget.

[hakermania]

Ok, sorry for being difficult to understand, the code is this:

for(int i=2; i < a; i++){
             //code here...
            for(int j=2;j<dir1_count;j++){
             //code here...
                //here adding a child to parnt...
                parnt->addChild(child1);
            }
             //after adding the children to the parnt I insert the parnt to the list
            qDebug() << parnt->text(i-2);
            list.insert(i-2,parnt);
        }
        //finally, i insert the list to the treewidget...
        ui->treeWidget->addTopLevelItems(list);

for(int i=2; i < a; i++){
             //code here...
            for(int j=2;j<dir1_count;j++){
             //code here...
                //here adding a child to parnt...
                parnt->addChild(child1);
            }
             //after adding the children to the parnt I insert the parnt to the list
            qDebug() << parnt->text(i-2);
            list.insert(i-2,parnt);
        }
        //finally, i insert the list to the treewidget...
        ui->treeWidget->addTopLevelItems(list);

To copy to clipboard, switch view to plain text mode 

The output of qDebug() as I said to the first post is as I expected to be (different month each time) but in the treewidget there is inserted the same month (the first one) without an obvious reason. This is the same code but it also outputs the list's contents:

qDebug() << "Parnt at " << i-2 << " has '" << parnt->text(i-2) << "'";
            list.insert(i-2,parnt);
            qDebug() << " List at " << i-2 << " has '" << list.at(i-2)->text(0) << "'";

qDebug() << "Parnt at " << i-2 << " has '" << parnt->text(i-2) << "'";
            list.insert(i-2,parnt);
            qDebug() << " List at " << i-2 << " has '" << list.at(i-2)->text(0) << "'";

To copy to clipboard, switch view to plain text mode 

So, by running this code by outputing the contents of the 'list', the output is:

Parnt at  0  has ' "2011 May" '  <- As expected
 List at  0  has ' "2011 May" '  <- As expected
Parnt at  1  has ' "2011 October" '  <- As expected
 List at  1  has ' "2011 May" '  <- No comment :/

Parnt at  0  has ' "2011 May" '  <- As expected
 List at  0  has ' "2011 May" '  <- As expected
Parnt at  1  has ' "2011 October" '  <- As expected
 List at  1  has ' "2011 May" '  <- No comment :/

To copy to clipboard, switch view to plain text mode 

Which i find weird...
Obviously, the error is while inserting the 'parnt' into the 'list' or while using addTopLevelItems()

Thx in advance again...

[wysota]

What is child1? How do you initialize this variable? And why are you iterating i from 2 and not from 0?

[hakermania]

The code is like this:

QTreeWidgetItem *child1 = new QTreeWidgetItem;
                QDir his_file;
                .........
                his_file.setPath(QString(subdir1.path() + "/" + subdir1.entryList().at(j)));
                child1->setText(0,his_file.dirName());
                parnt->addChild(child1);

QTreeWidgetItem *child1 = new QTreeWidgetItem;
                QDir his_file;
                .........
                his_file.setPath(QString(subdir1.path() + "/" + subdir1.entryList().at(j)));
                child1->setText(0,his_file.dirName());
                parnt->addChild(child1);

To copy to clipboard, switch view to plain text mode 

So, it starts from 2 in order to avoid the paths at 0 and 1 (which means the paths: ) . (current) and .. (one previous), because I actually read folders' names and I add them to parnt, then add parnt to list and list to treewidget...

[wysota]

Why don't you ask QDir to skip those special entries then?

[hakermania]

Thx for the suggestion but my solution at that point works perfectly and there's no point in changing it right now.
In the point now, I don't now what the point(problem) is xD

[wysota]

I think there is a point in changing it right now, because due to increased complexity of your code you can't be sure it is not the value of "i" that is causing the problem.