QListWidget::addItem and display immediately

[vlg789]

QListWidget::addItem() and the problem is that ... doesn't update the view

[jpn]

Yes, it doesn't update because of the busy loop which blocks the event loop. Paint events get scheduled but you are not giving the application any chance to actually deliver them. Behind the curtains the view gets informed about each and every insertion. This is most likely one of main reasons why it takes so long to insert items one by one. Could you give QListWidget::addItems() a try? If it's still too slow, we'll look for other possibilities.

for (int i = 0; i < 1000; ++i)
    listWidget->addItem(something); // each item added separately, view gets informed thousand times, once per insertion

for (int i = 0; i < 1000; ++i)
    listWidget->addItem(something); // each item added separately, view gets informed thousand times, once per insertion

To copy to clipboard, switch view to plain text mode 

QStringList items;
for (int i = 0; i < 1000; ++i)
    items += something;
listWidget->addItems(items); // everything gets inserted at one go, view gets informed only once in total

QStringList items;
for (int i = 0; i < 1000; ++i)
    items += something;
listWidget->addItems(items); // everything gets inserted at one go, view gets informed only once in total

To copy to clipboard, switch view to plain text mode 

[vlg789]

First of all thanks for the quick answer.

The main reasons why it takes so long to insert items are some operations that take place in background. The listview must keep user informed with the progress, so the problem is not to optimize insertion of items, but to update the listbox immediately.

The only problem i have is that QThread needs a run() and this makes my design more odd... The question is if i can avoid QThread ...

Thanks

[jpn]

One way would be to call QCoreApplication::processEvents() every once in a while to let the application process its events. This will make the overall process a bit longer, though:

// a busy loop
for (...)
{
    // do something
 
    // let the application process its events
    QCoreApplication::processEvents();
}

// a busy loop
for (...)
{
    // do something

    // let the application process its events
    QCoreApplication::processEvents();
}

To copy to clipboard, switch view to plain text mode 

[vlg789]

Thanks man,
this is what i was looking for.