分析一个错误使用MemoryCache导致的BUG-阿里云开发者社区

分析一个错误使用MemoryCache导致的BUG

2022-01-14 122

版权

本文内容由阿里云实名注册用户自发贡献，版权归原作者所有，阿里云开发者社区不拥有其著作权，亦不承担相应法律责任。具体规则请查看《阿里云开发者社区用户服务协议》和《阿里云开发者社区知识产权保护指引》。如果您发现本社区中有涉嫌抄袭的内容，填写侵权投诉表单进行举报，一经查实，本社区将立刻删除涉嫌侵权内容。

简介： 分析一个错误使用MemoryCache导致的BUG

这个Bug是我在项目中发现的，原因是MemoryCache使用不当造成了一个不小的Bug，虽说这个Bug很大部分人都知道，但是我觉得还是分享出来，记录一下。废话不多说，我们来看一下出错的代码（代码已经经过脱敏处理）

await using var services = 
  new ServiceCollection()
    .AddMemoryCache()
    .BuildServiceProvider();
GetValidValues(5).Dump();
GetValidValues(8).Dump();
List<int> GetValidValues(int valueInt)
{
    var memoryCache = services.GetRequiredService<IMemoryCache>();
    var vs= memoryCache.GetOrCreate("t1", entry =>
    {
        return Enumerable.Range(1, 10).ToList();
    });
    vs.RemoveAll(x => x > valueInt);
    return vs;
}

代码中Dump是扩展方法，它是把list内的元素输出出来，具体实现代码如下：

public static void Dump(this List<int> vs)
{
    string v= string.Join("--", vs);
    Console.WriteLine(v);
}

好了，来想一下上面的输出结果会是什么吧，期望的结果应该是每次都输出小于等于输入的值，实际是什么样的呢？实际输出结果如下：

从上图中第二次输出的结果是不是和你想的不一样呢，之所以出现上面问题是因为MemoryCache对象是直接保存在内存中的，缓存不变化时每次都返回同一个对象，如果发生了修改那么再次获取就是修改后的内容。因此正确做法是返回一个新对象而不是修改原来的对象，一个修改方法如下：

List<int> GetValidValues(int valueInt)
{
    var memoryCache = services.GetRequiredService<IMemoryCache>();
    var vs= memoryCache.GetOrCreate("t1", entry =>
    {
        return Enumerable.Range(1, 10).ToList();
    });
    return vs.Where(v => v <= valueInt).ToList();
}