这是异步记录某些事件的安全且*相对好的*方法吗？

Question

使用：

• .NET 4.5.1
• 网络表单
• 实体框架 6（每个请求的上下文）
• IIS 8、Windows 2012 数据中心

主要关注点：线程安全性和可靠性。

该项目由一个封闭的系统组成，可供多种类型的用户使用，这些用户将执行各种操作。在我们当前的项目中，我决定实现一些可能大多数开发人员认为绝对必要的东西。

过去，由于缺乏最简单的日志系统，我们无法跟踪一些常见的用户操作，尤其是操纵数据，因此存在严重的问题。

我知道有流行的日志框架，但我想实现一些相对简单的不会阻塞主线程的东西。

我的想法是在主线程上提取我需要的所有数据，因为事实证明有些数据更难从单独的线程访问，然后创建一个任务来处理数据库插入。我对多线程的了解是有限的，这就是我想出的。

public static class EventLogger
    {
        public static void LogEvent(int eventType, string descr, Exception ex = null)
        {
            //Get the page that is currently being executed;
            var executingPage = HttpContext.Current.CurrentHandler as Page;
            string sourcePage = executingPage != null ? executingPage.AppRelativeVirtualPath : string.Empty;

            var eventToAdd = new Event()
            {
                Date = DateTime.Now,
                EventTypeId = eventType,
                isException = ex != null ? true : false,
                ExceptionDetails = ex != null ? ex.Message : string.Empty,
                Source = sourcePage,
                UserId = UserHelper.GetUserGuid(),
                UserIP = UserHelper.GetUserIP(),
                UserName = UserHelper.GetUserName(),
                Description = descr
            };

            Task.Factory.StartNew(() => LogEventAsync(eventToAdd));
        }

        private static void LogEventAsync(Event eventToAdd)
        {
            using (var context = new PlaceholderEntities())
            {
                context.Events.Add(eventToAdd);
                context.SaveChanges();
            }
        }
    }

问题：

• 这是一种足够好的方式来记录我需要的内容吗？在多用户环境中是否安全？
• 如果您不想深入研究日志框架，您会怎么做？

Answer 1

是的，它在多线程环境中是安全的。因为每次将事件插入数据库时​​都会创建数据库上下文的新实例，所以您是安全的。仅当您尝试跨线程重用相同的上下文实例时，EF 才不是线程安全的。

唯一可能的问题是，以异步方式执行此操作可能意味着同时打开多个连接并且连接池可能会被耗尽。您登录的次数越多，发生这种情况的可能性就越高。

话虽如此，我仍然建议您使用 log4net 或任何现有的日志记录基础设施，并找到一种异步记录的方法。人们经常写博客如何使用 log4net 做到这一点，例如看看这里

http://www.ben-morris.com/using-asynchronous-log4net-appenders-for-high-performance-logging

请注意，此处还讨论了与异步日志记录相关的问题。要解决某些问题，例如条目排序错误的可能性，您可能会在此类自定义附加程序中设置一个队列。坚持现有的框架可以让你重用很多已经开发的想法。当需求发生变化时，迟早要靠自己做事会让你停顿更久。

Answer 2

正如@gbjbaanb 所提到的，您应该将其实现为一个队列，其中一个或多个线程实际上通过 EF 执行数据库工作。您可以使用由ConcurrentQueue<T> 支持的BlockingCollection<T> 以生产者/消费者的方式执行此操作。基本前提是每个调用者记录的东西只是添加到队列中（即生产者）。然后，您可以让一个或多个线程从队列中提取信息并持久保存到数据库（即消费者）。每个线程都需要一个单独的上下文。

BlockingCollection<T> 文档页面上有一个合理的示例。

Answer 3

不，不是。 EF6 is not thread-safe（嗯，DBContext 不是线程安全的，所以如果每个线程有 1 个 DBcontext，你应该没问题），所以添加日志事件可能（哈！将）干扰另一个线程写入不同的日志事件。

您需要做的是同步任务中对 EF 的所有调用，我要做的是将日志添加到集合中，然后让另一个线程将这些日志从集合中拉出并将它们写入数据库， 一次一个。对集合的添加/删除调用必须使用锁来保护，以阻止 2 个线程同时添加（或删除）一个条目。

或者我会使用 log4net，is thread-safe，所以你可以简单地在你的线程中调用它来进行写入或日志条目。

我的另一个建议是将日志写入文件而不是数据库。文件记录很快，因此您可以在主线程上以最小的性能影响执行它（log4net 也非常有效，所以只需使用它 - 我知道 IIS 无论如何都会将访问和错误日​​志写入文件）这将消除您的问题带螺纹。

我真正知道的一件事是，如果您对多线程不太感兴趣，那么您需要停止使用它，直到您对它感兴趣为止。没有不尊重您的尝试，但是线程错误可能是要解决的噩梦，您无法重现它们，也无法在调试器中轻松捕获它们，而且它们的原因通常与报告的症状无关。