Improve company productivity with a Business Account.Sign Up

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 177
  • Last Modified:

event-handling routines are not called

Hi!

I'm writing a program, that shows pictures for a user-defined time.  That works with

actual_time:=timeGetTime;
repeat until time_getTime>actual_time+t*1000;

So far everything is okay, but if the user enters -1 the picture should be shown until the user presses a key or clicks the mouse. I tried a similar approach with

Semaphore:=False
repeat until Semaphore;

where Semaphore is a boolean that is set to True in one of two procedures that are connected to the onKeyPress and OnClick events of the component that shows the image. (The routines are connected to the corresponding events of the window as well as to the events of the Image-component.)

My problem is, that these two procedures are not called, and even if I use a special button-window I have to show it as ShowModal. It seems that the empty repeat-until-loop binds all the attention of the system and it has no time to react on the event.

How can I force the events to be handled, is there some other way to wait for an user-caused event like in good old Pascal the "repeat until keypressed"?

Oh, by the way, I'm using Borland Delphi 2.0

Thanks in Advance
Martin
0
mdietz
Asked:
mdietz
  • 4
  • 2
  • 2
1 Solution
 
FrodoBegginsCommented:
Use Application.ProcessMessages; in your loop
0
 
FrodoBegginsCommented:
But be careful - prevent event handler reentrancy. It occures when your loop is in an event handler, and the verry same event occures and takes place when you call the message pump.
0
 
FrodoBegginsCommented:
example:
procedure TfrmListRez.pmenuPorListClick(Sender:TObject);
begin
  if bFlag the exit;
  bFlag := true;
  repeat
    ...; // do something
    Application.ProcessMessages;
  until time_getTime>actual_time+t*1000;
  bFlag := false;
end;
0
Get your problem seen by more experts

Be seen. Boost your question’s priority for more expert views and faster solutions

 
WiseGuyCommented:
Don't use a loop

Make Semaphore a property and in SetSemaphore react when the value is CHANGED to true
0
 
mdietzAuthor Commented:
Thanks, now it works fine.
0
 
WiseGuyCommented:
The solution is a busy form of waiting. It WILL slow down your machine.
My solution wouldn't cause an extra overhead.

0
 
FrodoBegginsCommented:
Well, WiseGuy, I can't see a big slowdown. Your solution relies in the message pump too - but indirectly. And you will  receive the messages later, then if you are in a loop waiting for them.
0
 
mdietzAuthor Commented:
To make it clear: After the PictureScreen.Show(); is called a routine WaitForSeconds(t:integer) is called. Inside this routine is the first time that the parameter t is interpreted, and after that routine the next picture is loaded. The interior of WaitForSeconds should not be cared about by the other procedures, and if I implement the Semaphore as an active element (which processes the List construct and loads the images) I would have to adapt the main procedure, and that would have great effect on the component structure.

 Anyway, there's nothing else to do for the computer while the image is shown, so why don't implement a busy waiting solution.
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

Get your problem seen by more experts

Be seen. Boost your question’s priority for more expert views and faster solutions

  • 4
  • 2
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now