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: