55
\$\begingroup\$

I've got this method and I realize that an instance of HttpClass is going to be created for each call to it. While working seemingly OK, I'm considering moving it out and placing it as a private property accessible to each call to the method. There might be other methods making use of it in the future as well. I have no information on how often these calls will occur. (There's a try/catch too but it's omitted for brevity's sake.)

[OperationContract]
[WebGet(UriTemplate = "Poof"]
public async Task<String> GetPoof()
{
  String url = BaseUrl + "poofy";
  using (HttpClient client = new HttpClient())
  {
    client.DefaultRequestHeaders.TryAddWithoutValidation(
      "Authorization", GetAuthorizationSchema());
    HttpResponseMessage message = await client.GetAsync(url);
    return await message.Content.ReadAsStringAsync();
  }
}

What I'm thinking of is something like this:

private HttpClient _Client
private HttpClient Client { get { return _Client ?? GetNewClient(); } }

[OperationContract]
[WebGet(UriTemplate = "Poof"]
public async Task<String> GetPoof()
{
  String url = BaseUrl + "poofy";
  HttpResponseMessage message = await client.GetAsync(url);
  return await message.Content.ReadAsStringAsync();
}

My worry is, though, that the different calls utilizing the same client instance will somehow collide and call huge issues later on, especially if the number of calls per time unit raises beyond a critical level. Should I be worried?

\$\endgroup\$
0

2 Answers 2

59
\$\begingroup\$

You don't need multiple instances. In fact, HttpClient is designed specifically to persist and be used for multiple requests (see: HttpClient.DefaultRequestHeaders).

There's already a lot written about this so I'll redirect you to those resources:

\$\endgroup\$
1
2
\$\begingroup\$

I suggest

private HttpClient Client { get { return _Client  = _Client ?? GetNewClient(); } }

Instead of

private HttpClient Client { get { return _Client ?? GetNewClient(); } }
\$\endgroup\$
2
  • \$\begingroup\$ This spells out how to exploit HttpClient safe to use concurrently - as opposed to trying. \$\endgroup\$
    – greybeard
    Commented Mar 7, 2018 at 5:39
  • \$\begingroup\$ Actually, I prefer to use an static HttpClient variable and the initialization inside the static constructor of the main class \$\endgroup\$ Commented Mar 7, 2018 at 10:41

Not the answer you're looking for? Browse other questions tagged or ask your own question.