13

Ao ler esta resposta, a uma pergunta relacionada com a utilização do using, fiquei curioso em relação à afirmação seguinte:

Na verdade todo mundo usa HttpClient errado, eu mesmo já usei e não sabia de tudo, porque eu confiei que a documentação tinha tudo o que eu precisava, mas aprendi a lição.

A forma como eu tenho visto HttpClient ser usado é dentro de um bloco using.

using (var http = new HttpClient())
{
    //
}

Não sabendo ao certo se era a isto que o AR se referia, perguntei-lhe e, através de um comentário, ele confirmou que sim.

Não tendo sido o comentário suficientemente esclarecedor, fica então a pergunta:

O HttpClient deve, ou não deve, ser utilizado dentro de um bloco using?

16

2 Respostas 2

7

O grande problema é justamente que ele é enganoso. Por ter implementado a interface IDisposble todo mundo acha que deveria usar o using. E não sei se na primeira implementação desta classe não precisava, o que eu sei é que a documentação inicialmente não falava nada disto, ou porque ensinava errado mesmo ou porque antes era assim (não pode confiar 100% em documentação, algo próximo disto pode, mas ela nunca dá todos detalhes).

Faz algum sentido ter essa interface nessa classe porque de fato pode ter motivos para em algum momento o recurso vinculado a este objeto seja descartado. Porém o mais comum no uso correto é que ele dure a aplicação toda.

Pensa bem no objeto, começando pelo nome. As pessoas pensam nele como se fosse uma conexão mas ele é um cliente. Quantos clientes HTTP você precisa na sua aplicação? Um, não é? Pra que outros? Ele é um mecanismo, é como uma file system, não tem porque ter mais que um. O fato de você ter um só não quer dizer que só pode fazer uma requisição, ele estando lá pode requisitar sempre que quiser, desde que tenha acesso ao objeto.

Por alguma razão não há algo estático nele como a file system, provavelmente porque o cliente é pouco pesado e criar sempre será algo caro para maioria das aplicações, ao mesmo tempo que criar toda vez que precisa é custoso demais (que é o erro que "todo mundo" faz).

Na forma correta é muito comum que este objeto seja colocado em um campo estático, assim cria uma vez, possivelmente na primeira utilização efetiva, e não precisa mais, aí é só usar.

O exemplo da outra resposta não é muito bom porque foi escrito de qualquer jeito. Eu imagino que a intenção ali era que o campo fosse estático. A não ser em um exemplo muito simples não parece adequado criar na classe principal, deveria ter uma classe com responsabilidade de cuidar disto, mas aí não é um erro de uso do HttpClient e sim de organização do código.

Eu aprendi que se a classe tiver IDisposable sempre deve usar using

Não é bem assim, deve em métodos, ou seja, quando o objeto deve ser criado localmente. E esse deve é relativo, também tem casos para não usar, ainda que provavelmente tem jeito melhor de fazer.

Existem casos que você pode criar o objeto ali, retorná-lo e aí o objeto deve reter o recurso vinculado a ele, então não é ali que o usará o using, mas provavelmente usará em outro lugar para que a liberação seja feita. Por sorte você não vê isso por aí, mas é algo que pode existir e até ser útil em algum cenário. Algo assim:

void Metodo() {
    ...
    using (var arquivo = CriaRecurso()) { ... }
    ...
}

FileStream CriaRecurso() {
    ...
    var file = new FileStream("abc.txt", FileMode.Create)
    ...
    return file;
}

Além disto pode ser que o objeto com o recurso pode ser guardado dentro de uma classe através de um campo e não uma variável local. Neste caso não tem como usar o using o que você terá que fazer é em algum momento chamar o Dispose() desse objeto. Isto pode ser através de um método destrutor/finalizador que todo objeto tem ou através de um Dispose() criada nesta classe, portanto se você usar um um objeto que seja IDisposable em um campo praticamente obriga essa classe implementar a IDisposable, é viral. Aí um objeto dessa sua classe provavelmente será colocado em um using.

É mais raro fazer isto, mas já vi vários cenários que precisa. Se as pessoas fizessem assim muito código seria otimizado quando há ganhos.

Em alguns casos deve ser até mesmo em um campo estático o que pode dispensar essa classe passar por alguma destruição.

O tempo de vida necessário para o objeto determinará onde e se terá o using.

Não pode usar nunca o using nessa classe?

Pode, se você sabe que sua aplicação só usará o cliente HTTP ali uma única vez ou uma quantidade muito limitada, então melhor não fazer. E não estou dizendo que dá na mesma. É pior fazer isto se for mais de uma vez, só faz sentido se for uma vez mesmo, em todo tempo de execução. De fato em um cenário assim é melhor com o using, mas é raro ter um cenário assim.

O que acontece de errado se fizer assim?

Ele vai criando sockets no sistema operacional. Não está claro porque mas ele parece não liberar imediatamente e por isso uma hora pode ficar sem sockets livres. Ou mesmo que isto não ocorra cada socket novo gasta recursos na máquina.

Há uma página que mostra isso com propriedade:

using static System.Console;
using System.Net.Http;

public class Program {
    public static async Task Main() {
        WriteLine("Starting connections");
        for (int i = 0; i<10; i++) {
            using (var client = new HttpClient()) {
                var result = await client.GetAsync("http://aspnetmonsters.com");
                WriteLine(result.StatusCode);
            }
        }
        WriteLine("Connections done");
    }
}

Isto funciona e produz isto:

C:\code\socket> dotnet run
Project socket (.NETCoreApp,Version=v1.0) will be compiled because inputs were modified
Compiling socket for .NETCoreApp,Version=v1.0

Compilation succeeded.
    0 Warning(s)
    0 Error(s)

Time elapsed 00:00:01.2501667


Starting connections
OK
OK
OK
OK
OK
OK
OK
OK
OK
OK
Connections done

Porém investigando no sistema operacional veja o que aconteceu:

C:\code\socket>NETSTAT.EXE
...
  Proto  Local Address          Foreign Address        State
  TCP    10.211.55.6:12050      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12051      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12053      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12054      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12055      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12056      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12057      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12058      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12059      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12060      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12061      waws-prod-bay-017:http  TIME_WAIT
  TCP    10.211.55.6:12062      waws-prod-bay-017:http  TIME_WAIT
  TCP    127.0.0.1:1695         SIMONTIMMS742B:1696    ESTABLISHED
...

Ruim, né? Imagina isso acontecendo milhares de vezes.

using static System.Console;
using System.Net.Http;

public class Program {
    private static HttpClient Client = new HttpClient();
    public static async Task Main() {
        WriteLine("Starting connections");
        for (int i = 0; i<10; i++) {
            var result = await client.GetAsync("http://aspnetmonsters.com");
            WriteLine(result.StatusCode);
        }
        WriteLine("Connections done");
    }
}

Coloquei no GitHub para referência futura.

O resultado é o mesmo mas agora o sistema operacional reporta só um socket.

Note que isto está ok porque é um exemplo só para testes, o objeto e recurso será descartado no fim da aplicação que é quando desejamos, mas há casos que precise de um controle mais sofisticado. Não ache que sempre essa forma é a mais adequada para seu caso, pro isso é necessário aprender fazer as coisas de forma ampla e não apenas decorar receitas de bolo.

Conclusão.

Precisamos entender tudo sobre o que vamos usar. Muitas vezes não basta ler a documentação oficial, tem que procurar outros documentos, questionar. Programar é difícil, mesmo tentando fazer tudo certo ainda pode dar algo errado.

O problema é que no passado a documentação não falava que o tempo de vida provavelmente deveria ser o mesmo da aplicação toda.

Algumas leituras recomendadas:

9
  • 1
    "Eu aprendi que se a classe tiver IDisposable sempre deve usar using" - Ter um professor que nos ensine é bom, mas há algumas vantagens em não ter. Não ter professor proporcionou que eu recolhesse o conhecimento de várias fontes. Dessa recolha "eu aprendi que se a classe tiver IDisposable deve chamar Dispose()". Seja caso de ter ou não professor, o importante, quando aprendemos, é termos sempre uma atitude critica e "desconfiada", no bom sentido, em relação ao que lemos e nos dizem. (...)
    – ramaral
    Commented 22/11/2019 às 14:04
  • (...) Apesar disso, foi apanhado pelo fenómeno do "o que é repetido vezes e vezes sem conta é verdadeiro/correcto", eu era um dos que usava mal o HttpClient.
    – ramaral
    Commented 22/11/2019 às 14:04
  • É como o Random só que este não implementa IDisposable e engana menos :)
    – Maniero
    Commented 22/11/2019 às 14:22
  • Jugo que se está a referir a Random retornando sempre o mesmo número
    – ramaral
    Commented 22/11/2019 às 14:39
  • 1
    @ramaral não vou afirmar que mata na hora que a aplicação encerrar, eu até acho que tem uma espécie de evento que o fim da aplicação manda encerrar isso (coisa interna do sistema operacional), e se não faz deveria fazer, mas mesmo neste caso ainda teria um encerramento por timeout ou algo assim, não tem como ficar pendurado sem um dono. Se ficasse seria bug e não by design.
    – Maniero
    Commented 22/11/2019 às 18:00
6
HttpClient client = new HttpClient();

    static async Task Main()
    {
      try   
      {
         HttpResponseMessage response = await client.GetAsync("http://www.contoso.com/");
         response.EnsureSuccessStatusCode();
         string responseBody = await response.Content.ReadAsStringAsync();

         Console.WriteLine(responseBody);
      }  
      catch(HttpRequestException e)
      {
         Console.WriteLine("\nException Caught!");  
         Console.WriteLine("Message :{0} ",e.Message);
      }
    }

A instância da classe HttpClient atua como uma sessão para enviar solicitações HTTP. Uma instância de HttpClient é uma coleção de configurações aplicadas a todas as solicitações executadas por essa instância. Além disso, cada instância de HttpClient usa seu próprio pool de conexões, isolando suas solicitações de solicitações executadas por outras instâncias de HttpClient.

De acordo com a documentação encontrada na Microsoft, quando existe uma única instância do HttpClient pra toda a aplicação o ideal é não usar o "Using{}", caso contrário é importante fazer uso do "Using{}" quando existir mais de uma instância do HttpClient.

3
  • 1
    Achei curioso as resposta ter poucos votos, ach oque só meu e do AP, mas o final talvez seja culpado disto. Dá um pouco a impressão que você pode usar vários vários objetos e aí ir dispensando com using, e até tecnicamente pode mesmo, mas é errado. Essa é a única ressalva que faço.
    – Maniero
    Commented 21/11/2019 às 18:32
  • 2
    @Maniero estava preste a comentar algo do género. O segundo voto não é meu e a razão é essa mesmo. Acho que a resposta não responde completamente à pergunta. É necessário explicar o porquê de usar apenas uma instância ou as consequências de repetidamente usar o HttpClient dentro de um bloco using,
    – ramaral
    Commented 21/11/2019 às 18:47
  • 1
    @ramaral no fim achei que realmente deixa a desejar, mas não posso tirar o voto :(
    – Maniero
    Commented 22/11/2019 às 12:14

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 .