remove widgets from layout in run time

[Lykurg]

but old widgets still in ui and when i add new thumbnail widgets it continue add to lay out
for example first time i have 3 thumbs
second i removed the first 3 and add 5
my result is 8 thumb in the scrollarea

That's wired, does thumb->hide() "removes" the pictures from your layout? debug the values, are they pointing to the right elements?

[Lykurg]

I just remember better use something like

QList<QPushButton *> allPButtons = parentWidget.findChildren<QPushButton *>();

QList<QPushButton *> allPButtons = parentWidget.findChildren<QPushButton *>();

To copy to clipboard, switch view to plain text mode 

then you have not to cast and your pointers are valid

And where did you set the layout m_thumbnailsWidgetHLayout? The layout is for which widget?

[bindigi01]

i set the layout in the constructor

[Lykurg]

i set the layout in the constructor

What class is ui.thumbnailsScrollAreaWidgetContents? And please post the whole function where you remove the old and set the new images. Only so we can find your error because the code is right.

[bindigi01]

What class is ui.thumbnailsScrollAreaWidgetContents? the widget inside the scroll area

void ViewerMainWindow::fillThumbnails() {
    if (FrameTile::activeFrameTile != 0) {
        FrameTile* frameTile = FrameTile::activeFrameTile;
 
        dicom::ImageIODFacade * fcd = frameTile->getImgFcd();
        if (fcd != 0) {
 
            QList<QObject*> objlist = m_thumbnailsWidgetHLayout->children();
            for (int i = 0; i < objlist.count(); i++) {
                ThumbnailWidget* thumb = (ThumbnailWidget*) objlist.at(i);
                m_thumbnailsWidgetHLayout->removeWidget(thumb);
                delete thumb;
            }
 
            QMap<size_t, QPixmap> map = fcd->getThumpnails();
 
            foreach(size_t key, map.keys()) {
                QPixmap pixmap = map.value(key);
                ThumbnailWidget* thumb = new ThumbnailWidget;
                QPixmap scaled = pixmap.scaled(ThumbnailWidget::THUMBNAIL_SIZE,
                        ThumbnailWidget::THUMBNAIL_SIZE,
                        Qt::KeepAspectRatioByExpanding,
                        Qt::FastTransformation);
                thumb->setPixmap(scaled);
                thumb->setThumbnailIndex(key);
                m_thumbnailsWidgetHLayout->addWidget(thumb);
            }
        }
    }
}

void ViewerMainWindow::fillThumbnails() {
    if (FrameTile::activeFrameTile != 0) {
        FrameTile* frameTile = FrameTile::activeFrameTile;

        dicom::ImageIODFacade * fcd = frameTile->getImgFcd();
        if (fcd != 0) {
            
            QList<QObject*> objlist = m_thumbnailsWidgetHLayout->children();
            for (int i = 0; i < objlist.count(); i++) {
                ThumbnailWidget* thumb = (ThumbnailWidget*) objlist.at(i);
                m_thumbnailsWidgetHLayout->removeWidget(thumb);
                delete thumb;
            }
            
            QMap<size_t, QPixmap> map = fcd->getThumpnails();

            foreach(size_t key, map.keys()) {
                QPixmap pixmap = map.value(key);
                ThumbnailWidget* thumb = new ThumbnailWidget;
                QPixmap scaled = pixmap.scaled(ThumbnailWidget::THUMBNAIL_SIZE,
                        ThumbnailWidget::THUMBNAIL_SIZE,
                        Qt::KeepAspectRatioByExpanding,
                        Qt::FastTransformation);
                thumb->setPixmap(scaled);
                thumb->setThumbnailIndex(key);
                m_thumbnailsWidgetHLayout->addWidget(thumb);
            }
        }
    }
}

To copy to clipboard, switch view to plain text mode 

[Lykurg]

Ok, sorry, I don't have a clue. The code seems right. The only thing I can guess is that fcd->getThumpnails() is still returning the old pictures. You want run following simple extension-debug-variant of your function:

