Question

I’m working on a console application which will be scheduled and run at set intervals, say every 30 minutes. Its only purpose is to query a Web Service to update a batch of database rows.

The Web Service API reccommends calling once every 30 seconds, and timeout after a set interval. The following pseudocode is given as an example:

listId := updateList(<list of terms>)
LOOP
  WHILE NOT isUpdatingComplete(listId)
END LOOP
statuses := getStatuses(“LIST_ID = {listId}”)

I have coded this roughly in C# as:

int callCount = 0;
while( callCount < 5 && !client.isUpdateComplete(listId, out messages) )
{
    listId = client.updateList(options, terms, out messages);
    callCount++;
    Thread.Sleep(30000);
}
// Get resulting status...

Is it OK in this situation to use Thread.Sleep()? I’m aware it is not generally good practice but from reading reasons not to use it this seems like acceptable usage.

Thanks.

Answer 1

Thread.Sleep ensures the current thread doesn’t return until at least the specified milliseconds have passed. There are plenty of places it’s appropriate to do that, and your example seems fine, assuming it’s running on a background thread.

Some example places you don’t want to use it – on the UI thread or where you need to do exact timing.