IT-Swarm.Net

等到文件完全写入

在一个目录中创建文件(FileSystemWatcher_Created)时,我将其复制到另一个目录中。但是当我创建一个大(> 10MB)文件时,它无法复制文件,因为它已经开始复制,当文件尚未完成创建时...
这会导致 无法复制文件,因为它被另一个进程使用 被引发。 (
任何帮助?

class Program
{
    static void Main(string[] args)
    {
        string path = @"D:\levan\FolderListenerTest\ListenedFolder";
        FileSystemWatcher listener; 
        listener = new FileSystemWatcher(path);
        listener.Created += new FileSystemEventHandler(listener_Created);
        listener.EnableRaisingEvents = true;

        while (Console.ReadLine() != "exit") ;
    }

    public static void listener_Created(object sender, FileSystemEventArgs e)
    {
        Console.WriteLine
                (
                    "File Created:\n"
                   + "ChangeType: " + e.ChangeType
                   + "\nName: " + e.Name
                   + "\nFullPath: " + e.FullPath
                );
        File.Copy(e.FullPath, @"D:\levan\FolderListenerTest\CopiedFilesFolder\" + e.Name);
        Console.Read();
    }
}
62
levi

您面临的问题只有解决方法。.

在开始复制过程之前检查文件是否正在进行中。您可以调用以下函数,直到获得False值。.

第一种方法,直接从 这个答案复制

private bool IsFileLocked(FileInfo file)
{
    FileStream stream = null;

    try
    {
        stream = file.Open(FileMode.Open, FileAccess.ReadWrite, FileShare.None);
    }
    catch (IOException)
    {
        //the file is unavailable because it is:
        //still being written to
        //or being processed by another thread
        //or does not exist (has already been processed)
        return true;
    }
    finally
    {
        if (stream != null)
            stream.Close();
    }

    //file is not locked
    return false;
}

第二种方法:

const int ERROR_SHARING_VIOLATION = 32;
const int ERROR_LOCK_VIOLATION = 33;
private bool IsFileLocked(string file)
{
    //check that problem is not in destination file
    if (File.Exists(file) == true)
    {
        FileStream stream = null;
        try
        {
            stream = File.Open(file, FileMode.Open, FileAccess.ReadWrite, FileShare.None);
        }
        catch (Exception ex2)
        {
            //_log.WriteLog(ex2, "Error in checking whether file is locked " + file);
            int errorCode = Marshal.GetHRForException(ex2) & ((1 << 16) - 1);
            if ((ex2 is IOException) && (errorCode == ERROR_SHARING_VIOLATION || errorCode == ERROR_LOCK_VIOLATION))
            {
                return true;
            }
        }
        finally
        {
            if (stream != null)
                stream.Close();
        }
    }
    return false;
}
39
Romil Kumar Jain

快速,可靠且价格合理的云托管

注册并在30天内获得$50奖金!

这是一个旧线程,但我会为其他人添加一些信息。.

我在编写PDF文件的程序时遇到了类似的问题,有时它们需要30秒才能渲染..这与我的watcher_FileCreated类在复制文件之前等待的时间相同。.

文件未锁定。.

在这种情况下,我检查了PDF的大小,然后在比较新大小之前等了2秒,如果它们不相等,则线程将睡眠30秒并再次尝试。.

10
Johnny Grimes

FileSystemWatcher的文档:

创建文件后立即引发OnCreated事件。如果正在将文件复制或传输到监视目录中,则会立即引发OnCreated事件,然后是一个或多个OnChanged事件。.

因此,如果复制失败,(捕获异常),将其添加到仍需要移动的文件列表中,并在OnChanged事件期间尝试复制。最终,它应该工作。.

像(不完整;捕获特定异常,初始化变量等):

    public static void listener_Created(object sender, FileSystemEventArgs e)
    {
        Console.WriteLine
                (
                    "File Created:\n"
                   + "ChangeType: " + e.ChangeType
                   + "\nName: " + e.Name
                   + "\nFullPath: " + e.FullPath
                );
        try {
            File.Copy(e.FullPath, @"D:\levani\FolderListenerTest\CopiedFilesFolder\" + e.Name);
        }
        catch {
            _waitingForClose.Add(e.FullPath);
        }
        Console.Read();
    }

    public static void listener_Changed(object sender, FileSystemEventArgs e)
    {
         if (_waitingForClose.Contains(e.FullPath))
         {
              try {
                  File.Copy(...);
                  _waitingForClose.Remove(e.FullPath);
              }
              catch {}
         }
   }
9
Steve Czetty

你真的很幸运 - 编写文件的程序将其锁定,因此你无法打开它。如果它没有锁定它,你会复制一个部分文件,而不知道有问题。.

当您无法访问文件时,您可以假设它仍在使用中(更好的是 - 尝试以独占模式打开它,看看是否有其他人正在打开它,而不是从File.Copy的失败中猜测)。如果文件被锁定,您将不得不在其他时间复制它。如果它没有被锁定,你可以复制它(这里有一个竞争条件的可能性很小)。.

什么时候是'其他时间'?我不记得FileSystemWatcher每个文件发送多个事件 - 检查出来,它可能足以让你简单地忽略该事件并等待另一个事件。如果没有,您可以随时设置时间并在5秒内重新检查文件。.

5
zmbq

那你已经自己给出了答案;你必须等待文件的创建才能完成。一种方法是通过检查文件是否仍在使用。这方面的一个例子可以在这里找到: 有没有办法检查文件是否正在使用?

请注意,您必须修改此代码才能使其适用于您的情况。你可能想要像(伪代码)这样​​的东西:

public static void listener_Created()
{
   while CheckFileInUse()
      wait 1000 milliseconds

   CopyFile()
}

显然,如果所有者应用程序永远不会释放锁,你应该保护自己免受无限while的攻击。此外,可能值得查看您可以订阅的FileSystemWatcher中的其他事件。可能有一个事件可以用来规避整个问题。.

2
pyrocumulus

因此,快速浏览了其中一些和其他类似的问题,我今天下午进行了一次快乐的追逐尝试使用文件作为同步(以及文件保存)方法解决两个独立程序的问题。有点不寻常的情况,但它肯定突出了'检查文件是否被锁定的问题,然后打开它,如果它不是'接近。.

问题是:文件可以在您检查它的时间和实际打开文件的时间之间 变成 锁定。它很难追踪零星的 无法复制文件,因为它被另一个进程使用 错误,如果你也没有找到它。.

基本的解决方案是尝试在catch块中打开文件,这样如果它被锁定,你可以再试一次。这样,在检查和打开之间没有经过的时间,操作系统同时执行它们。.

这里的代码使用File.Copy,但它与File类的任何静态方法一样工作:File.Open,File.ReadAllText,File.WriteAllText等。.

/// <param name="timeout">how long to keep trying in milliseconds</param>
static void safeCopy(string src, string dst, int timeout)
{
    while (timeout > 0)
    {
        try
        {
            File.Copy(src, dst);

            //don't forget to either return from the function or break out fo the while loop
            break;
        }
        catch (IOException)
        {
            //you could do the sleep in here, but its probably a good idea to exit the error handler as soon as possible
        }
        Thread.Sleep(100);

        //if its a very long wait this will acumulate very small errors. 
        //For most things it's probably fine, but if you need precision over a long time span, consider
        //   using some sort of timer or DateTime.Now as a better alternative
        timeout -= 100;
    }
}

关于parellelism的另一个小注释: 这是一个同步方法,它会在等待和处理线程时阻塞它的线程。这是最简单的方法,但如果文件长时间保持锁定状态,您的程序可能会无响应。在这里,Parellelism是一个太深入的主题,(并且你可以设置异步读/写的方式的数量有点荒谬),但这是一种可以被解决的方法。.

public class FileEx
{
    public static async void CopyWaitAsync(string src, string dst, int timeout, Action doWhenDone)
    {
        while (timeout > 0)
        {
            try
            {
                File.Copy(src, dst);
                doWhenDone();
                break;
            }
            catch (IOException) { }

            await Task.Delay(100);
            timeout -= 100;
        }
    }

    public static async Task<string> ReadAllTextWaitAsync(string filePath, int timeout)
    {
        while (timeout > 0)
        {
            try {
                return File.ReadAllText(filePath);
            }
            catch (IOException) { }

            await Task.Delay(100);
            timeout -= 100;
        }
        return "";
    }

    public static async void WriteAllTextWaitAsync(string filePath, string contents, int timeout)
    {
        while (timeout > 0)
        {
            try
            {
                File.WriteAllText(filePath, contents);
                return;
            }
            catch (IOException) { }

            await Task.Delay(100);
            timeout -= 100;
        }
    }
}

以下是它的使用方法:

public static void Main()
{
    test_FileEx();
    Console.WriteLine("Me First!");
}    

public static async void test_FileEx()
{
    await Task.Delay(1);

    //you can do this, but it gives a compiler warning because it can potentially return immediately without finishing the copy
    //As a side note, if the file is not locked this will not return until the copy operation completes. Async functions run synchronously
    //until the first 'await'. See the documentation for async: https://msdn.Microsoft.com/en-us/library/hh156513.aspx
    CopyWaitAsync("file1.txt", "file1.bat", 1000);

    //this is the normal way of using this kind of async function. Execution of the following lines will always occur AFTER the copy finishes
    await CopyWaitAsync("file1.txt", "file1.readme", 1000);
    Console.WriteLine("file1.txt copied to file1.readme");

    //The following line doesn't cause a compiler error, but it doesn't make any sense either.
    ReadAllTextWaitAsync("file1.readme", 1000);

    //To get the return value of the function, you have to use this function with the await keyword
    string text = await ReadAllTextWaitAsync("file1.readme", 1000);
    Console.WriteLine("file1.readme says: " + text);
}

//Output:
//Me First!
//file1.txt copied to file1.readme
//file1.readme says: Text to be duplicated!
2
ashbygeek

当文件以二进制(逐字节)写入时,创建FileStream及以上解决方案不起作用,因为文件已准备好并且在每个字节中编写,因此在这种情况下,您需要其他解决方法,如下所示:在创建文件时执行此操作或者您需要开始处理文件

long fileSize = 0;
currentFile = new FileInfo(path);

while (fileSize < currentFile.Length)//check size is stable or increased
{
  fileSize = currentFile.Length;//get current size
  System.Threading.Thread.Sleep(500);//wait a moment for processing copy
  currentFile.Refresh();//refresh length value
}

//Now file is ready for any process!
1
Mohsen.Sharify

您可以使用以下代码检查是否可以使用独占访问权打开文件(即,它不是由其他应用程序打开的)。如果文件未关闭,您可以稍等片刻再次检查,直到文件关闭,您可以安全地复制它。.

您仍应检查File.Copy是否失败,因为另一个应用程序可能会在您检查文件和复制文件之间打开文件。.

public static bool IsFileClosed(string filename)
{
    try
    {
        using (var inputStream = File.Open(filename, FileMode.Open, FileAccess.Read, FileShare.None))
        {
            return true;
        }
    }
    catch (IOException)
    {
        return false;
    }
}
0
Michael