私のメソッドを最適化する

Question

テンプレートファイルで差し込み印刷のような機能を実行するユーティリティメソッドを作成しようとしています。文字列は不変なので、正しく書いたかどうかは不明です。誰かが一目見てフィードバックをくれますか？

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

あなたはディスク

からファイルを読む

1. しようとしているようですが、それはだ場合は、検索/置換して、名前/値のマップ上

をもとくださいに見えますcase then yesこれはうまくいくでしょう。

私が持っている唯一のフィードバックは、置き換えられる名前/値のペアの数によっては、一時的な文字列をたくさん作成することです。これはおそらく小さなファイルでは問題ありませんが、比較的大きなファイルをアプリケーションに読み込むと、かなりの違いが見られるかもしれません。

より良いアプローチは、StringBuilderを使用し、そのオブジェクトに対してReplace呼び出しを行うことです。それは一時的な文字列の不必要な作成を減らすでしょう。

Answer 2

文字列の代わりにStringBuilderを使用する方がよいでしょう。

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

文字列の代わりにStringBuilderを使用します。それは私の唯一のアドバイスです。