void ViewerMainWindow::fillThumbnails() {
    if (FrameTile::activeFrameTile != 0) {
        FrameTile* frameTile = FrameTile::activeFrameTile;
 
        dicom::ImageIODFacade * fcd = frameTile->getImgFcd();
        if (fcd != 0) {
 
            QList<QObject*> objlist = m_thumbnailsWidgetHLayout->children();
            qWarning() << "Object count" << objlist.count();  //DEBUG
            for (int i = 0; i < objlist.count(); i++) {
                ThumbnailWidget* thumb = (ThumbnailWidget*) objlist.at(i);
                m_thumbnailsWidgetHLayout->removeWidget(thumb);
                delete thumb;
            }
            qWarning() << "Object count (after)" << m_thumbnailsWidgetHLayout->children().count();  //DEBUG
 
            QMap<size_t, QPixmap> map = fcd->getThumpnails();
 
            foreach(size_t key, map.keys()) {
                qWarning() << "add" << key;  //DEBUG
                QPixmap pixmap = map.value(key);
                ThumbnailWidget* thumb = new ThumbnailWidget;
                QPixmap scaled = pixmap.scaled(ThumbnailWidget::THUMBNAIL_SIZE,
                        ThumbnailWidget::THUMBNAIL_SIZE,
                        Qt::KeepAspectRatioByExpanding,
                        Qt::FastTransformation);
                thumb->setPixmap(scaled);
                thumb->setThumbnailIndex(key);
                m_thumbnailsWidgetHLayout->addWidget(thumb);
            }
        }
    }
}

void ViewerMainWindow::fillThumbnails() {
    if (FrameTile::activeFrameTile != 0) {
        FrameTile* frameTile = FrameTile::activeFrameTile;
 
        dicom::ImageIODFacade * fcd = frameTile->getImgFcd();
        if (fcd != 0) {
            
            QList<QObject*> objlist = m_thumbnailsWidgetHLayout->children();
            qWarning() << "Object count" << objlist.count();  //DEBUG
            for (int i = 0; i < objlist.count(); i++) {
                ThumbnailWidget* thumb = (ThumbnailWidget*) objlist.at(i);
                m_thumbnailsWidgetHLayout->removeWidget(thumb);
                delete thumb;
            }
            qWarning() << "Object count (after)" << m_thumbnailsWidgetHLayout->children().count();  //DEBUG
            
            QMap<size_t, QPixmap> map = fcd->getThumpnails();
 
            foreach(size_t key, map.keys()) {
                qWarning() << "add" << key;  //DEBUG
                QPixmap pixmap = map.value(key);
                ThumbnailWidget* thumb = new ThumbnailWidget;
                QPixmap scaled = pixmap.scaled(ThumbnailWidget::THUMBNAIL_SIZE,
                        ThumbnailWidget::THUMBNAIL_SIZE,
                        Qt::KeepAspectRatioByExpanding,
                        Qt::FastTransformation);
                thumb->setPixmap(scaled);
                thumb->setThumbnailIndex(key);
                m_thumbnailsWidgetHLayout->addWidget(thumb);
            }
        }
    }
}

To copy to clipboard, switch view to plain text mode 

What's the output?

[bindigi01]

i have triggered called this function many times
here is the outPut

############first 16 thumb
count Before 0
count after 0
add key 1
add key 2
add key 3
add key 4
add key 5
add key 6
add key 7
add key 8
add key 9
add key 10
add key 11
add key 12
add key 13
add key 14
add key 15
add key 16

############first 16 thumb again
count Before 0
count after 0
add key 1
add key 2
add key 3
add key 4
add key 5
add key 6
add key 7
add key 8
add key 9
add key 10
add key 11
add key 12
add key 13
add key 14
add key 15
add key 16

############ another image with 1 thumb
count Before 0
count after 0
add key 1

[bindigi01]

solved thanks Lykurg for helping

here is the code that solvs

int i = 0;
            QLayoutItem *child;
            while ((child = m_thumbnailsWidgetHLayout->takeAt(0)) != 0) {
                delete child->widget();
                delete child;
            }

int i = 0;
            QLayoutItem *child;
            while ((child = m_thumbnailsWidgetHLayout->takeAt(0)) != 0) {
                delete child->widget();
                delete child;
            }

To copy to clipboard, switch view to plain text mode