多线程环境下静态List初始化异常问题排查求助

public class Helper
{
    private static List<TimeZoneMap> timeZoneList = null;

    public static List<TimeZoneMap> GetTimeZoneList()
    {
        if (timeZoneList == null)
        {
            timeZoneList = new List<TimeZoneMap>();
            timeZoneList.Add(new TimeZoneMap() { Id = "Dateline Standard Time", DisplayName = "(UTC-12:00) International Date Line West" });
            timeZoneList.Add(new TimeZoneMap() { Id = "UTC-11", DisplayName = "(UTC-11:00) Coordinated Universal Time-11" });
            timeZoneList.Add(new TimeZoneMap() { Id = "Aleutian Standard Time", DisplayName = "(UTC-10:00) Aleutian Islands" });
        }
    }
}

背景说明 #

Helper为实例类，但内部包含静态List<TimeZoneMap>类型的timeZoneList字段；静态方法GetTimeZoneList()通过null检查完成列表的初始化与元素添加。在多线程环境中，该方法偶尔会返回元素不全的列表或null，200次调用仅随机出现2-3次，无法稳定复现。

问题1：故障根源对应哪项？ #

选项：

• 1. Static List
• 2. Static Method
• 3. 静态列表内初始化实例类？
• 4. 父类不同实例调用同一静态方法？

问题2：将列表设为static readonly集合能否解决初始加载元素不全的问题？ #

具体解答 #

故障根源定位 #

问题出在选项2（Static Method），核心原因是静态方法内的初始化逻辑不具备线程安全性：
多线程并发调用GetTimeZoneList()时，可能多个线程同时通过timeZoneList == null的判断，导致多次执行new List<TimeZoneMap>()，后续线程的初始化会覆盖之前线程创建的列表；或者某个线程正在执行Add元素的过程中，其他线程已经获取到未完全初始化的列表，最终出现元素不全或返回null的情况。静态列表本身不是问题，问题在于初始化逻辑缺少线程同步保护。

static readonly集合的作用分析 #

单纯将列表设为static readonly无法直接解决问题，必须配合线程安全的初始化时机才能生效：
static readonly仅限制字段只能在静态构造函数或声明时赋值，无法被后续修改。如果仍在GetTimeZoneList()的null检查中初始化，依然会存在多线程并发初始化的问题。

正确的解决方案

有两种可靠的线程安全初始化方式：

1. 静态构造函数初始化：CLR会保证静态构造函数的执行是线程安全的，只会执行一次

public class Helper
{
    private static readonly List<TimeZoneMap> timeZoneList;

    static Helper()
    {
        timeZoneList = new List<TimeZoneMap>();
        timeZoneList.Add(new TimeZoneMap() { Id = "Dateline Standard Time", DisplayName = "(UTC-12:00) International Date Line West" });
        timeZoneList.Add(new TimeZoneMap() { Id = "UTC-11", DisplayName = "(UTC-11:00) Coordinated Universal Time-11" });
        timeZoneList.Add(new TimeZoneMap() { Id = "Aleutian Standard Time", DisplayName = "(UTC-10:00) Aleutian Islands" });
    }

    public static List<TimeZoneMap> GetTimeZoneList()
    {
        return timeZoneList;
    }
}

2. 使用Lazy<T>实现延迟线程安全初始化：Lazy<T>默认保证初始化过程的线程安全性，仅在首次访问Value时执行初始化逻辑

public class Helper
{
    private static readonly Lazy<List<TimeZoneMap>> timeZoneList = new Lazy<List<TimeZoneMap>>(() =>
    {
        var list = new List<TimeZoneMap>();
        list.Add(new TimeZoneMap() { Id = "Dateline Standard Time", DisplayName = "(UTC-12:00) International Date Line West" });
        list.Add(new TimeZoneMap() { Id = "UTC-11", DisplayName = "(UTC-11:00) Coordinated Universal Time-11" });
        list.Add(new TimeZoneMap() { Id = "Aleutian Standard Time", DisplayName = "(UTC-10:00) Aleutian Islands" });
        return list;
    });

    public static List<TimeZoneMap> GetTimeZoneList()
    {
        return timeZoneList.Value;
    }
}

以上两种方式都能确保列表仅被初始化一次，且初始化过程不会被多线程干扰，彻底解决元素不全或返回null的问题。

内容的提问来源于stack exchange，提问作者Sheethal