非同期ゲームエンジンループを実装するためのベストプラクティスはありますか？

Question

次のように私は、ゲームエンジンのループを実装している：

public static Boolean Start () 
{ 
    if (hasBoard) 
    { 
     // start engine on worker thread 
     asyncTask = new AsyncResult (stopEngine, asyncTask); 
     isRunning = ThreadPool.QueueUserWorkItem (startEngine, asyncTask); 

     if (isRunning) 
     { 
      Console.WriteLine ("[{0}] Engine started", 
       DateTime.Now.ToString ("hh:mm:ss")); 
     } 
     else 
     { 
      Console.WriteLine ("[{0}] Engine failed to start", 
       DateTime.Now.ToString ("hh:mm:ss")); 
     } 
    } 

    return isRunning; 
} 

public static void Stop () 
{ 
    Console.WriteLine ("[{0}] Engine stopping", 
     DateTime.Now.ToString ("hh:mm:ss")); 

    asyncTask.SetAsCompleted (null, false); 
} 

private static void startEngine (Object task) 
{ 
    while (!((IAsyncResult)task).IsCompleted) 
    { 
     Thread.Sleep (10000); 

     Console.WriteLine ("[{0}] Engine running", 
      DateTime.Now.ToString ("hh:mm:ss")); 
    } 
} 

private static void stopEngine (IAsyncResult iaResult) 
{ 
    // clean up resources 

    Console.WriteLine ("[{0}] Engine stopped", 
     DateTime.Now.ToString ("hh:mm:ss")); 

    isRunning = false; 
} 

私は彼の記事、Implementing the CLR Asynchronous Programming Modelにジェフ・リヒターが推奨するAsyncResultクラスを使用しています。 UIからエンジンを停止できるように、私が使った実装は、標準の非同期パターンからちょっと離れていました。この実装は期待どおりに機能しますが、標準的な習慣から逸脱すると、私はSOコミュニティに戻って、正しい方法で作業するようにします。

誰にでも見えるこの実装に関する問題はありますか？

Answer 1

これはあなたがコントロールしているプロジェクトのように聞こえるので、私はあなたにAPMを捨て、.NET4で提供されているTaskベースのモデルを使用することをお勧めします。これは、APMではなく.NET4の推奨アプローチです。 Taskクラスはタスク並列ライブラリ（TPL）の一部ですが、これらの基本的な非同期ジョブにも最適です。

private CancellationTokenSource cts; 

public void StartEngine() 
{ 
    if (cts == null) 
    { 
     cts = new CancellationTokenSource(); 
     Task.Factory.StartNew(() => GameLoop(cts.Token), cts.Token); 
    } 
} 

private void GameLoop(CancellationToken token) 
{ 
    while (true) 
    { 
     token.ThrowIfCancellationRequested(); 
     Thread.Sleep(1000); 
     Debug.WriteLine("working..."); 
    } 
} 

public void StopEngine() 
{ 
    if (cts != null) 
    { 
     cts.Cancel(); 
     cts = null; 
    } 
} 

Answer 2

私は、スレッドプールのスレッド数が限られているため、スレッドプールを使用するとタスクが短命であることを暗示しています。

winformsについて言及したので、代わりにBackgroundWorkerを使用します。 BWはあなたのために同期を処理します。したがって、InvokeRequiredなどを使用せずにGUIを更新することができます。