[Solved]Runtime Bug in Image Slidshow Application

Haresh

hello everyone
I have created a gui application for slide show of images consisting of a push-button to select a directory from file system and a label to display images at interval of 5 seconds.
It compiles and runs fine if I use it for a single directory but if it is running a slide show and I select another directory then it displays images of both directories.
so anyone can please help me out of this???

Huulivoide

I think you have forgotten to empty the list of image files before you build
the list from the new directory. But without any code it hard to say for sure.

Haresh

I have clear the stringlist before reusing it.

void MainWindow::on_pushButton_clicked()
{
int index=0;
QString DirectryPath,ImagePath;
QStringList SetFilter,FileList;
QDir DirName;

FileList.clear();

DirectryPath=QFileDialog::getExistingDirectory(this,tr("Select Folder"),"/home",QFileDialog::ShowDirsOnly|QFileDialog::DontResolveSymlinks);
DirectryPath=DirectryPath +'/';
qDebug() << DirectryPath;

DirName=DirectryPath;
qDebug()<<DirName;

SetFilter<<"*.jpg"<<"*.jpeg"<<"*.png";
DirName.setNameFilters(SetFilter);

FileList=DirName.entryList(QDir::Files,QDir::NoSort);

qDebug() << FileList;
qDebug()<<FileList.size();

while(index<FileList.count())
{
    ImagePath=DirectryPath+FileList[index];
    qDebug()<<ImagePath;
    QImage image(ImagePath);
    qDebug()<<image;
    ui->label_pic->setPixmap(QPixmap::fromImage(image));
    ui->label_pic->setScaledContents(true);
    index++;
    delay();
}

ui->label_pic->setText(tr("No More Images Avalable"));

}

SGaist

Hi,

You might be having your loop running twice. You should modify your architecture and rather use a QTimer for your 5 second interval. Start it once you created your list and stop it before you create it.

Haresh

hello,
It is now working as I wanted so thank you for your help.
But you said my loop is running twice so is there any another way I can stop the previous loop from executing when same signal is generated again.

SGaist

Yes, rather than use a while loop like that, use a QTimer