C#重載方法泛型參數的一點小問題

現在有一個HttpHelper類

 public async Task<string> ExecuteAsync<H>(H header) where H : class, new()
 {
     var client = new RestClient(_url);
     if (!string.IsNullOrWhiteSpace(_accountSid) && !string.IsNullOrWhiteSpace(_secretKey))
     {
         client.Authenticator = new HttpBasicAuthenticator(_accountSid, _secretKey);
     }
     var headerString = JsonConvert.SerializeObject(header);
     Dictionary<string, string> headerParameters = JsonConvert.DeserializeObject<Dictionary<string, string>>(headerString);
     RestRequest request = new RestRequest();
     if (headerParameters != null)
     {
         foreach (var p in headerParameters)
         {
             request.AddHeader(p.Key, p.Value);
         }
     }
     var response = await client.ExecuteAsync(request, Method.GET);
     return response.Content;

 }

 public async Task<string> ExecuteAsync<T>(T data, bool isFrom = true) where T : class, new()
 {
     if (data == null) return "";
     var client = new RestClient(_url);
     var dataString = JsonConvert.SerializeObject(data);
     RestRequest request = new RestRequest();
     if (isFrom)
     {
         Dictionary<string, string> parameters = JsonConvert.DeserializeObject<Dictionary<string, string>>(dataString);
         foreach (var p in parameters)
         {
             request.AddParameter(p.Key, p.Value);
         }
     }
     else
     {
         //request.AddHeader("Accept", "application/json");
         request.AddJsonBody(dataString);
     }
     var response = await client.ExecuteAsync(request, Method.POST);
     return response.Content;
 }

這種寫法是完全符合規則,但是調用的時候會有點小問題。

第二個方法調用,我們一般會忽略第二個入參,按照定義第二個入參我們設置了默認值,目的就是讓在調用的時候省點代碼。

ExecuteAsync(request)

但是現在因爲有另一個重載泛型方法出現,導致調用了第一個方法,是不是有點意外?

所以是不是含泛型參數的方法,不建議使用有默認值的入參?靠北了這是。

那就不重載了重新命個名ExecuteAsyncGet

發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章