Skip to main content
adicionou 90 caracteres ao conteúdo; adicionou 27 caracteres ao conteúdo; removeu 1 caractere do conteúdo
Fonte Link
Francisco
  • 7,6mil
  • 4
  • 28
  • 58

Tenho a seguinte implementação de MemoryCache:

public Task<News[]> GetCandidateNewsAsync(string candidate)
{
    return _cache.GetOrCreateAsync(candidate, async factory =>// _cache é um IMemoryCache
    {
        var candidateUri = _candidateUris[candidate];

        if (string.IsNullOrEmpty(candidateUri))
            return null; //retorno nulo aqui (erro)

        var candidateNews = await _candidateNewsClient.GetCandidateNewsAsync(1, candidateUri);

        return candidateNews?.News;
    });
}

Rodei um analisador de código em cima do meu projeto e ele reportou o seguinte:

Do not return null from this method, instead return 'Task.FromResult(null)', 'Task.CompletedTask' or 'Task.Delay(0)'

Só que eu não entendi direito, qual o problema de retornar nulo nessa ocasião? Qual o sentido retornar um delay de 0ms como ele recomenda?

Tenho a seguinte implementação de MemoryCache:

_cache.GetOrCreateAsync(candidate, async factory =>
    {
        var candidateUri = _candidateUris[candidate];

        if (string.IsNullOrEmpty(candidateUri))
            return null; //retorno nulo aqui (erro)

        var candidateNews = await _candidateNewsClient.GetCandidateNewsAsync(1, candidateUri);

        return candidateNews?.News;
    });

Rodei um analisador de código em cima do meu projeto e ele reportou o seguinte:

Do not return null from this method, instead return 'Task.FromResult(null)', 'Task.CompletedTask' or 'Task.Delay(0)'

Só que eu não entendi direito, qual o problema de retornar nulo nessa ocasião? Qual o sentido retornar um delay de 0ms como ele recomenda?

Tenho a seguinte implementação de MemoryCache:

public Task<News[]> GetCandidateNewsAsync(string candidate)
{
    return _cache.GetOrCreateAsync(candidate, async factory =>// _cache é um IMemoryCache
    {
        var candidateUri = _candidateUris[candidate];

        if (string.IsNullOrEmpty(candidateUri))
            return null; //retorno nulo aqui (erro)

        var candidateNews = await _candidateNewsClient.GetCandidateNewsAsync(1, candidateUri);

        return candidateNews?.News;
    });
}

Rodei um analisador de código em cima do meu projeto e ele reportou o seguinte:

Do not return null from this method, instead return 'Task.FromResult(null)', 'Task.CompletedTask' or 'Task.Delay(0)'

Só que eu não entendi direito, qual o problema de retornar nulo nessa ocasião? Qual o sentido retornar um delay de 0ms como ele recomenda?

Fonte Link
Francisco
  • 7,6mil
  • 4
  • 28
  • 58

É errado retornar nulo em uma Task?

Tenho a seguinte implementação de MemoryCache:

_cache.GetOrCreateAsync(candidate, async factory =>
    {
        var candidateUri = _candidateUris[candidate];

        if (string.IsNullOrEmpty(candidateUri))
            return null; //retorno nulo aqui (erro)

        var candidateNews = await _candidateNewsClient.GetCandidateNewsAsync(1, candidateUri);

        return candidateNews?.News;
    });

Rodei um analisador de código em cima do meu projeto e ele reportou o seguinte:

Do not return null from this method, instead return 'Task.FromResult(null)', 'Task.CompletedTask' or 'Task.Delay(0)'

Só que eu não entendi direito, qual o problema de retornar nulo nessa ocasião? Qual o sentido retornar um delay de 0ms como ele recomenda?