在打开过程中放置​​使用块?

时间:2013-10-15 15:32:59

标签: c#

我有正在打开的流程并在dist上创建文件并订阅此过程ProcessExitedEvent

public int InvokeProcess(WiresharkProcesses process, string args)
{
    try
    {
        string processToInvoke = null;
        ProcessStartInfo startInfo = new ProcessStartInfo();
        Process pros = new Process();
        startInfo .FileName = processToInvoke;
        startInfo .RedirectStandardOutput = true;
        startInfo .RedirectStandardError = true;
        startInfo .RedirectStandardInput = true;
        startInfo .UseShellExecute = false;
        startInfo .CreateNoWindow = true;
        startInfo .Arguments = args;
        pros.StartInfo = startInfo ;
        pros.ErrorDataReceived += pros_ErrorDataReceived;
        pros.OutputDataReceived += pros_OutputDataReceived;
        pros.Exited += (object sender, EventArgs e) =>
        {
            if (ProcessExitedEvent != null)
                ProcessExitedEvent(pros.Id);
        };
        pros.EnableRaisingEvents = true;
        pros.Start();
        pros.BeginOutputReadLine();
        pros.BeginErrorReadLine();

        return pros.Id;
    }
    catch (Exception)
    {
        return -1;
    }
}

private void ProcessExitedEvent(int processId)
{
   // Copy file
   // Delete file
}

此过程在磁盘上创建文件,在ProcessExitedEvent事件触发后我想将此文件复制到另一个位置而不是删除旧文件,但是虽然事件在进程退出后触发但我的代码无法删除旧文件,因为该文件仍在使用中,所以我希望使用using block确保我的进程死亡 所以我需要把这个块放在哪里?

4 个答案:

答案 0 :(得分:4)

我非常怀疑using声明会对此有所帮助。 Process.Dispose不会杀死其他进程 - 它只是将CLR句柄释放给另一个进程。 (老实说,我不记得每个人都使用using语句Process - 与大多数可支配资源不同,我不会担心这个。)

我很惊讶您无法删除该文件,但可能是操作系统只是刷新缓冲区或某些东西,因为进程终止。您可能想要在删除文件之前等待一秒钟......虽然不应该真的是必要的。你确定它不像病毒检查器那样拿起文件吗?

答案 1 :(得分:0)

我会移动文件而不是复制和删除它。是否有理由需要复制文件然后删除原始文件?为什么不移动文件呢?如果它被移动到同一个物理硬盘驱动器上,通常移动它比复制和删除它要便宜得多,如果不是,你仍然可以至少保存文件操作。

答案 2 :(得分:0)

在我看来,使用它可能会解决您的问题而不是解雇事件。有些事情发生在我身上,我使用WaitForExit()来解决它。

var process = Process.Start(...);
process.WaitForExit();

// Delete your file.

答案 3 :(得分:0)

试试这个并告诉我这是否解决了这个问题:

    public void InvokeProcess(WiresharkProcesses process, string args)
    {
        Task.Factory.StartNew(() =>
        {
            string processToInvoke = null;
            ProcessStartInfo startInfo = new ProcessStartInfo();
            using (Process pros = new Process())
            {
                startInfo.FileName = processToInvoke;
                startInfo.RedirectStandardOutput = true;
                startInfo.RedirectStandardError = true;
                startInfo.RedirectStandardInput = true;
                startInfo.UseShellExecute = false;
                startInfo.CreateNoWindow = true;
                startInfo.Arguments = args;
                pros.StartInfo = startInfo;

                //pros.EnableRaisingEvents = true;

                pros.Start();
                pros.WaitForExit();
            }
            // Copy file
            // Delete file
        });
    }