Question

示例程序：在某些文件夹上侦听FileSystem事件，并在Timer事件触发时将FileSystem事件信息打印到控制台。

class Program
{
    public static string location = @"D:\TestEvents";
    public static double interval = 15000; 

    public static System.Timers.Timer timer;
    public static List<string> listOfChanges = new List<string>();

    static void Main(string[] args)
    {
        StartWatch();
        StartTimer();

        Console.ReadLine();
    }

    private static void StartWatch()
    {
        FileSystemWatcher Watcher = new FileSystemWatcher();
        Watcher.Path = location;
        Watcher.Created += new FileSystemEventHandler(OnFileCreatedOrDeleted);
        Watcher.Deleted += new FileSystemEventHandler(OnFileCreatedOrDeleted);
        Watcher.EnableRaisingEvents = true;
    }

    static void OnFileCreatedOrDeleted(object sender, FileSystemEventArgs e)
    {
        listOfChanges.Add(String.Format("Change Type: {0}, Name: {1}, Time: {2}", e.ChangeType, e.Name, DateTime.Now));
    }

    private static void StartTimer()
    {
        timer = new System.Timers.Timer();
        timer.AutoReset = false;
        timer.Elapsed += new System.Timers.ElapsedEventHandler(OnTimerEpleased);
        timer.Interval = interval;
        timer.Start();
    }

    private static void OnTimerEpleased(object sender, System.Timers.ElapsedEventArgs e)
    {
        Console.WriteLine("Timer event fired: " + DateTime.Now);
        foreach (var item in listOfChanges)
        {
            Console.WriteLine(item);
        }
        Console.WriteLine();
        listOfChanges.Clear();

        timer.Interval = interval;
        timer.Start();
    }
}

从两个事件处理程序访问相同的存储静态List<string> listOfChanges是否安全？我真的不明白事件是如何运作的。是否创建了一些全局事件处理程序队列，并且尽管有事件类型，它仍然逐个运行所有事件处理程序？或者它为每个事件处理程序类型创建不同的线程？

修改我想最好的解决方案是将BlockingCollection与ConcurrentQueue一起使用，所以它应该是这样的：

public static BlockingCollection<string> listOfChanges = new BlockingCollection<string>();

static void OnFileCreatedOrDeleted(object sender, FileSystemEventArgs e)
{
    listOfChanges.Add(String.Format("Change Type: {0}, Name: {1}, Time: {2}", e.ChangeType, e.Name, DateTime.Now));
}

private static void OnTimerEpleased(object sender, System.Timers.ElapsedEventArgs e)
{
    Console.WriteLine("Timer event fired: " + DateTime.Now);
    while (listOfChanges.Count > 0)
    {
        string item;
        bool b = listOfChanges.TryTake(out item);
        if (b)
        {
            Console.WriteLine(item);
        }
    }
    Console.WriteLine();

    timer.Interval = interval;
    timer.Start();
}

Answer 1

通用List不是线程安全的。遍历listOfChanges时（在OnTimerEpleased中）可能会出现错误，并且新条目会在列表中添加一个条目（OnFileCreatedOrDeleted）。见http://msdn.microsoft.com/en-us/library/6sh2ey19.aspx。您可以同步对列表的访问权限，也可以使用内置的线程安全集合：http://msdn.microsoft.com/en-us/library/system.collections.concurrent.aspx

Answer 2

我不确定FileSystemWatcher是否使用多个线程，但只是为了安全地包装您对

中的列表的访问权限

lock (listOfChanges)
{
    //Code that reads or writes to the listOfChanges.
}

Answer 3

您不应该假设，这些事件处理程序将从单个线程调用。 FileSystemWatcher和Timer的文档都没有提到这些处理程序是如何被调用的，所以我会在这里选择安全的方法并确保我自己对这个列表的访问是同步的。

C＃事件和线程

3 个答案: