C#多线程环境下类级别变量跨函数使用的风险与疑问

首先直接给结论：你这段代码在多线程环境下绝对会出现不可预期的故障，而且当两个用户访问不同项目版本时，key的值大概率会随机混乱，完全不符合你的预期。

问题根源 #

先澄清一个小细节：你代码里的key是实例级别变量（不是类级别，类级别需要加static修饰），但不管是实例还是类静态变量，只要多个线程共享同一个Test实例，对key的无同步读写就会触发竞态条件（Race Condition）。

举个具体场景：

• 线程1（用户A，对应版本2.3）调用Get方法，正在执行key = setToSOmething();，还没完成赋值；
• 同时线程2（用户B，对应版本3.0）调用Set方法，直接完成了key的赋值操作；
• 最终线程1拿到的key会是用户B的版本值，完全搞混了。

而且就算字符串赋值在C#里是原子操作，如果setToSOmething()是复杂计算，或者后续有基于key的其他读写操作，线程间的干扰会导致key的值完全不可控。

解决方案 #

根据你的实际需求，有两种常见的处理方式：

1. 每个线程独立持有变量（推荐如果不需要共享）

如果每个用户/线程的key不需要在方法间共享，直接把key改成方法内部的局部变量：

class Test() { 
    public void Get(string something) { 
        string key = setToSOmething(); 
        // 在这里使用key，每个线程有自己的独立副本，完全不会冲突
    } 

    public void Set(string something) { 
        string key = setToSOmething(); 
        // 同理，这里的key也是当前线程独有的
    } 
}

2. 必须共享时添加同步机制

如果确实需要多个线程共享同一个key，必须给读写操作加锁，保证同一时间只有一个线程能修改或读取key：

class Test() { 
    private string key; 
    // 用一个专用的锁对象，不要用this或其他可能被外部锁定的对象
    private readonly object _locker = new object();

    public void Get(string something) { 
        lock(_locker) {
            key = setToSOmething(); 
            // 如果后续需要读取key，也要放在lock块里
        }
    } 

    public void Set(string something) { 
        lock(_locker) {
            key = setToSOmething(); 
        }
    } 
}

这样就能保证key的修改是线程安全的，不会出现值被随机覆盖的情况。

额外提醒 #

多线程编程中，共享可变状态是最大的坑之一，如果没有必要，尽量避免让多个线程访问同一个可变变量。如果必须共享，一定要用合适的同步机制（lock、Interlocked、线程安全容器等）来保护。

内容的提问来源于stack exchange，提问作者learntogrow-growtolearn