Поиск слов в файле в Qt

[stasJun]

Немного переделал(смотреть скрин), додал кнопку кансел в прогрес виджет, выход при нажатии хрестика. Но когда я жму на старт второй раз оно сразу показывает что нашло 4 файла(смотреть другой скрин, там в рамке 3 файла , они появляются при 0% прогреса), тоесть не очищает полностю то ранше нашло. я ж делаю клеар

 mFoundFilesList->clear();

Код функции нажатия на кнопку старт

void FindFilesWidget::startClicked()
{
    mFoundFilesList->clear();
    const int idealThreadCount = QThread::idealThreadCount();
    for (size_t index = 0; index < mThreads.size(); ++index)
        delete mThreads[index];
    mThreads.clear();
    mThreads.reserve(idealThreadCount);

    mPaths.clear();
    QDirIterator dirIt(mSearchDirectoryLine->text(), QDirIterator::Subdirectories);
    qint64 directorySizeInBytes = 0;
    while (dirIt.hasNext())
    {
        dirIt.next();
        QFileInfo fileInfo(dirIt.filePath());
        if (fileInfo.isFile())
        {
            if (fileInfo.suffix() == "txt")
            {
                mPaths.push_back(dirIt.filePath());
                directorySizeInBytes += fileInfo.size();
            }
        }
    }

    const std::vector<Range> ranges = GenerateRanges(idealThreadCount, mPaths.size());
    for (size_t index = 0; index < idealThreadCount; ++index)
        mThreads.push_back(new FindWordInFilesThread(mPaths, mSearchWordLine->text(), ranges[index].mStart, ranges[index].mLength));

    //if (mProgressWidget == nullptr)
       mProgressWidget = new ProgressWidget(directorySizeInBytes);

    QObject::connect(mProgressWidget, SIGNAL(finished()), this, SLOT(finished()));
    QObject::connect(mProgressWidget, SIGNAL(paused()), this, SLOT(pausedClicked()));
    QObject::connect(mProgressWidget, SIGNAL(continued()), this, SLOT(continuedClicked()));
    QObject::connect(mProgressWidget, SIGNAL(canceled()), this, SLOT(cancelClicked()));

    mProgressWidget->show();

    for (size_t index = 0; index < idealThreadCount; ++index)
    {
        QObject::connect(mThreads[index], SIGNAL(progressChanged(qint64)), mProgressWidget, SLOT(progressChanged(qint64)));
        QObject::connect(mThreads[index], SIGNAL(fileFound(const QString&)), this, SLOT(fileFound(const QString&)));
    }

    for (size_t index = 0; index < idealThreadCount; ++index)
        mThreads[index]->start();

}

QGroupBox* FindFilesWidget::createFoundFilesGroup()
{
    QGroupBox* foundFilesGroup = new QGroupBox("&Found files:");
    mFoundFilesList = new QListWidget(foundFilesGroup);

    QHBoxLayout* hLayout = new QHBoxLayout();
    hLayout->setSpacing(6);
    hLayout->setContentsMargins(11, 11, 11, 11);
    hLayout->addWidget(mFoundFilesList);

    foundFilesGroup->setLayout(hLayout);
    return foundFilesGroup;
}

И спп прогрес виджета:

#include "ProgressWidget.h"
#include <cassert>

ProgressWidget::ProgressWidget(qint64 directorySizeInBytes, QDialog *parent)
    : QDialog(parent)
    , mActionState(RUNNING)
    , mDirectorySizeInBytes(directorySizeInBytes)
    , mBytesProcessed(0)
{
    setWindowTitle("Progress");
    setModal(true);

    mProgressBar = new QProgressBar();
    mProgressBar->setMaximum(100);
    mProgressBar->setMinimumWidth(200);
    mProgressBar->setAlignment(Qt::AlignCenter);
    mProgressBar->setValue(0);

    mPauseOrContinueButton = new QPushButton("&Pause");
    mCancelButton = new QPushButton("&Cancel");


    QHBoxLayout* hLayout = new QHBoxLayout();
    hLayout->addWidget(mPauseOrContinueButton);
    hLayout->addWidget(mCancelButton);
    hLayout->setSpacing(6);
    hLayout->setContentsMargins(11, 11, 11, 11);

    QVBoxLayout* vLayout = new QVBoxLayout();
    vLayout->addWidget(mProgressBar);
    vLayout->addLayout(hLayout);

    connect(mPauseOrContinueButton, SIGNAL(clicked()), SLOT(pauseClicked()));
    connect(mCancelButton, SIGNAL(clicked()), SLOT(cancelClicked()));

    setLayout(vLayout);
    resize(300, 100);
}

ProgressWidget::~ProgressWidget()
{
    delete mPauseOrContinueButton;
    delete mCancelButton;
}

