Using C# 4.0, I've created a System.Diagnostics.Process
that I expect to take a short amount of time to run. If for some reason the process hasn't exited after some amount of time (e.g, I've called .WaitForExit(timeout)
and the return value was false
), I need to cleanup. I开发者_开发技巧've decided that it's safe to use .Kill()
in this particular situation (there are no data structures I'm worried about corrupting).
Given this setup, do I also need to call the .Close()
method of the Process? If so, should I call .Close()
before or after .Kill()
?
System.Diagnostics.Process
implements IDisposable
via the Component
class, and the protected Dispose(bool)
method calls Close()
. It is generally considered good practice to dispose of disposable resources when you are done with them, as this will immediately release any resources associated with the class (not having to wait for the GC to run).
So to answer your question:
Yes, please call Close()
by calling the Dispose()
method directly or via the C# using
construct as follows.
using(Process proc = CreateProcess())
{
StartProcess(proc);
if (!proc.WaitForExit(timeout))
{
proc.Kill();
}
}
Process.Kill()
and Process.Close()
both internally call Close()
on the private SafeProcessHandle object associated with the process. However, Process.Close()
also explicitly resets several other internal members.
Since Process implements IDisposable
, you should always call Close()
on it when you are done, or wrap it in a using
block.
精彩评论