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

これを行うためのより良い方法はありますか...

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

正規表現オプションのスコアはひどいです。辞書オプションが最も速く表示されます。stringbuilder replaceの長巻きバージョンは、速記よりもわずかに高速です。

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

このメソッドStringsがかなりの長さの倍数でアクティブに呼び出されることが予想される場合は、その実装をStringBuilderクラスに「移行」する方がよい場合があります。これを使用すると、変更はそのインスタンスで直接実行されるため、不要なコピー操作を省くことができます。

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();
    }
}

あなたが単にかなりの解決策を求めていて、数ナノ秒を節約する必要がないのなら、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));

これはより効率的になります：

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();
    }
}

多分もう少し読みやすいですか？

    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に関するNewInTownの提案も追加してください...

提案されたソリューションで最適化できることが1つあります。を何度も呼び出すとReplace()、同じ文字列に対して複数のパスを実行するコードが作成されます。文字列が非常に長い場合、CPUキャッシュ容量が失われるため、ソリューションが遅くなる可能性があります。1回のパスで複数の文字列を置き換えることを検討する必要があるかもしれません。

linqを使用する別のオプションは

[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);
}

私は似たようなことをしていますが、私の場合はシリアル化/逆シリアル化を行っているので、両方向に進むことができる必要があります。string [] []の使用は、初期化を含め、辞書とほぼ同じように機能しますが、逆の方向に進んで、置換を元の値に戻すこともできます。これは、辞書では実際には設定されていません。

編集：Dictionary<Key,List<Values>>string [] []と同じ結果を得るために使用できます

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]);
}