Re: CFileDialog failure

From:

"Tom Serface" <tom.nospam@camaswood.com>

Newsgroups:

microsoft.public.vc.mfc

Date:

Thu, 18 Oct 2007 07:11:20 -0700

Message-ID:

<80822A9A-E612-49D2-94D2-98C8CAC28A98@microsoft.com>

You could try creating the dialog on the stack rather than creating it with
new. I don't see anywhere here where you are deleting the object before
looping either so perhaps that is part of the problem (previous object still
existing). Why not just do something like:

CFileDialog dlg(TRUE, 0, "C:\\",OFN_ALLOWMULTISELECT, "All Files
(*.*)|*.*||",
CWnd::FromHandle(hStatusWnd));

Or, alternatively, you could delete the previous one before recreating it at
the top of the while loop.

Tom

"Raj" <rajaramank@msn.com> wrote in message
news:%23e8CXGUEIHA.1208@TK2MSFTNGP05.phx.gbl...

    #define MAX_FILE_COUNT 1 //define it to 1. It make it easy to
reproduce this error

    CString strFilesSelected;
     CFileDialog *pFileDialog=NULL;
     while(1) {
           pFileDialog = new CFileDialog(TRUE, 0,
"C:\\",OFN_ALLOWMULTISELECT, "All Files (*.*)|*.*||",
CWnd::FromHandle(hStatusWnd));
           pFileDialog->GetOFN().lpstrFile =
strFilesSelected.GetBuffer((_MAX_PATH + 1) * MAX_FILE_COUNT + 1);
           pFileDialog->GetOFN().nMaxFile = (_MAX_PATH + 1) *
MAX_FILE_COUNT + 1;

           nResult = pFileDialog->DoModal();
           strFilesSelected.ReleaseBuffer();

           if(IDOK != nResult) {
                 delete pFileDialog;
                 DWORD dwCommDlgFailure = CommDlgExtendedError();
                 if(FNERR_BUFFERTOOSMALL & dwCommDlgFailure) {
                       AfxMessageBox("Too many files selected.");
                       continue;
                 }
                 break;
           }
           break;
     }

This is what I expect to happen
1. First time when the File Selection dialog appears, select a large
number of files to fail the selection
2. "Too many files selected" message appear
3. After pressing <OK> button I expect the File selection dialog to
appear

What is happening is after the first "Too many files selected" failure
DoModal() never successfully brings up the file selection dialog and
returns last error code, hence gets into endless loop.