Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 175
  • 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
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.

 
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

Featured Post

Hire Technology Freelancers with Gigs

Work with freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely, and get projects done right.

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