Thread.Sleep 或 Task.Delay 在具有同步和异步调用方的同步方法中
Thread.Sleep or Task.Delay in a synchronous method with both a sync and async caller
我有一个同步方法:
public void DoStuff() {
DoThings();
GraphClient.SetExtendedProperty(user, propertyName, value); // this method occasionally throws an exception
DoOtherThings();
}
Call3rdPartyMethod
使用 Azure Ad Graph Client API 进行 REST API 调用,如果我尝试在 Active Directory 上设置扩展 属性 的值并且它会抛出异常没有找到。这通常发生在将新用户添加到目录并且扩展属性功能尚未扩展用户架构之前我想设置值(似乎需要几秒钟)。
我用我自己的包装器替换了 SetExtendedProperty 调用,该包装器包含忙等待循环中的调用,因此:
public void TrySetProperty(GraphObject user, string propertyName, string value)
{
var exceptions = new List<Exception>();
for (int retry = 0; retry < 5; retry++)
{
try
{
if (retry > 0)
Thread.Sleep(1000);
GraphClient.SetExtendedProperty(user, propertyName, value);
}
catch (Exception ex)
{
exceptions.Add(ex);
}
}
throw new AggregateException(exceptions);
}
}
问题是我希望能够从同步和异步方法调用 TrySetProperty
:
public void DoStuff() {
DoThings();
TrySetProperty(user, propertyName, value);
DoOtherThings();
}
public Task DoOtherStuffAsync() {
await DoAsyncThings();
TrySetProperty(user, propertyName, value);
await DoOtherAsyncThings();
}
我无法将 SetExtendedProperty 更改为异步,而且我担心如果我从异步方法调用它,我不应该使用 Thread.Sleep - 而是 Task.Delay() .谁能给点建议?
我会推荐:
- 如果可能制作一个fully-asynchronous版本。 REST API 天生就是异步的,但一些客户端库仍然过时(即只有同步方法)。
- 如果可能,只公开异步版本(因为操作自然是异步的)。如果您必须支持同步 API(例如,为了向后兼容),则使用 boolean argument hack in my article on brownfield async.
- 使用 Polly 重试逻辑。
假设您有 GraphClient.SetExtendedProperty
的 fully-asynchronous 版本,那么您的代码可能如下所示:
private static readonly Policy syncPolicy = Policy.Handle<Exception>().WaitAndRetry(5, _ => TimeSpan.FromSeconds(1));
private static readonly Policy asyncPolicy = Policy.Handle<Exception>().WaitAndRetryAsync(5, _ => TimeSpan.FromSeconds(1));
private static async Task TrySetProperty(GraphObject user, string propertyName, string value, bool sync)
{
if (sync)
syncPolicy.Execute(() => GraphClient.SetExtendedProperty(user, propertyName, value));
else
await asyncPolicy.ExecuteAsync(() => GraphClient.SetExtendedPropertyAsync(user, propertyName, value));
}
public static Task TrySetPropertyAsync(GraphObject user, string propertyName, string value) =>
TrySetProperty(user, propertyName, value, sync: false);
public static void TrySetProperty(GraphObject user, string propertyName, string value) =>
TrySetProperty(user, propertyName, value, sync: true).GetAwaiter().GetResult();
如果您的 TrySetProperty
逻辑真的那么简单(即,它实际上只调用 GraphClient
上的一个方法),那么您可以取消布尔参数 hack 以获得更简单的代码:
private static readonly Policy syncPolicy = Policy.Handle<Exception>().WaitAndRetry(5, _ => TimeSpan.FromSeconds(1));
private static readonly Policy asyncPolicy = Policy.Handle<Exception>().WaitAndRetryAsync(5, _ => TimeSpan.FromSeconds(1));
public static async Task TrySetPropertyAsync(GraphObject user, string propertyName, string value)
{
await asyncPolicy.ExecuteAsync(() => GraphClient.SetExtendedPropertyAsync(user, propertyName, value));
}
public static void TrySetProperty(GraphObject user, string propertyName, string value)
{
syncPolicy.Execute(() => GraphClient.SetExtendedProperty(user, propertyName, value));
}
我有一个同步方法:
public void DoStuff() {
DoThings();
GraphClient.SetExtendedProperty(user, propertyName, value); // this method occasionally throws an exception
DoOtherThings();
}
Call3rdPartyMethod
使用 Azure Ad Graph Client API 进行 REST API 调用,如果我尝试在 Active Directory 上设置扩展 属性 的值并且它会抛出异常没有找到。这通常发生在将新用户添加到目录并且扩展属性功能尚未扩展用户架构之前我想设置值(似乎需要几秒钟)。
我用我自己的包装器替换了 SetExtendedProperty 调用,该包装器包含忙等待循环中的调用,因此:
public void TrySetProperty(GraphObject user, string propertyName, string value)
{
var exceptions = new List<Exception>();
for (int retry = 0; retry < 5; retry++)
{
try
{
if (retry > 0)
Thread.Sleep(1000);
GraphClient.SetExtendedProperty(user, propertyName, value);
}
catch (Exception ex)
{
exceptions.Add(ex);
}
}
throw new AggregateException(exceptions);
}
}
问题是我希望能够从同步和异步方法调用 TrySetProperty
:
public void DoStuff() {
DoThings();
TrySetProperty(user, propertyName, value);
DoOtherThings();
}
public Task DoOtherStuffAsync() {
await DoAsyncThings();
TrySetProperty(user, propertyName, value);
await DoOtherAsyncThings();
}
我无法将 SetExtendedProperty 更改为异步,而且我担心如果我从异步方法调用它,我不应该使用 Thread.Sleep - 而是 Task.Delay() .谁能给点建议?
我会推荐:
- 如果可能制作一个fully-asynchronous版本。 REST API 天生就是异步的,但一些客户端库仍然过时(即只有同步方法)。
- 如果可能,只公开异步版本(因为操作自然是异步的)。如果您必须支持同步 API(例如,为了向后兼容),则使用 boolean argument hack in my article on brownfield async.
- 使用 Polly 重试逻辑。
假设您有 GraphClient.SetExtendedProperty
的 fully-asynchronous 版本,那么您的代码可能如下所示:
private static readonly Policy syncPolicy = Policy.Handle<Exception>().WaitAndRetry(5, _ => TimeSpan.FromSeconds(1));
private static readonly Policy asyncPolicy = Policy.Handle<Exception>().WaitAndRetryAsync(5, _ => TimeSpan.FromSeconds(1));
private static async Task TrySetProperty(GraphObject user, string propertyName, string value, bool sync)
{
if (sync)
syncPolicy.Execute(() => GraphClient.SetExtendedProperty(user, propertyName, value));
else
await asyncPolicy.ExecuteAsync(() => GraphClient.SetExtendedPropertyAsync(user, propertyName, value));
}
public static Task TrySetPropertyAsync(GraphObject user, string propertyName, string value) =>
TrySetProperty(user, propertyName, value, sync: false);
public static void TrySetProperty(GraphObject user, string propertyName, string value) =>
TrySetProperty(user, propertyName, value, sync: true).GetAwaiter().GetResult();
如果您的 TrySetProperty
逻辑真的那么简单(即,它实际上只调用 GraphClient
上的一个方法),那么您可以取消布尔参数 hack 以获得更简单的代码:
private static readonly Policy syncPolicy = Policy.Handle<Exception>().WaitAndRetry(5, _ => TimeSpan.FromSeconds(1));
private static readonly Policy asyncPolicy = Policy.Handle<Exception>().WaitAndRetryAsync(5, _ => TimeSpan.FromSeconds(1));
public static async Task TrySetPropertyAsync(GraphObject user, string propertyName, string value)
{
await asyncPolicy.ExecuteAsync(() => GraphClient.SetExtendedPropertyAsync(user, propertyName, value));
}
public static void TrySetProperty(GraphObject user, string propertyName, string value)
{
syncPolicy.Execute(() => GraphClient.SetExtendedProperty(user, propertyName, value));
}