Подтвердить что ты не робот

HttpClient - обработка совокупных исключений

Привет, я использую HttpClient, подобный этому:

public static Task<string> AsyncStringRequest(string url, string contentType)
{
    try
    {
        var client = new HttpClient();
        client.DefaultRequestHeaders.Accept.Add(new System.Net.Http.Headers.MediaTypeWithQualityHeaderValue(contentType));

        return client.GetStringAsync(url).ContinueWith(task => {
            return task.Result;
        });
    }
    catch (AggregateException ex)
    {
        throw ex;
    }
    catch (WebException ex)
    {
        throw ex;
    }       
    catch (Exception ex)
    {
        throw ex;
    }
}

Но у меня возникают трудности с исключениями. Я добавил дополнительные блоки catch только для того, чтобы попытаться выполнить шаг, но ни одна из точек останова не попадает ни в один из блоков catch. Я понимаю, что с помощью Task исключение может возникать в другом потоке, чем вызывающий, поэтому исключение обернуто в контейнер агрегата, но я не уверен, что лучший способ справиться с этими исключениями.

Например, я делаю запрос к веб-службе и задаю недопустимый параметр в запросе, и генерируется исключение. Я хочу, чтобы я мог уловить общие исключения и посмотреть на innerexceptions, чтобы понять, почему запрос не сработал и вернуть дружеское сообщение.

Итак, мой вопрос: какой лучший способ поймать эти агрегированные исключения и справиться с ними?

4b9b3361

Ответ 1

Исключение выбрано task.Result:

var client = new HttpClient();
client.DefaultRequestHeaders.Accept.Add(...);
return client.GetStringAsync(url).ContinueWith(task =>
{
    try
    {
        return task.Result;
    }
    catch (AggregateException ex)
    {
        throw ex;
    }
    catch (WebException ex)
    {
        throw ex;
    }       
    catch (Exception ex)
    {
        throw ex;
    }
});

Лучше: проверьте, не выполнила ли задача faulted доступ к task.Result:

var client = new HttpClient();
client.DefaultRequestHeaders.Accept.Add(...);
return client.GetStringAsync(url).ContinueWith(task =>
{
    if (task.IsFaulted)
    {
        var ex = task.Exception;
    }
    else if (task.IsCancelled)
    {
    }
    else
    {
        return task.Result;
    }
});

Если вы на самом деле ничего не делаете в ContinueWith, вы можете просто опустить его:

var client = new HttpClient();
client.DefaultRequestHeaders.Accept.Add(...);
return client.GetStringAsync(url);