-1

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?

2
  • Você poderia colocar o cabeçalho do método?
    – Maniero
    Commented 2/08/2018 às 22:07
  • Feito @Maniero ;)
    – Francisco
    Commented 2/08/2018 às 22:32

2 Respostas 2

1

Essa é uma resposta retirada do SOen:

Não é certo retornar uma Task nula. Uma Task nula é como se ela nunca tivesse sido chamada, mas ela foi.

Então, uma Task/Task<T> retornado de um método nunca dever ser null. Mas ainda assim você pode retornar o valor nulo dentro da Task.


Qual o problema de retornar nulo nessa ocasião?

No meu caso, provavelmente foi um falso alarme do meu analisador, já que eu não estava retornando a Task como nula, e sim o valor nulo dentro da Task, o que é totalmente correto.

Explicando melhor, o método GetOrCreateAsync do IMemoryCache recebe como parâmetro um tipo genérico(<T>), e uma Func<ICacheEntry, out Task<T>>, e como a minha task era assíncrona, quando eu retornava nulo, ele não estava retornando a Task como nula, e sim o valor dentro da Task, do tipo News[] como nulo.


Qual o sentido retornar um delay de 0ms como ele recomenda?

Um Task.Delay de 0ms é a mesma coisa de retornar um Task.CompletedTask, veja como é implementado no código oficial do : (linha 5407 no momento que estou escrevendo)

public static Task Delay(int millisecondsDelay, CancellationToken cancellationToken)
{
    // ...
    if (cancellationToken.IsCancellationRequested)
    {
        // return a Task created as already-Canceled
        return Task.FromCanceled(cancellationToken);
    }
    else if (millisecondsDelay == 0)
    {
        // return a Task created as already-RanToCompletion
        return Task.CompletedTask;
    }
    // ...
}
0

Do ponto de vista semântico não faz sentido uma Task retornar null. No mundo de execução assíncrona, você retornar null é 'como se você tivesse falando' pro código que uma tarefa não foi executada ou não existe, porém, no seu caso, ela existe.

O erro está te dizendo pra você fazer com que a Task retorne um valor null (TaskFromReult(null)) ou então que retorne que a tarefa foi executada (Task.CompletedTask).

Isso tudo pois é assim que se usa as tasks normalmente, faz parte das boas práticas. Quando você executa uma tarefa assíncrona, você espera que ela retorne se foi executada ou não, você não espera ela ser null. Pense que esse tipo de análise é para as outras pessoas que possam vir utilizar seu código futuramente.

5
  • Você pode colocar como o código dele ficaria?
    – Maniero
    Commented 2/08/2018 às 21:56
  • Basta trocar return null por return Task.FromResult(null) . Commented 2/08/2018 às 22:02
  • Adicionado o exemplo @Maniero Commented 2/08/2018 às 22:55
  • @Francisco isso é porque o método precisa retornar um vetor do tipo News[], não tinha essa informação anteriormente tinha? Sendo assim você precisa retornar algo do mesmo tipo. Pode tentar algo do tipo: Crie um vetor do tipo News com valor null e retorne-o. News[] nw = null retorne return Task.FromResul(nw). Isto deve funcionar. Sobre o Task.Delay(0) : segundo a referência da MS, ele executa a Task.CompletedTask quando o tempo é igual a 0ms. Pode ver mais aqui Commented 6/08/2018 às 16:04
  • Cara, fiz um pequeno programinha de teste aqui e funcionou.. Segue o link Commented 6/08/2018 às 20:03

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .