Implementacja C# timeout

Question

Czy używanie tej pętli jest złym rozwiązaniem? Może lepiej jest użyć stopera, a może to rozwiązanie ma pewne pułapki?

public void DoWork() 
    { 
     //do some preparation 
     DateTime startTime = DateTime.Now; 
     int rowsCount = 0; 
     int finalCount = getFinalCount(); 
     do 
     { 
      Thread.Sleep(1000); 
      rowsCount = getRowsCount(); // gets rows count from database, rows are added by external app. 
     } while (rowsCount < finalCount && DateTime.Now - startTime < TimeSpan.FromMinutes(10)); 

    } 

Widziałem ten artykuł Implement C# Generic Timeout, ale jest zbyt złożona, aby korzystać w prostych scenariuszy - trzeba myśleć o synchronizacji wątków, jest to właściwe, aby je przerwać, czy nie i tak dalej.

Answer 1

Jak rozumiem, chcesz, aby twoja metoda wykonywała jakąś pracę, dopóki nie zostanie ukończona lub dopóki nie upłynie pewien okres czasu? Chciałbym użyć Stopwatch za to, i sprawdzić czas, który upłynął w pętli:

void DoWork() 
{ 
    // we'll stop after 10 minutes 
    TimeSpan maxDuration = TimeSpan.FromMinutes(10); 
    Stopwatch sw = Stopwatch.StartNew(); 
    DoneWithWork = false; 

    while (sw.Elapsed < maxDuration && !DoneWithWork) 
    { 
     // do some work 
     // if all the work is completed, set DoneWithWork to True 
    } 

    // Either we finished the work or we ran out of time. 
} 

Answer 2

Lepiej jest używać klasy System.Timers.Timer.