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

[stasJun]

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

А разве paths уже заполнен в этот момент?

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

 mFoundFilesList->clear();

на начале

void FindFilesWidget::startClicked()
{
    if (mActionState == ActionState::STOPPED)
    {
        mFoundFilesList->clear();

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;
}

[Alex11223]

А выводите-то как?

[stasJun]

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

А выводите-то как?

если я Вас правильно понял:

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

[Alex11223]

А как потоки создаются? Может быть всегда используются одни и те же экземпляры, и если подписка на сигналы происходит в Start, то они будут подписаны 2 (3, 4, ...) раза при повторном запуске.

[stasJun]

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

А как потоки создаются? Может быть всегда используются одни и те же экземпляры, и если подписка на сигналы происходит в Start, то они будут подписаны 2 (3, 4, ...) раза при повторном запуске.

void FindFilesWidget::startClicked()
{
    if (mActionState == ActionState::STOPPED)
    {
        mFoundFilesList->clear();
        mProgressBar->setValue(0);
        mBytesProcessed = 0;
        mActionState = ActionState::RUNNING;
        mStartButton->setText("&Cancel");
        const int idealThreadCount = QThread::idealThreadCount();
        mThreads.reserve(idealThreadCount);
        mPaths.clear();
        QDirIterator dirIt(mSearchDirectoryLine->text(), QDirIterator::Subdirectories);
        while (dirIt.hasNext())
        {
            dirIt.next();
            if (QFileInfo(dirIt.filePath()).isFile())
            {
                if (QFileInfo(dirIt.filePath()).suffix() == "txt")
                    mPaths.push_back(dirIt.filePath());
            }
        }
        for (int index = 0; index < mPaths.size(); ++index)
        {
            QFileInfo fileInfo(mPaths[index]);
            mDirectorySizeInBytes += fileInfo.size();
        }
        const std::vector<Range> ranges = GenerateRanges(idealThreadCount, mPaths.size());
        for (size_t index = 0; index < idealThreadCount; ++index)
            mThreads.push_back(new FindWordInFileThread(mPaths, mSearchWordLine->text(), ranges[index].mStart, ranges[index].mLength));

        for (size_t index = 0; index < mThreads.size(); ++index)
        {
            QObject::connect(mThreads[index], SIGNAL(progressChanged(qint64)), this, 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();
    }
    else
    {
        mActionState = ActionState::STOPPED;
        mStartButton->setText("&Start");
        for (size_t index = 0; index < mThreads.size(); ++index)
            mThreads[index]->cancel();
    }
}

[Alex11223]

... или просто создаются новые потоки каждый раз, но старые не уничтожаются + остаются в списке.

Как кстати определяется завершение поиска? (для смены кнопка обратно на Start и т.п.)

[stasJun]

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

... или просто создаются новые потоки каждый раз, но старые не уничтожаются + остаются в списке.

Как кстати определяется завершение поиска? (для смены кнопка обратно на Start и т.п.)

for (size_t index = 0; index < idealThreadCount; ++index)
           mThreads[index]->start();
    }
    else
    {
        mActionState = ActionState::STOPPED;
        mStartButton->setText("&Start");
        for (size_t index = 0; index < mThreads.size(); ++index)
            mThreads[index]->cancel();
    }

... или просто создаются новые потоки каждый раз, но старые не уничтожаются + остаются в списке.

- нет я с разными папаками пробую, то что было удаляется но другое выводит по 2 раза просто

[Alex11223]

Причем тут этот код? Это ж при отмене, а как при просто завершении?

[stasJun]

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

Причем тут этот код? Это ж при отмене, а как при просто завершении?

внимательно смотрите))

for (size_t index = 0; index < idealThreadCount; ++index)
           mThreads[index]->start(); - это при  завершении меняет с кансел на старт
    }
    else
    {
        mActionState = ActionState::STOPPED;
        mStartButton->setText("&Start");
        for (size_t index = 0; index < mThreads.size(); ++index)
            mThreads[index]->cancel();
    }

[stasJun]

скину чтобы было понятно весь код функции

void FindFilesWidget::startClicked()
{
    if (mActionState == ActionState::STOPPED)
    {
        mFoundFilesList->clear();
        mProgressBar->setValue(0);
        mBytesProcessed = 0;
        mActionState = ActionState::RUNNING;
        mStartButton->setText("&Cancel");
        const int idealThreadCount = QThread::idealThreadCount();
        mThreads.reserve(idealThreadCount);
        mPaths.clear();
        QDirIterator dirIt(mSearchDirectoryLine->text(), QDirIterator::Subdirectories);
        while (dirIt.hasNext())
        {
            dirIt.next();
            if (QFileInfo(dirIt.filePath()).isFile())
            {
                if (QFileInfo(dirIt.filePath()).suffix() == "txt")
                    mPaths.push_back(dirIt.filePath());
            }
        }
        for (int index = 0; index < mPaths.size(); ++index)
        {
            QFileInfo fileInfo(mPaths[index]);
            mDirectorySizeInBytes += fileInfo.size();
        }
        const std::vector<Range> ranges = GenerateRanges(idealThreadCount, mPaths.size());
        for (size_t index = 0; index < idealThreadCount; ++index)
            mThreads.push_back(new FindWordInFileThread(mPaths, mSearchWordLine->text(), ranges[index].mStart, ranges[index].mLength));

        for (size_t index = 0; index < mThreads.size(); ++index)
        {
            QObject::connect(mThreads[index], SIGNAL(progressChanged(qint64)), this, 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();
    }
    else
    {
        mActionState = ActionState::STOPPED;
        mStartButton->setText("&Start");
        for (size_t index = 0; index < mThreads.size(); ++index)
            mThreads[index]->cancel();
    }
}