C # 'da Birden Çok Dize Öğesini Değiştirin

Bunu yapmanın daha iyi bir yolu var mı...

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

String sınıfını tek bir işe indirgemek için genişlettim ama daha hızlı bir yol var mı?

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

Sırf eğlence için (ve yorumlardaki argümanları durdurmak için) aşağıdaki çeşitli örnekleri karşılaştıran bir özet getirdim.

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

Normal ifade seçeneği korkunç bir puan alır; sözlük seçeneği en hızlı şekilde gelir; stringbuilder'ın uzun soluklu versiyonu, kısa elden biraz daha hızlıdır.

Daha hızlı - hayır. Daha etkili - evet, StringBuildersınıfı kullanacaksanız . Uygulamanızla her işlem, koşullar altında performansı bozabilecek bir dizenin bir kopyasını oluşturur. Dizeler değişmez nesnelerdir, bu nedenle her işlem yalnızca değiştirilmiş bir kopya döndürür.

Bu yöntemin Stringsönemli uzunlukta birden çok sayıda etkin olarak çağrılmasını bekliyorsanız , uygulamasını StringBuildersınıfa "geçirmek" daha iyi olabilir . Bununla birlikte, herhangi bir değişiklik doğrudan bu örnekte gerçekleştirilir, böylece gereksiz kopyalama işlemlerinden kurtulursunuz.

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

Güzel bir çözüm peşindeyseniz ve birkaç nanosaniye tasarruf etmeniz gerekmiyorsa, LINQ şekerine ne dersiniz?

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

bu daha verimli olacak:

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

Belki biraz daha okunabilir mi?

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

Ayrıca New In Town'ın StringBuilder hakkındaki önerisini ekleyin ...

Önerilen çözümlerde optimize edilebilecek bir şey var. Çok sayıda çağrıya sahip olmak Replace(), kodun aynı dizede birden çok geçiş yapmasını sağlar. Çok uzun dizelerde, CPU önbellek kapasitesi kayıpları nedeniyle çözümler yavaş olabilir. Tek geçişte birden çok dizeyi değiştirmeyi düşünmek gerekebilir .

Linq kullanan başka bir seçenek de

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

Benzer bir şey yapıyorum, ancak benim durumumda serileştirme / serileştirme yapıyorum, bu yüzden her iki yöne de gidebilmem gerekiyor. Bir [] [] dizesi kullanmanın sözlükle neredeyse aynı şekilde çalıştığını görüyorum, buna başlatma dahil, ama siz de diğer yöne gidebilir, ikameleri orijinal değerlerine geri döndürebilirsiniz, bu sözlüğün gerçekten yapamayacağı bir şeydir.

Düzenle: Dictionary<Key,List<Values>>[] [] dizesi ile aynı sonucu elde etmek için kullanabilirsiniz

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