How to handle recurring execution?
- by ShaneC
I am trying to validate the solution I came up for what I think is a fairly typical problem. I have a service running and every 10 minutes it should do something. I've ended up with the following:
private bool isRunning = true;
public void Execute()
{
while(isRunning)
{
if(isRunning)
{
DoSomething();
m_AutoResetEvent.WaitOne(new Timespan(0, 10, 0));
}
}
}
public void Stop()
{
isRunning = false;
m_AutoResetEvent.Set();
}
The immediate potential problems I can see is that I'm not doing any sort of locking around the isRunning modification in Stop() which gets called by another thread but I'm not sure I really need to? The worst that I think could happen is that it runs one extra cycle.
Beyond that are there any obvious problems with this code? Is there a better way to solve this problem that I'm unaware of?
