Когда создавать новую задачу

Question

Я узнаю о параллелизме задач в C# .NET 4.5, и я немного смущен о example. Вот код, который я не понимаю:

public static Task<string> DownloadStringAsync(string address) 
{ 
    // First try to retrieve the content from cache. 
    string content; 
    if (cachedDownloads.TryGetValue(address, out content)) 
    { 
    return Task.FromResult<string>(content); 
    } 

    // If the result was not in the cache, download the 
    // string and add it to the cache. 
    return Task.Run(async() => // why create a new task here? 
    { 
    content = await new WebClient().DownloadStringTaskAsync(address); 
    cachedDownloads.TryAdd(address, content); 
    return content; 
    }); 
} 

В частности, я не понимаю, почему они оборачивают DownloadStringTaskAsync() в другой задаче. Разве не DownloadStringTaskAsync() уже работает в своей собственной теме?

Вот как я бы закодировал:

public static async Task<string> DownloadStringAsync(string address) 
{ 
    // First try to retrieve the content from cache. 
    string content; 
    if (cachedDownloads.TryGetValue(address, out content)) 
    { 
     return content; 
    } 

    // If the result was not in the cache, download the 
    // string and add it to the cache. 
    content = await new WebClient().DownloadStringTaskAsync(address); 
    cachedDownloads.TryAdd(address, content); 
    return content; 
} 

В чем разница между ними? Какая из них лучше?

Answer 1

Ну, пример показывает, как использовать Task.FromResult, который ваш второй код не использует. Тем не менее, я не согласен с использованием Task.Run в этом примере.

Я бы написать это, я:

public static Task<string> DownloadStringAsync(string address) 
{ 
    // First try to retrieve the content from cache. 
    string content; 
    if (cachedDownloads.TryGetValue(address, out content)) 
    { 
    return Task.FromResult(content); 
    } 

    // If the result was not in the cache, download the 
    // string and add it to the cache. 
    return DownloadAndCacheStringAsync(address); 
} 

private static async Task<string> DownloadAndCacheStringAsync(string address) 
{ 
    var content = await new WebClient().DownloadStringTaskAsync(address); 
    cachedDownloads.TryAdd(address, content); 
    return content; 
} 

отметить также, что в данном примере используется датированную WebClient, который должен быть заменен HttpClient в новом коде.

В целом, это похоже на плохой пример ИМО.