C＃で複数の文字列要素を置き換える

Question

これを行う良い方法がありますか？

MyString.Trim().Replace("&", "and").Replace(",", "").Replace(" ", " ") 
     .Replace(" ", "-").Replace("'", "").Replace("/", "").ToLower(); 

文字列クラスを拡張して1つのジョブにしましたが、より速い方法がありますか？

public static class StringExtension 
{ 
    public static string clean(this string s) 
    { 
     return s.Replace("&", "and").Replace(",", "").Replace(" ", " ") 
       .Replace(" ", "-").Replace("'", "").Replace(".", "") 
       .Replace("eacute;", "é").ToLower(); 
    } 
} 

---

ちょっと面白くて（そしてコメントの引数を止めるために） 私は下のさまざまな例をベンチマークして大事にしました。

https://gist.github.com/ChrisMcKee/5937656

正規表現オプションはひどくスコアを付けます。辞書オプションが最も速くなります。ストリングビルダの置き換えられた長いバージョンは、短い手よりもやや速いです。

Answer 1

Quicker - no。より効果的 - はい、あなたがStringBuilderクラスを使用する場合。実装では、各操作によって文字列のコピーが生成され、状況によってはパフォーマンスが低下する可能性があります。文字列は不変オブジェクトなので、各操作は変更されたコピーを返します。

Stringsの長さがこのメソッドを積極的に呼び出すと予想される場合は、その実装をStringBuilderクラスに「移行する」方がよい場合があります。これにより、そのインスタンスに対して直接変更が行われるため、不要なコピー操作が不要になります。

public static class StringExtension 
{ 
    public static string clean(this string s) 
    { 
     return new StringBuilder(s) 
       .Replace("&", "and") 
       .Replace(",", "") 
       .Replace(" ", " ") 
       .Replace(" ", "-") 
       .Replace("'", "") 
       .Replace(".", "") 
       .Replace("eacute;", "é") 
       .ToString() 
       .ToLower(); 
    } 
} 

Answer 2

これは、より効率的になりますか？

public static class StringExtension { 

     private static Dictionary<string, string> _replacements = new Dictionary<string, string>(); 

     static StringExtension() { 
      _replacements["&"] = "and"; 
      _replacements[","] = ""; 
      _replacements[" "] = " "; 
      // etc... 
     } 

     public static string clean(this string s) { 
      foreach (string to_replace in _replacements.Keys) { 
       s = s.Replace(to_replace, _replacements[to_replace]); 
      } 
      return s; 
     } 
    } 

またStringBuilderの程度タウンの提案では、新規を追加...

Answer 3

たぶんもう少し読み：

public static class StringExtention 
{ 
    public static string clean(this string s) 
    { 
     StringBuilder sb = new StringBuilder (s); 

     sb.Replace("&", "and"); 
     sb.Replace(",", ""); 
     sb.Replace(" ", " "); 
     sb.Replace(" ", "-"); 
     sb.Replace("'", ""); 
     sb.Replace(".", ""); 
     sb.Replace("eacute;", "é"); 

     return sb.ToString().ToLower(); 
    } 
} 

Answer 4

私は似た何かをやっているが、私の場合、私は行くことができるようにする必要があるので、シリアライズ/デシリアライズをやっています双方向。私は文字列[] []を使用して初期化を含む辞書とほぼ同じように動作することを知っていますが、あなたは元の値に代用品を返すこともできます。

編集：あなたは、文字列と同じ結果を得るためにDictionary<Key,List<Values>>を使用することができます[] []

Answer 5

あなたはかなり解決した後、単純であり、数ナノ秒を保存する必要がない場合は、どのようにいくつかのLINQ糖について？

var input = "test1test2test3"; 
var replacements = new Dictionary<string, string> { { "1", "*" }, { "2", "_" }, { "3", "&" } }; 

var output = replacements.Aggregate(input, (current, replacement) => current.Replace(replacement.Key, replacement.Value)); 

Answer 6

提案されている解決策では、最適化されるものが1つあります。 Replace()を多く呼び出すと、同じ文字列に対して複数のパスを実行するコードになります。非常に長い文字列を使用すると、CPUのキャッシュ容量ミスのためにソリューションが遅くなることがあります。考慮する必要がありますreplacing multiple strings in a single pass。LINQを使用して

Answer 7

string input = "it's worth a lot of money, if you can find a buyer."; 
for (dynamic i = 0, repl = new string[,] { { "'", "''" }, { "money", "$" }, { "find", "locate" } }; i < repl.Length/2; i++) { 
    input = input.Replace(repl[i, 0], repl[i, 1]); 
} 

Answer 8

別のオプションあなたは辞書のバージョンがそれよりも速くなっている私がいる疑いがある代替品のすべてをやっていないように見えるあなたのベンチマークに持っているものに基づいて

[TestMethod] 
public void Test() 
{ 
    var input = "it's worth a lot of money, if you can find a buyer."; 
    var expected = "its worth a lot of money if you can find a buyer"; 
    var removeList = new string[] { ".", ",", "'" }; 
    var result = input; 

    removeList.ToList().ForEach(o => result = result.Replace(o, string.Empty)); 

    Assert.AreEqual(expected, result); 
}