提高 FileSystemWatcher 的性能

Question

我有以下代码注册FileSystemWatcher，然后在指定文件上发生写入事件时运行一些方法DoRequiedWork()。还有一些逻辑可以防止在每次写入时触发多个事件（使用lastRead 和lastWriteTime）：

// To prevent multiple events from firing
static DateTime lastRead = DateTime.MinValue;
static string UserName = GetUserName();

private static void Run()
{
    // Create a new FileSystemWatcher and set its properties.
    var watcher = new FileSystemWatcher
    {
        Path = $@"\\file\home$\{UserName}\Application Data",
        NotifyFilter =  
            NotifyFilters.LastWrite,
        Filter = "filetowatch.txt"
    };

    // Activate
    watcher.Changed += OnChanged;
    watcher.EnableRaisingEvents = true;

    while (true)
    {
        System.Threading.Thread.Sleep(10000);
    }
}

private static void OnChanged(object source, FileSystemEventArgs e)
{
    var lastWriteTime = File.GetLastWriteTime(e.FullPath);

    if (lastWriteTime == lastRead) return;

    DoRequiredWork();

    lastRead = lastWriteTime; 
}

我想使用c++ api ReadDirectoryChangesW 来实现这一点，以尝试提高性能，但我不知道该怎么做。

查看pinvoke，可以看到签名可以定义为：

[DllImport("kernel32.dll")]
 static extern bool ReadDirectoryChangesW(IntPtr hDirectory, IntPtr lpBuffer,
    uint nBufferLength, bool bWatchSubtree, uint dwNotifyFilter, out uint
    lpBytesReturned, IntPtr lpOverlapped,
    ReadDirectoryChangesDelegate lpCompletionRoutine);  

我想开始研究如何创建它，但首先想检查它是否实际上会比标准的托管 c#FileSystemWatcher 执行得更好。

另外（或除此之外），我想知道是否有比以下方法更好的方法让应用程序在后台运行（没有 UI）：

while (true)
{
    System.Threading.Thread.Sleep(10000);
}

我觉得这可能不是保持应用程序打开的最佳方式，但我不确定。

有人能给点建议吗？

Answer 1

看起来您在 Run 方法中保留了一个线程。不要那样做。而是保留一个静态参考。看起来您正在观看网络共享。不要期望高性能，因为它不仅涉及磁盘子系统，还涉及网络子系统（在您的终端和服务器上）。涉及的因素太多了，无法预期在本机调用相同的 API 会突然带来巨大的性能提升。

static FileSystemWatcher watcher;

private static void Run()
{
    // Create a new FileSystemWatcher and set its properties.
    // if you're watching a network share, don't expect huge performance
    // as the network is involved
    watcher = new FileSystemWatcher
    {
        Path = $@"\\file\home$\{UserName}\Application Data",
        NotifyFilter =  
            NotifyFilters.LastWrite,
        Filter = "filetowatch.txt"
    };

    // Activate
    watcher.Changed += OnChanged;
    watcher.EnableRaisingEvents = true;

    AppDomain.CurrentDomain.DomainUnload += (s,e) => { 
      var w = watcher as IDisposable;
      if (w != null) w.Dispose(); 
    };
}