Vermeiden Sie doppelten Code in SortedDictionary C#

Question

So habe ich zwei, wenn stament Wörterbuch Wert zu behandeln.Wenn die Methode ist POST Ich muss nur die status Params zum Wörterbuch hinzufügen.Alle anderen Parameter sind die gleichen.

var stringBuilder = new StringBuilder();      
var dictionary = new SortedDictionary<string, string>(); 
if (method == "GET") 
{ 
    stringBuilder.Append("GET&"); 
    stringBuilder.Append(Uri.EscapeDataString(url)); 
    stringBuilder.Append("&"); 
    dictionary = new SortedDictionary<string, string> 
    { 
     { "oauth_version" , oauthVersion }, 
     { "oauth_consumer_key", consumerKey }, 
     { "oauth_nonce" , oauthNonce }, 
     { "oauth_signature_method" , oauthSignatureMethod }, 
     { "oauth_timestamp" , oauthTimeStamp }, 
     { "oauth_token" , accessToken }, 
    }; 
} 

if (method == "POST") 
{ 
    stringBuilder.AppendFormat("POST&{0}&", Uri.EscapeDataString(url)); 
    dictionary = new SortedDictionary<string, string> 
    { 
     { "oauth_version" , oauthVersion }, 
     { "oauth_consumer_key", consumerKey }, 
     { "oauth_nonce" , oauthNonce }, 
     { "oauth_signature_method" , oauthSignatureMethod }, 
     { "oauth_timestamp" , oauthTimeStamp }, 
     { "oauth_token" , accessToken }, 
     { "status" , status } 
    }; 
} 

Gibt es sowieso Duplikat in meinem code.Any Vorschlag praktisch wäre zu vermeiden.

Mit freundlichen Grüßen.

Answer 1

Ich sehe nicht viel Beziehung zwischen Ihrer StringBuilder Instanzvariable und dem Wörterbuch, das Sie konstruieren, also unter der Annahme der Verdoppelung des Wörterbuchcodes, kann es aus diesen if Bedingungen herausgenommen werden, und dann könnten Sie mit dem umgehen Rest StringBuilder Beispiel wie folgt aus:

var stringBuilder = new StringBuilder(); 
stringBuilder.AppendFormat("{0}&{1}&", method, Uri.EscapeDataString(url)); 

var dictionary = new SortedDictionary<string, string> 
{ 
    { "oauth_version", oauthVersion }, 
    { "oauth_consumer_key", consumerKey }, 
    { "oauth_nonce", oauthNonce }, 
    { "oauth_signature_method", oauthSignatureMethod }, 
    { "oauth_timestamp", oauthTimeStamp }, 
    { "oauth_token", accessToken }, 
}; 

if (method == "POST") 
{ 
    // Only add the status parameter if the method is POST 
    dictionary["status"] = status; 
} 

Also im Grunde nach dieser Code ausgeführt, sowohl die stringBuilder und dictionary Instanzvariablen wie in Ihrer Frage gezeigt werden die erwarteten Werte.

Dies bedeutet, dass Ihr Wörterbuch aussieht, als ob es Abfragezeichenfolgenparameterwerte enthält. Wenn dies der Fall ist, werden Sie definitiv diese URLs richtig codieren wollen, bevor Sie überhaupt über das HTTP-Protokoll senden: