現在有一個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