私はTheWhiteRabbitの答えに同意しますが、HttpClientを使用して多くの呼び出しがある場合、コードは私の考えでは少し繰り返しのようです。
答えを少し改善するには2つの方法があると思います。
クライアントを作成するヘルパークラスを作成します。
public static class ClientHelper
{
// Basic auth
public static HttpClient GetClient(string username,string password)
{
var authValue = new AuthenticationHeaderValue("Basic", Convert.ToBase64String(Encoding.UTF8.GetBytes($"{username}:{password}")));
var client = new HttpClient(){
DefaultRequestHeaders = { Authorization = authValue}
//Set some other client defaults like timeout / BaseAddress
};
return client;
}
// Auth with bearer token
public static HttpClient GetClient(string token)
{
var authValue = new AuthenticationHeaderValue("Bearer", token);
var client = new HttpClient(){
DefaultRequestHeaders = { Authorization = authValue}
//Set some other client defaults like timeout / BaseAddress
};
return client;
}
}
使用法:
using(var client = ClientHelper.GetClient(username,password))
{
//Perform some http call
}
using(var client = ClientHelper.GetClient(token))
{
//Perform some http call
}
拡張メソッドを作成します。
美容賞を受賞していませんが、うまくいきます:)
public static class HttpClientExtentions
{
public static AuthenticationHeaderValue ToAuthHeaderValue(this string username, string password)
{
return new AuthenticationHeaderValue("Basic",
Convert.ToBase64String(
System.Text.Encoding.ASCII.GetBytes(
$"{username}:{password}")));
}
}
使用法:
using (var client = new HttpClient())
{
client.DefaultRequestHeaders.Authorization = _username.ToAuthHeaderValue(_password);
}
繰り返しになりますが、上記の2つのオプションを使用すると、クライアントのステートメントの繰り返しがやや少なくなります。複数のhttp呼び出しを行う場合はHttpClientを再利用することがベストプラクティスであることを覚えておいてください。ただし、これはこの質問の範囲から少し外れていると思います。