1

Кто сможет мне объяснить почему данная функция иногда мне возвращает null:

public async Task<T> PostData<T>(string apiFunction, object data) where T : new()
{
    try
    {
        using (HttpResponseMessage response = await Client.PostAsJsonAsync(apiFunction, data))
        {
            if (response.IsSuccessStatusCode)
            {
                var jsonData = await response.Content.ReadAsStringAsync();
                return JsonConvert.DeserializeObject<T>(jsonData);
            }
            else return new T();
        }
    }
    catch (HttpRequestException)
    {
        return new T();
    }
    catch (JsonReaderException)
    {
        return new T();
    }
}

Используется в сервисе, который объявлен как синглтон. Вызывается в контроле. Например:

[HttpGet]
public async Task<IActionResult> Connect()
{
    ResponseServer responseServer = await _request.PostData<ResponseServer>("connection", userEntity);
    if (responseServer == null) responseServer = new ResponseServer();
}

Приходится делать дополнительную проверку на null.

6
  • Вы не все ловите эксепшены. Не хватает catch(Exception) в самом конце блока catch'ей
    – A K
    24 янв 2019 в 15:24
  • Спасибо, попробую. Я подозревал об этом, но думал если бы была эксепшен отличная от тех, которые ловлю, то при отладке она должна была бы отображаться. Но никак не ожидал, что это будет только NullReferenceException.
    – Emigrant
    24 янв 2019 в 15:35
  • По моему по этому: else return new T();
    – NewView
    24 янв 2019 в 16:14
  • @NewView Не, точно не в этом. new T не будет равно null никогда. Тут что-то другое. Я подозреваю, что и я неправильный ответ дал
    – A K
    24 янв 2019 в 17:48
  • 1
    @NewView При указанных топикстартером условиях (ResponseServer + where T : new()) на практике точно не будет - а за какие-то теоретические конструкции и я не поручусь, мало ли какие извраты компилятор пропустит и в рантайме возможны.
    – A K
    24 янв 2019 в 18:29

1 ответ 1

2

Кажется, я созрел до ответа.

Давайте во-первых, покажем, что дело не в else return new T(), не в отсутствии ещё одного блока catch как я предположил в комментариях ну и заодно накидаем классы, на которых будем моделировать разные ситуации.

Берём linqpad, пишем:

void Main()
{
    var myClass = new MyClass();
    var test = myClass.Test<Data>();
    if(test == null)
    {
        Console.WriteLine("null");
    }
    else
    {
        Console.WriteLine("not null");
    }
}

// Define other methods and classes here

public class MyClass
{
    public T Test<T>()
        where T : new()
    {
        return new T();
    }
}

public class Data
{
    public int Id { get; set; }
}

Запускаем, смотрим — не null.

Попытка два. Попробуем выкинуть исключение, причём типа, который мы не обрабатываем:

public T Test<T>()
    where T : new()
{
    try
    {
        throw new NotImplementedException();
    }
    catch (ArgumentException)
    {
        return new T();
    }
}

Всё логично — мы получаем необработанное исключение. Тоже не вариант.

Подключим Json и попробуем посмотреть что будет в результате у нас Null:

public T Test<T>()
    where T : new()
{
    try {
        string data = null;
        return JsonConvert.DeserializeObject<T>(data);
    }
    catch (ArgumentException)
    {
        return new T();
    }
}

Снова не null, так как Json.Net выкинет ArgumentException, которое отловится на catch.

Но вот если мы попробуем указать, что data — пустая строка, то мы получим null:

public T Test<T>()
    where T : new()
{
    string data = string.Empty;
    return JsonConvert.DeserializeObject<T>(data);
}

Вот оно и найдено ваше условие. Вы получаете из http client'а пустую строку, а в результате Json.Net выкидывает вам Null.

2
  • Аплодисменты, одназначно!
    – NewView
    24 янв 2019 в 18:23
  • Спасибо за помощь!
    – Emigrant
    25 янв 2019 в 4:08

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.