Запрет многократного вызова диалога выбора папки

[dark_rain]

Доброго времени всем! Прошу помощи, есть кнопочка Fill к которой прикручен стандартный диалог выбора папок, проблема в том, что при многократном нажатии на нее открывается множество таких диалогов.  Как мне запретить вызов более одного экземпляра диалога?

Код: (C++)

LRESULT CMainDlg::OnFill(WORD /*wNotifyCode*/, WORD wID, HWND /*hWndCtl*/, BOOL& /*bHandled*/)
{
        TCHAR szPath[MAX_PATH];
        LPMALLOC pMalloc;
        if (::SHGetMalloc(&pMalloc) == NOERROR)
        {
                BROWSEINFO bi;
                ::ZeroMemory(&bi,sizeof bi);
                bi.ulFlags = BIF_RETURNONLYFSDIRS;
                LPITEMIDLIST pidl = ::SHBrowseForFolder(&bi);
                if (pidl != NULL)
                {
                        if (::SHGetPathFromIDList(pidl, szPath))
                        {
                                if (szPath)// если нажато ОК то выполнить следующие действия
                                {
                               
                                        if (fScanner_->my_thread_)
                                        {
                                                fScanner_->StopThread ();
                                        }
                                        fScanner_->StartThread (szPath);
                                }
                        }
                }
        }
        return 0;
}
 

[Джон]

Да как угодно. Хоть статическим флажком, хоть мембером.

[dark_rain]

Проблема в том, что по нажатию кнопочки Fill у меня запускается поток, который, при выборе новой директории из диалога, я должен прервать и запустить заново. Я совсем запутался, если я устанавливаю флажок то мне эта кнопка уже недоступна, а мне она очень нужна.

Добавлено через 17 минут и 53 секунды:
Все, разобрался.. Совсем запарился уже))

Код: (C++)

LRESULT CMainDlg::OnFill(WORD /*wNotifyCode*/, WORD wID, HWND /*hWndCtl*/, BOOL& /*bHandled*/)
{
        static bool once = false;
        if(!once)
        {
                once = true;
        TCHAR szPath[MAX_PATH];
        LPMALLOC pMalloc;
        if (::SHGetMalloc(&pMalloc) == NOERROR)
        {
                BROWSEINFO bi;
                ::ZeroMemory(&bi,sizeof bi);
                bi.ulFlags = BIF_RETURNONLYFSDIRS;
                LPITEMIDLIST pidl = ::SHBrowseForFolder(&bi);
                if (pidl != NULL)
                {
                        if (::SHGetPathFromIDList(pidl, szPath))
                        {
                                if (szPath)
                                {
                               
                                        if (fScanner_->my_thread_)
                                        {
                                                fScanner_->StopThread ();
                                        }
                                        fScanner_->StartThread (szPath);
                                }
                        }
                }
        }
                once = false;
        }
        return 0;
}

[Алик]

к примеру, так:

глобальная переменная

Код: (C++)

bool bIsDialogOpen = false;

Код: (C++)

if (!bIsDialogOpen) {
        bIsDialogOpen = true;
        bool bDialogResult = ::SHGetPathFromIDList(pidl, szPath);
        bIsDialogOpen = false;
        if (bIsDialogOpen){
                ...
        }
}

но грамотнее, если речь идет о многопоточности, вместо bIsDialogOpen использовать объект синхронизации

[Джон]

к примеру, так:

глобальная переменная

Код: (C++)

bool bIsDialogOpen = false;

Код: (C++)

if (!bIsDialogOpen) {
        bIsDialogOpen = true;
        bool bDialogResult = ::SHGetPathFromIDList(pidl, szPath);
        bIsDialogOpen = false;
        if (bIsDialogOpen){
                ...
        }
}

но грамотнее, если речь идет о многопоточности, вместо bIsDialogOpen использовать объект синхронизации

Алик, зачем нужно во второй строчке:

Код: (C++)

        bIsDialogOpen = true;

?

И тело:

Код: (C++)

if (bIsDialogOpen)

никогда не будет выполняться, тк непосредственно перед этим ты устанавливаешь:

Код: (C++)

bIsDialogOpen = false;

Так что ты хотел этим кодом сказать? Попробуй словами, без кода.

[dark_rain]

В результате переделал диалог и сделал его модальным относительно главного окна. Я новичок и о понятии модальности не знал.

[Джон]

Хммм... Для задачи из первого сообщения даже не знаю, есть ли модальный диалог гут? В общем случае получается, что ты в любой момент можешь открывать папку. Обычно такое действие должно быть законченым. А модальные диалоги предназначены в основнм для постоянного присутствия на экране. Для, например, отображения какой-либо инфы, или для редактирвания сиюминутных данных, например, настроек (громкость, яркость, цвет фона и тд).

[dark_rain]

В моем случае это оптимальный вариант. Красиво и просто. Фактически я запрещаю пользователю обращаться к главному окну, пока открыт диалог выбора папки, тем самым запрещаю снова жать на кнопочку "Fill", как только диалог отработал - главное окно снова доступно. Во многих приложения механизм открытия файла/папки реализован именно так, например в том же блокноте.

[Джон]

Ааа блин. Долблюсь. Забудь всё что я сказал. Просто обычно все диалоги по умолчанию модальные, и упоминание "модальный" я воспринял с точностью наоборот. Те как НЕ модальный. Собственно его и имел ввиду. Ты всё сделал абсолютно правильно.