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

喵叔

发布于 2021-11-24 09:40:14

3990

发布于 2021-11-24 09:40:14

文章被收录于专栏：喵叔's 专栏

这个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();
}

修改后的输出结果如下：