C＃ビットマップメモリ​​の使用を処理する

Question

私は現在私が作成しているアプリにウェブカメラの機能を実装していますが、メモリの使用に関するいくつかの問題がありました。これは、しかし、このコードが（私は仮定ラムの）2ギガバイトを実行して、巨大なメモリリークを引き起こした最初の実装

private void webcam_newFrame(object sender, NewFrameEventArgs eventArgs) { 
    if (!_pause) { 
     var img = (Bitmap) eventArgs.Frame.Clone(); 
     pbPicture.BackgroundImage = img;    
    } 
} 

ある

以下のように

私の関数です。メモリ不足のためにクラッシュする前に。 そう

private void webcam_newFrame(object sender, NewFrameEventArgs eventArgs) { 
    if (!_pause) { 
     var img = (Bitmap) eventArgs.Frame.Clone(); 
     pbPicture.BackgroundImage = img; 
     img.Dispose(); // This causes exception 
    } 
} 

これは、例外がスローされた原因となるように私はその後、ビットマップを処分するためにコードを変更：System.Drawing.dllの「System.ArgumentExceptionの」。

追加情報：パラメータが無効です。表示されるためにこのフォームを呼び出して、次のコードで

：

WebcamForm wForm = new WebcamForm(); 
wForm.ShowDialog(); 

だから私はその後、これに機能を変更：

private void webcam_newFrame(object sender, NewFrameEventArgs eventArgs) { 
    if (!_pause) { 
     pbPicture.BackgroundImage = (Bitmap)eventArgs.Frame.Clone(); 
     GC.Collect(); 
    } 
} 

をこのコードは正しく動作し、メモリ使用量が360メガバイトの周りにいるまま一貫性があり、アプリケーションを考慮した合理的な数字です。

私の質問は、すべてのフレームにガベージコレクションを強制するのですか？潜在的な問題はありますか？

ガベージコレクションを強制することは悪いことですが、私はこの問題を回避する方法については考えていませんでした。あなたはあなたが渡している新しい価値、背景画像の古い値を配置する必要はありません。

private void webcam_newFrame(object sender, NewFrameEventArgs eventArgs) { 
    if (!_pause) { 
     var img = (Bitmap) eventArgs.Frame.Clone(); 
     var oldImg = pbPicture.BackgroundImage; 
     pbPicture.BackgroundImage = img; 
     oldImg?.Dispose(); 
    } 
} 

Answer 1

は悪い習慣です。

newoneの後に古いビットマップを配置してみます。

private void webcam_newFrame(object sender, NewFrameEventArgs eventArgs) { 
    if (!_pause) { 
     Bitmap oldBitmap = (Bitmap)pbPicture.BackgroundImage; 
     pbPicture.BackgroundImage = (Bitmap)eventArgs.Frame.Clone(); 
     oldBitmap.Dispose(); 
    } 
} 

Answer 2

しばしばガベージコレクタを呼び出す

乾杯