Still celebrating National IT Professionals Day with 3 months of free Premium Membership. Use Code ITDAY17

x
?
Solved

How can I have a modal dialog box responsive while application is busy

Posted on 2011-03-01
7
Medium Priority
?
430 Views
Last Modified: 2012-05-11
Hi,
I'm working on a MFC application that is iterates performs some intensive work. In order to provide some feedback on the current progress of the processing I have a small CDialog
 derived class that I comunicate to it with messages.  In order to update the contentI send it a message like this. Inside the class I have the handling for this message.
m_pProgessDlg->SendMessage(WM_USER +5000, nCurrentCount, nTotall);

Open in new window

The handling class contains a pointer to the object of this class that is allocated like this:
m_pProgressDlg = new CProgressDlg;
  m_pProgressDlg->Create( CProgressDlg::IDD);

Open in new window


The problem I have is that when the executable is very busy the window of this CProgress instance seems not to be very instance. In fact clicking it shows the message "Window does not react".
 How can I make it stay responsive to UI events?

Cheers,
r

0
Comment
Question by:rjorge
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
7 Comments
 
LVL 86

Expert Comment

by:jkr
ID: 35011513
In which way is your app "very busy"? If you are performing some processing in a loop, you can keep the UI responsive by draining the message queue every once in a while, e.g.
void DoEvents ()
{

  while   (   PeekMessage (   &msg,   NULL,   0,  0,  PM_REMOVE))
  {
       DispatchMessage     (   &msg);
  }
}

for (;;)
{

   // a 'busy', time consuming loop
   // ...
   // ...
   // ...

   DoEvents(); // keep the UI happy
  
}

Open in new window

0
 
LVL 16

Expert Comment

by:HooKooDooKu
ID: 35011577
Two thoughts come to mind:

1. Run the "intensive work" and the UI Dialog is seperate threads (requires a little effort to learn how to do multi-threaded applications if you don't already know).

2. Include some logic within the "intensive work" that occationally takes a "time-out" and checks the message que for any messages that need to be processed.  The following will process a single message (you might need to set up a loop to process multiple messages)

MSG msg;
if( ::PeekMessage(&msg, NULL, NULL, NULL, PM_REMOVE) )
{
    ::TranslateMessage(&msg);
    ::DispatchMessage(&msg);
}

0
 
LVL 35

Accepted Solution

by:
sarabande earned 2000 total points
ID: 35013395
i recommend against opening a second message queue handling cause you badly can prevent from getting trapped if the user for example closes the dialog while you currently were doing some work. instead move the lengthy work to a thread like hookoodooku has suggested:


void mydlg::OnButtonClickedLengthyWorkBegin()
{
    m_threadRunning = true;
     _beginthread(doLengthyWork, 0, this);
}

 doLengthyWork is declared a static member function of your dialog class:

// mydlg.h
...

class mydlg : public CDialog
{  
public:
   bool m_threadRunning; // init false in constructor
   bool m_threadStop; // init false in constructor

   static void doLengthyWork(void * p);

// put here your current members

};

// mydlg.cpp

...
void mydlg::doLengthyWork(void * p)
{
    mydlg * pthis = (mydlg*)p;   // now we got a pointer to the dialog

    // here put the code of work
    // you can communicate with main thread by pthis->PostMessage(....);

        // check for termination as often as  possible
        if (pthis->threadStop)
        {
            pthis->m_threadRunning = false;
            return;    // close thread
        }
   ....
   
   pthis->m_threadRunning = false;
}

note when the dialog is closed by the user you have to signal the thread that it should terminate

void mydlg::terminateThread()
{
      m_threadStop = true;
     while (m_threadRunning == true)
      {
           Sleep(10);
      }
}

void mydlg::OnOk()
{
     terminateThread();
     EndDialog(IDOK);
}

void mydlg::OnCancel()
{
     terminateThread();
     EndDialog(IDCANCEL);
}

Sara

0
VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

 

Author Comment

by:rjorge
ID: 35022990
Thanks Sara,
I-ve tried teh code but I have a problem. Inside the lengthy function I access a database by opning a CRecorset object. When I perform this from within the thread th e open never returns. Do you have a suggestion?
0
 
LVL 35

Expert Comment

by:sarabande
ID: 35025011
you can use a second CDataBase * pointer in your thread where you made a new connection to the db. the pointer would be passed into the constructor of your recordset class derived from CRecordset and makes the access from thread safe. you never should call functions of your dialog/view/recordset/application/document classes directly from thread. instead call PostMessage and sent a private message from worker thread to main thread which you can catch via message map.

Sara
0
 
LVL 11

Expert Comment

by:Deepu Abraham
ID: 35091427
Congratulations Sara for getting the Master Certification within a month !!
0
 
LVL 35

Expert Comment

by:sarabande
ID: 35092375
thank you guru deepu :)

Sara
0

Featured Post

Concerto Cloud for Software Providers & ISVs

Can Concerto Cloud Services help you focus on evolving your application offerings, while delivering the best cloud experience to your customers? From DevOps to revenue models and customer support, the answer is yes!

Learn how Concerto can help you.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

As more and more people are shifting to the latest .Net frameworks, the windows presentation framework is gaining importance by the day. Many people are now turning to WPF controls to provide a rich user experience. I have been using WPF controls fo…
Go is an acronym of golang, is a programming language developed Google in 2007. Go is a new language that is mostly in the C family, with significant input from Pascal/Modula/Oberon family. Hence Go arisen as low-level language with fast compilation…
The viewer will learn how to user default arguments when defining functions. This method of defining functions will be contrasted with the non-default-argument of defining functions.
The viewer will be introduced to the technique of using vectors in C++. The video will cover how to define a vector, store values in the vector and retrieve data from the values stored in the vector.

704 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question