Optimieren meiner Methode

Question

Ich versuche, eine Hilfsmethode zu erstellen, um Seriendruck-ähnliche Funktionen für eine Vorlagendatei auszuführen. Da Strings unveränderlich sind, bin ich mir nicht sicher, ob ich es richtig geschrieben habe - kann jemand einen Blick darauf werfen und mir Feedback geben?

public static string LoadTemplateFile(string fileName, 
             NameValueCollection mergeFields) 
{ 
    string result = System.IO.File.ReadAllText(fileName); 

    if (mergeFields != null) 
    { 
     for (int index = 0; index < mergeFields.Count; index++) 
     { 
      result = result.Replace(mergeFields.Keys[index], 
            mergeFields[index]); 
     } 
    } 

    return result; 
} 

Answer 1

Es sieht aus wie Sie

versuchen

1. eine Datei Lesen von der Festplatte
2. Führen Sie eine Suche/ersetzen, basierend auf einem bereitgestellten Name/Wert-Karte

Wenn das der ist Fall dann ja das wird gut funktionieren.

Die einzige echte Rückmeldung, die ich habe, ist, dass abhängig von der Anzahl der Ersatz Name/Wert-Paare, Sie eine Menge temporäre Zeichenfolgen erstellen werden. Dies ist wahrscheinlich für kleine Dateien gut, aber sobald Sie beginnen, relativ große Dateien in Ihre Anwendung zu laden, können Sie einen beträchtlichen Unterschied feststellen.

Ein besserer Ansatz wäre, einen StringBuilder zu verwenden und die Replace-Aufrufe für dieses Objekt auszuführen. Es würde die unnötige Erzeugung von temporären Strings reduzieren.

Answer 2

Sie würden wahrscheinlich besser einen StringBuilder anstelle einer Zeichenfolge verwenden.

public static string LoadTemplateFile(
    string fileName, NameValueCollection mergeFields) 
{  
    System.Text.StringBuilder result = new System.Text.StringBuilder(
     System.IO.File.ReadAllText(fileName)); 

    if (mergeFields != null) 
    { 
     for (int index = 0; index < mergeFields.Count; index++) 
     { 
      result.Replace(mergeFields.Keys[index], 
            mergeFields[index]); 
     } 
    } 

    return result.ToString(); 
} 

Answer 3

Verwenden Sie StringBuilder anstelle von Zeichenfolge. das ist mein einziger Rat, sein Weg schneller.