Замінити кілька рядкових елементів на C #

Чи є кращий спосіб зробити це ...

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

Я розширив клас рядків, щоб звести його до одного завдання, але чи є швидший спосіб?

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клас. З вашою реалізацією кожна операція генерує копію рядка, що за певних обставин може погіршити продуктивність. Рядки є незмінними об'єктами, тому кожна операція просто повертає змінену копію.

Якщо ви очікуєте, що цей метод буде активно викликати множину 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();
    }
}

це буде ефективніше:

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

Якщо ви просто шукаєте гарне рішення і вам не потрібно економити кілька наносекунд, як щодо цукру 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 {

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

Також додайте пропозицію New In Town щодо StringBuilder ...

Є одне, що може бути оптимізовано у запропонованих рішеннях. Наявність багатьох дзвінків Replace()змушує код робити кілька проходів по одному рядку. За дуже довгих рядків рішення можуть бути повільними через помилки в обсязі кеш-пам'яті процесора. Можливо, слід подумати про заміну кількох рядків за один прохід .

Іншим варіантом використання 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);
}

Я роблю щось подібне, але в моєму випадку я роблю серіалізацію / десеріалізацію, тому мені потрібно мати можливість рухатися в обох напрямках. Я вважаю, що використання рядка [] [] працює майже ідентично словнику, включаючи ініціалізацію, але ви також можете піти в інший бік, повернувши замінники до їх початкових значень, чого словник насправді не налаштований робити.

Редагувати: Ви можете використовувати Dictionary<Key,List<Values>>для того, щоб отримати той самий результат, що і рядок [] []

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