void ProgressWidget::pauseClicked()
{
    if ((mActionState == STOPPED) || (mActionState == PAUSED))
    {
        mActionState = RUNNING;
        mPauseOrContinueButton->setText("&Pause");
        emit continued();
    }
    else if (mActionState == RUNNING)
    {
        mActionState = PAUSED;
        mPauseOrContinueButton->setText("&Continue");
        emit paused();
    }
    else
    {
        assert(false);
    }
}

void ProgressWidget::cancelClicked()
{
    emit canceled();
    close();
}

void ProgressWidget::progressChanged(qint64 bytesProcessed)
{
    mBytesProcessed += bytesProcessed;
    int progressValue = 100 * ((float)mBytesProcessed / (float)mDirectorySizeInBytes);
    mProgressBar->setValue(progressValue);
    if (mBytesProcessed == mDirectorySizeInBytes)
    {
        emit canceled();
        close();
    }
}

void ProgressWidget::closeEvent(QCloseEvent* event)
{
    if (mBytesProcessed != mDirectorySizeInBytes)
    {
        emit paused();
        QMessageBox::StandardButton message = QMessageBox::question(this, "Progress", tr("Are you sure you want to exit?"), QMessageBox::Cancel | QMessageBox::No | QMessageBox::Yes, QMessageBox::Yes);
        if (message == QMessageBox::Yes)
        {
            emit canceled();
            event->accept();
        }
        else
        {
            emit continued();
            event->ignore();
        }
        emit finished();

    }
    else
    {
        emit canceled();
        close();
    }
}

[Alex11223]

Так а что в слоте fileFound?

[stasJun]

Сообщение от Alex11223

Так а что в слоте fileFound?

void FindFilesWidget::fileFound(const QString& fileName)
{
    mFoundFilesList->addItem(fileName);
}

[Alex11223]

Тогда странно, да.
Ну поставьте breakpoint'ы куда-нибудь и посмотрите что-нибудь, программирование это ж не только кодинг, но и отладка
http://doc.qt.io/qtcreator/creator-debug-mode.html

[stasJun]

Сообщение от Alex11223

Тогда странно, да.
Ну поставьте breakpoint'ы куда-нибудь и посмотрите что-нибудь, программирование это ж не только кодинг, но и отладка
http://doc.qt.io/qtcreator/creator-debug-mode.html

и еще вопрос, я делаю в функции старт :

void FindFilesWidget::startClicked()
{
  .......
    mProgressWidget = nullptr;
........
 if (mProgressWidget == nullptr)
       mProgressWidget = new ProgressWidget(directorySizeInBytes);
..........

чтобы не создавать каждый раз новый екземпляр mProgressWidget , но я вижу что я создаю это каждый раз, как избежать этого(тоесть иф всегда срабатывает)?

[Alex11223]

А почему он должен не срабатывать если вы присвоили nullptr и проверяете, что он nullptr?оО

Если нужен только 1 экземпляр без пересоздания во время работы программы, то почему просто не создать его в конструкторе основного окна? (а лучше http://www.programmersforum.ru/showt...14#post1641051)

[stasJun]

Сообщение от Alex11223

А почему он должен не срабатывать если вы присвоили nullptr и проверяете, что он nullptr?оО

Если нужен только 1 экземпляр без пересоздания во время работы программы, то почему просто не создать его в конструкторе основного окна? (а лучше http://www.programmersforum.ru/showt...14#post1641051)

а нет, я имел введу что он всегда будет срабатывать. А мне нужно чтобы только первый раз, а потом в старт просто обнулять прогрес. Но как обнулить прогрес по другому я хз

[Alex11223]

добавить в него public функцию/слот обнуления прогресс и вызывать ее.

[stasJun]

Сообщение от Alex11223

Если нужен только 1 экземпляр без пересоздания во время работы программы, то почему просто не создать его в конструкторе основного окна? (а лучше http://www.programmersforum.ru/showt...14#post1641051)

я об этом думал, но у меня аргумент directorySizeInBytes щитается встарте.

 mProgressWidget = new ProgressWidget(directorySizeInBytes);

[stasJun]

Сообщение от Alex11223

добавить в него public функцию/слот обнуления прогресс и вызывать ее.

ProgressWidget:

signals:
   void restart();

public slots:
    void restartClicked();

void ProgressWidget::restartClicked()
{
    mProgressBar->setValue(0);
}

FindWidget:

public slots:
    void restartClicked();

void FindFilesWidget::startClicked()
{
   .......
    restartClicked();
    .........
    QObject::connect(mProgressWidget, SIGNAL(restart()), this, SLOT(restartClicked()));
.........
}
void FindFilesWidget::restartClicked()
{
    mProgressWidget->restart();
}

Что я не так делаю? слоты создаю, сигнали тоже