ASP.NET MVC缓存SelectListItems被DropDownListFor修改问题求助

这个问题我之前也碰到过，咱们一步步来拆解和解决：

问题根源：是的，DropDownListFor确实修改了缓存的实例 #

因为SelectListItem是引用类型，你缓存的是它的集合实例本身。当DropDownListFor渲染下拉框时，会遍历这个集合，找到和personnelSelected匹配的项，并直接修改它的Selected属性为true——这个修改会直接作用在缓存中的内存对象上，因为它们是同一个实例。

解决方案：避免修改缓存实例的几种实用方式 #

1. 从缓存获取后返回集合副本（最直接快速）

每次调用GetAllPersonnelCached()时，不要直接返回缓存的集合，而是返回全新的SelectListItem实例副本，这样页面修改的是副本，不会影响缓存里的原数据：

public IEnumerable<SelectListItem> GetAllPersonnelCached() {
    var cachedItems = CacheHelper.GetFromCache<IEnumerable<SelectListItem>>("mykey");
    
    // 如果缓存为空，先初始化并缓存原始集合
    if (cachedItems == null) {
        var originalItems = GetPersonnelSourceData() // 从DB/其他数据源获取原始数据
            .Select(p => new SelectListItem {
                Text = p.Name,
                Value = p.Id.ToString()
            }).ToList();
        CacheHelper.SaveToCache("mykey", originalItems, 240);
        cachedItems = originalItems;
    }

    // 返回新的实例副本，避免修改缓存对象
    return cachedItems.Select(item => new SelectListItem {
        Text = item.Text,
        Value = item.Value
    });
}

2. 缓存原始业务数据而非SelectListItem（更推荐，符合分层设计）

SelectListItem是UI层专用对象，并不适合直接缓存。更好的做法是缓存业务层的原始实体（比如Personnel），每次需要下拉列表时再转换成SelectListItem：

// 缓存原始业务实体
public IEnumerable<Personnel> GetAllPersonnelCached() {
    var cachedData = CacheHelper.GetFromCache<IEnumerable<Personnel>>("personnelData");
    if (cachedData == null) {
        cachedData = GetPersonnelFromDb().ToList(); // 从数据库获取原始数据
        CacheHelper.SaveToCache("personnelData", cachedData, 240);
    }
    return cachedData;
}

// 在控制器中转换为UI层需要的SelectListItem
personnelList = GetAllPersonnelCached().Select(p => new SelectListItem {
    Text = p.Name,
    Value = p.Id.ToString()
});

这种方式完全规避了缓存对象被修改的风险，同时也符合分层架构的设计原则，让缓存的数据更通用。

3. 封装只读版下拉项（局限性较大，不优先推荐）

如果一定要缓存类似SelectListItem的结构，可以自己封装一个只读类，避免外部修改属性：

public class ReadOnlySelectItem {
    public string Text { get; }
    public string Value { get; }

    public ReadOnlySelectItem(string text, string value) {
        Text = text;
        Value = value;
    }
}

缓存这个只读类的集合，每次需要时再转换成SelectListItem。不过这种方式需要额外维护一个类，性价比不如前两种方案。

总结 #

最推荐第二种方案：缓存业务原始数据，每次使用时生成全新的SelectListItem实例。既解决了缓存被修改的问题，也让代码结构更清晰合理。

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