whileループでのメモリリーク

Question

whileループでコードを実行して、プログラムが常に実行されていることを確認しています。プログラムは、プログラムがバックアップからコピーし、それがないし、それを開始した場合、それは、それを起動する実行されていない場合は、空想何も：

while (true) 
{ 
    Process backup = new Process(); 
    ProcessStartInfo check = new ProcessStartInfo(file); 
    if (Process.GetProcessesByName(file).Length == 0) 
    { 
     if(File.Exists(file)) 
     { 
      backup.StartInfo = check; 
      backup.Start(); 
     } 
     else if (!File.Exists(file)) 
     { 
      File.Copy(backupFile, file); 
      Thread.Sleep(250); 
      backup.StartInfo = check; 
      backup.Start(); 
     } 
    } 
    backup.Close(); 
    Thread.Sleep(2000); 
} 

問題は各サイクルの後ではありませんRAM使用量が上がります私が知っていることはあまりありませんが、これが1時間ほど実行されていれば、それは大きな問題を引き起こすでしょう。

私はそれを一時停止しようとしましたが、プロセス上で.Close()を使用しましたが、喜んではありませんでした。どんなアイデアも高く評価されています。

Answer 1

1. プロセスが実行されていない場合を除き、それらが実行されないようにif文の内部でこれらの行を入れてください：

   Process backup = new Process(); 
   ProcessStartInfo check = new ProcessStartInfo(file); 
   

2. をProcessがIDisposableを実装しているので、あなたがオデッドとして、usingステートメントでそれをラップすることができます提案する。

3. 実際のメモリリークはありません。ガベージコレクタが実行されると、メモリが再利用されます。一時的なメモリ使用量が問題であれば、GCを強制的に実行することができますが、一度すれば一時的なメモリ使用量に問題はありません。

Answer 2

投稿したコードにはメモリリークはありません。

あなたは、ガベージコレクタ自分でこれは一般的に悪いデザインとみなされ、あなたは絶対にRAMの量を制御する必要があればあなたはこれだけを使用する必要があること

System.GC.Collect(); 

注意を使用して反復のすべてのX量を起動しようとすることができます使っている。 このようなハードリソース要件がある場合は、いずれの場合でもC＃が選択された言語にならないことがあります。

Answer 3

多くのことが言われているように、実際にはメモリリークはありません（少なくともそれはそのようには見えません）。

しかし、初期化をどこかで行うことができます。バックアップを初期化し、必要がないときにチェックすることがよくあります。

while (true) 
{ 
    Process backup; 
    ProcessStartInfo check; 
    if (Process.GetProcessesByName(file).Length == 0) 
    { 
     check = new ProcessStartInfo(file);//moved init 
     backup = new Process();//moved init 
     if(File.Exists(file)) 
     { 
      backup.StartInfo = check; 
      backup.Start(); 
     } 
     else if (!File.Exists(file)) 
     { 
      File.Copy(backupFile, file); 
      Thread.Sleep(250); 
      backup.StartInfo = check; 
      backup.Start(); 
     } 
    } 
    backup.Close(); 
    Thread.Sleep(2000); 
} 

Answer 4

私は同じ問題を抱えていたので、私はこれを使用：

[DllImport("KERNEL32.DLL", EntryPoint = "SetProcessWorkingSetSize", SetLastError = true, CallingConvention = CallingConvention.StdCall)] 
internal static extern bool SetProcessWorkingSetSize(IntPtr pProcess, int dwMinimumWorkingSetSize, int dwMaximumWorkingSetSize); 

[DllImport("KERNEL32.DLL", EntryPoint = "GetCurrentProcess", SetLastError = true, CallingConvention = CallingConvention.StdCall)] 
internal static extern IntPtr GetCurrentProcess(); 

は、これらのメソッドを呼び出して、メモリが解放されます。

Answer 5

Look !,私はこれらのオプションをすべて確認しましたが、何もしませんでした。一方、私はあなたの問題を解決しました。 Create新しいスレッドのApartmentState = STAを設定して、その中にあるコードをこの新しいスレッドで実行するようにします。これを使用すると、すべてのリソースがスレッドの破棄後に解放されます。私はそれを試して、それは動作します！