ValueTask仅可被等待一次是什么意思？如何修改代码实现多次调用？

首先明确：你完全可以多次调用并await GetMoney方法。

“ValueTask仅可被等待一次”的规则，约束的是同一个ValueTask实例不能被多次await，而非同一个方法不能被多次调用。你当前的代码中，每次调用GetMoney都会生成并返回一个全新的ValueTask实例，所以多次await这些不同的实例是完全合规的，不会有问题。

什么时候会触发违规场景？ #

只有当你把单个ValueTask实例存储下来，然后多次await这个实例时，才会违反规则，比如：

// 错误示例：同一个ValueTask实例被多次await
var moneyTask = creditCardService.GetMoney(3);
Console.WriteLine(await moneyTask);
Console.WriteLine(await moneyTask); // 这里会引发问题

你的代码可以优化的点 #

虽然当前代码的多次调用没问题，但存在一些可以提升性能和正确性的优化：

1. 避免不必要的async状态机
   当缓存命中时，你的代码因为标记了async，会额外生成异步状态机，浪费性能。可以改成非async方法，同步场景直接返回ValueTask.FromResult，异步逻辑抽成局部async方法：

   public class CreditCardService
   {
       private readonly Dictionary<int, int> _cache = new Dictionary<int, int>()
       {
           { 1, 500 },
           { 2, 600 }
       };
       // 线程安全的随机数生成器（避免多线程下重复随机数）
       private static readonly Random _rnd = new Random();
   
       public ValueTask<int> GetMoney(int userId)
       {
           if (_cache.TryGetValue(userId, out var money))
           {
               return ValueTask.FromResult(money);
           }
           // 异步逻辑交给局部async方法处理
           return GetMoneyAndCacheAsync(userId);
       }
   
       private async ValueTask<int> GetMoneyAndCacheAsync(int userId)
       {
           // 模拟异步操作（比如数据库查询）
           await Task.Delay(100);
           var money = _rnd.Next(100);
           // 注意：Dictionary非线程安全，如果是多线程场景请改用ConcurrentDictionary
           _cache[userId] = money;
           return money;
       }
   }
   

2. 移除多余的await Task.FromResult
   你的原代码中return await Task.FromResult(money)完全多余，Task.FromResult是同步生成的Task，直接返回money即可，async方法会自动将其包装为ValueTask。

3. 线程安全优化（可选）
   如果你的服务会被多线程调用，Dictionary不是线程安全的，建议替换为ConcurrentDictionary，避免并发访问时的异常：

   private readonly ConcurrentDictionary<int, int> _cache = new ConcurrentDictionary<int, int>()
   {
       { 1, 500 },
       { 2, 600 }
   };
   

   对应的缓存写入逻辑也需要调整为ConcurrentDictionary的方法：

   var money = _rnd.Next(100);
   _cache.TryAdd(userId, money);
   

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