Question

我创建了一个使用async / await返回对象的函数。我想使函数通用，以便它可以返回我传入的任何对象。除了返回的对象之外，代码是样板文件。我希望能够调用GetAsync并让它返回正确的对象

public Patron getPatronById(string barcode)
{
    string uri = "patrons/find?barcode=" + barcode;
    Patron Patron =  GetAsync(uri).Result;
    return Patron;
}

private async Task<Patron> GetAsync(string uri)
{
    var client = GetHttpClient(uri);
    var content = await client.GetStringAsync(uri);
    JavaScriptSerializer ser = new JavaScriptSerializer();
    Patron Patron = ser.Deserialize<Patron>(content);
    return Patron;
}

Answer 1

通用方法怎么样？

private async Task<T> GetAsync<T>(string uri)
{
    var client = GetHttpClient(uri);
    var content = await client.GetStringAsync(uri);
    var serializer = new JavaScriptSerializer();
    var t = serializer.Deserialize<T>(content);
    return t;
}

通常，您应该将此方法放入另一个类并将其设为public，以便它可以被不同类中的方法使用。

关于调用此方法的方式，您可以尝试以下方法：

 // I capitalized the first letter of the method, 
 // since this is a very common convention in .NET
 public Patron GetPatronById(string barcode)
 {
     string uri = "patrons/find?barcode=" + barcode;
     var Patron =  GetAsync<Patron>(uri).Result;
     return Patron;
 }

注意：在上面的代码片段中，我假设您没有将GetAsync移到另一个类中。如果你移动它，那么你必须稍作改动。

<强>更新

我没注意你的意思是什么意思。我是否还需要让GetPatronById成为一个任务函数 - 就像Yuval在下面做的那样？

我的意思是这样的：

// The name of the class may be not the most suitable in this case.
public class Repo
{
    public static async Task<T> GetAsync<T>(string uri)
    {
        var client = GetHttpClient(uri);
        var content = await client.GetStringAsync(uri);
        var serializer = new JavaScriptSerializer();
        var t = serializer.Deserialize<T>(content);
        return t;
    }
}

public Patron GetPatronById(string barcode)
{
     string uri = "patrons/find?barcode=" + barcode;
     var Patron =  Repo.GetAsync<Patron>(uri).Result;
     return Patron;
}

Answer 2

通用可以通过以下方式轻松完成：

private async Task<T> GetAsync(string uri)
{
    var client = GetHttpClient(uri);
    var content = await client.GetStringAsync(uri);
    return JsonConvert.DeserializeObject<T>(content);
}

注意事项：

JavaScriptSerializer已被弃用多年，请避免使用它。请改为Json.NET。

此：

Patron Patron =  GetAsync(uri).Result;

很危险并且可能导致潜在的死锁，尤其是在Web API中。你需要一直“异步”＃34;：

public Task<Patron> GetPatronByIdAsync(string barcode)
{
   string uri = $"patrons/find?barcode={barcode}";
   return GetAsync<Patron>(uri);
}

await只有您最高级别的调用者需要Task。可能是一些控制器动作：

public async Task SomeAction()
{
     await GetPatronByIdAsync("hello");
}

创建通用异步任务功能

2 个答案: