.NET 4.8中如何线程安全读取NLog MemoryTarget的Logs属性

核心问题：NLog的MemoryTarget.Logs内部基于List<string>实现，NLog仅保证写入操作的线程安全，但List<T>本身并非线程安全集合。当读取时（如调用Reverse()、ToList()或直接访问Count），若其他线程正在执行写入（如Add），会触发集合修改异常或读取到不一致状态。

你的思路是否正确？ #

思路有一定合理性，但存在关键风险：

1. 依赖NLog内部实现的脆弱性：
   通过Logger.Logs as List<string>强转，完全依赖当前NLog版本的内部实现（MemoryTarget.Logs确实返回List<string>）。若未来NLog修改Logs的返回类型，这段代码会进入降级分支，而降级分支的Logger.Logs.Reverse().ToList()依然存在线程安全问题。
2. CopyTo的并发隐患：
   List<T>.CopyTo并非线程安全方法。当其他线程执行Add触发内部数组扩容时，CopyTo可能读取到新旧数组混合的内容，甚至抛出异常。“日志只增不减”的假设仅能避免元素移除的问题，无法解决扩容时的并发冲突。

更可靠的解决方案 #

针对.NET 4.8场景，推荐以下方案：

方案1：使用NLog官方线程安全API（优先推荐） #

NLog的MemoryTarget内置了线程安全的日志获取方法GetLogs()，该方法会原子性复制当前所有日志条目，返回新数组，彻底规避并发问题。

修改Logs属性实现：

public List<string> Logs 
{ 
    get
    {
        // GetLogs()是线程安全的，返回当前日志的副本数组
        return Logger.GetLogs().Reverse().ToList();
    }
}

注：GetLogs()在NLog 4.0+版本已存在，自动处理并发写入逻辑，无需手动实现复制。

方案2：借助NLog内部锁实现安全读取（备选） #

若无法使用GetLogs()，可通过反射获取MemoryTarget内部的同步锁，读取时锁定该对象，阻塞写入操作以保证线程安全：

public List<string> Logs 
{ 
    get
    {
        // 反射获取MemoryTarget内部的_syncRoot锁对象
        var syncRootField = typeof(MemoryTarget).GetField("_syncRoot", BindingFlags.NonPublic | BindingFlags.Instance);
        var syncRoot = syncRootField?.GetValue(Logger) ?? new object();
        
        lock(syncRoot)
        {
            // 锁定期间读取并反转日志，写入操作会被阻塞
            return Logger.Logs.Reverse().ToList();
        }
    }
}

注意：反射依赖NLog内部实现，未来版本可能失效，仅作为方案1的备选。

方案3：自定义线程安全日志目标（灵活扩展） #

若需更频繁的读取操作或担心MemoryTarget性能，可自定义继承Target的日志目标，内部使用ConcurrentQueue<string>存储日志：

public class ThreadSafeMemoryTarget : Target
{
    private readonly ConcurrentQueue<string> _logs = new ConcurrentQueue<string>();

    public IEnumerable<string> Logs => _logs.Reverse();

    protected override void Write(LogEventInfo logEvent)
    {
        var logMessage = RenderLogEvent(Layout, logEvent);
        _logs.Enqueue(logMessage);
    }

    public string[] GetLogs()
    {
        return _logs.ToArray();
    }
}

替换原MemoryTarget使用该自定义目标即可。

总结 #

你的思路试图绕过线程安全问题，但依赖内部实现且仍有并发风险。最稳妥的方式是使用NLog官方提供的GetLogs()方法，它已封装了线程安全的细节，无需手动实现复杂逻辑。

内容的提问来源于stack exchange，提问作者Jean-Baptiste Zeller