良いコーディングですか？ （複数のメッセージループ）

Question

私は適切なコーディングでいくつかの助言が必要です：

私は複数のシリアル接続が使用されているプログラムで作業しています。各通信回線には、抽象化レイヤとして動作するコントローラがあります。コントローラとシリアルポートの間にプロトコルが挿入されてデータをパッケージに包み込み、転送の準備ができます。プロトコルが失敗した配送、再送信などを処理します。 GUIがハングしないように、各接続回線（プロトコルとシリアルポート）は別のスレッドで作成されます。コントローラーはGUIのコントロールを持っているので、メインスレッドによって処理されます。

現在、私はスレッドを作成するときに、メッセージループ（Application.Run（））を作成するように選択したので、バッファをポーリングする代わりに仕事がなければ単にスレッド（BeginInvoke）バッファとしてのメッセージループ。これは現在うまく機能し、これまでのところ深刻な問題はありません。

これは「良いコーディング」ですか、それともトレッドにwhileループを使用してポーリングバッファを使用するべきですか？

私はコードを表示したいと思いますが、今のところ数千のコード行ですので、コードの一部を見たい場合は具体的にしてください。 :)

ありがとうございます。

Answer 1

各スレッドでメッセージループを使用すると問題はありません。 Windowsはこのシナリオに合わせて最適化されています。あなたはポーリングを回避するのは当然ですが、転送のためのパッケージを準備して、準備ができたスレッド、またはセマフォとスレッドセーフのキューを通知するためにSetEventを呼び出すなど、より効率的な他のイベントベースの設計を調べることができます。マーティン・ジェイムスが示唆

Answer 2

私はあなたがここでやっていることは100％確実ではないんだけど、あなたのアプリケーションがアイドル状態の場合、それは悪い:)

を鳴らない「に充填」（なし途切れ）のビットと、ありますCPU使用率は0％ですか？

あなたのアプリはスリープ（0）/スリープ（1）、または同様のポーリングループから解放されていますか？

それはかなり低い待ち時間で動作しますか？答えは3つある場合

「YES」、あなたは問題ないはずです:)

あり、数、（非常に少ない！）、などの結果のポーリングは良いアイデア、（ある場合など。ときスレッド内のイベントの頻度は非常に高く、GUIへのすべての進捗イベントを通知するとそれを圧倒するでしょう）が、ほとんどの場合、設計が貧弱です。