我有一个复杂的基于任务/锁的混乱来执行'长'数据操作,我正在尝试用async / await替换它。我是异步等待的新手,所以我担心我会犯一些重大错误。
为简化起见,我的UI有几个依赖于相同数据的页面。现在,我只需要获取一次这个数据。所以我缓存它,进一步调用只是从缓存“CachedDataObjects”中获取它而不是每次都进行长时间的调用。
像这样(半伪代码):
private Dictionary<Guid,List<Data>> CachedDataObjects;
public async Task<List<Data>> GetData(Guid ID)
{
List<Data> data = null;
//see if we have any cached
CachedDataObjects.TryGetValue(ID, out data);
if (data == null)
{
if (ConnectedToServer)
{
data = new List<Data>();
await Task.Run(() =>
{
try
{
//long data call
data = Service.GetKPI(ID);
}
catch (Exception e)
{
//deal with errors (passes an action to do if resolved)
PromptForConnection(new Task(async () => { data = await GetData(ID); }), e);
}
});
}
CachedDataObjects.Add(ID, data);
}
return data;
}
但是,根据异步调用的性质,这两个页面触发时会调用此方法。
因此,有一个例外 - 带有ID的项目已经添加到字典中。即使我修补了这个问题,基本问题仍然存在。 Data对象是不同的版本,我正在做两个网络调用,我应该只有一个等等。
以前,我通过将整个方法封装在一个锁定语句中来“破解”一个解决方案 - 因此只允许对它进行一次调用。我的所有数据加载都是在后台工作人员中完成的,第一次完成了调用,一旦完成,其他人就会解锁以执行快速抓取。
但是我不能在异步方法中使用lock,并且解决方案无论如何都感觉不好。
是否有任何方法可以使用异步方法“等待”其他异步调用来完成?
答案 0 :(得分:3)
你的问题是你在将任务添加到字典之前await
完成了任务。在这种情况下,您需要将任务添加到字典中,以便调用此方法的下一页将获得相同的任务:
public Task<List<Data>> GetData(Guid ID)
{
Task<List<Data>> task = null;
CachedDataObjects.TryGetValue(ID, out task);
if (task == null)
{
if (ConnectedToServer)
{
task = Task.Run(() =>
{
try
{
//long data call
return Service.GetKPI(ID);
}
catch (Exception e)
{
//deal with errors
}
});
}
DataObjects.Add(ID, task);
}
return task;
}
这将缓存任务。但是,如果//deal with errors
传播异常,那么这也会缓存该异常。
为避免这种情况,您可以使用更复杂的代码,也可以采用我的AsyncLazy<T>
type:
private readonly ConcurrentDictionary<Guid, AsyncLazy<List<Data>>> CachedDataObjects;
public Task<List<Data>> GetData(Guid ID)
{
var lazy = CachedDataObjects.GetOrAdd(ID, id =>
new AsyncLazy<List<Data>>(() => Task.Run(() =>
{
try
{
return Service.GetKPI(ID);
}
catch (Exception e)
{
//deal with errors
throw;
}
}, AsyncLazyFlags.RetryOnFailure | AsyncLazyFlags.ExecuteOnCallingThread)));
return lazy.Task;
}