3

Можно ли написать как то компактнее данный фрагмент кода или нет?

string newString = oldString.Replace(" ","")
                                .Replace(" ", "")
                                .Replace("-", "")
                                .Replace(".", "")
                                .Replace(",", "")
                                .Replace("(", "")
                                .Replace(")", "")
                                .Replace("{", "")
                                .Replace("}", "")
                                .Replace("[", "")
                                .Replace("]", "")
                                .Replace("!", "")
                                .Replace("@", "")
                                .Replace("#", "")
                                .Replace("$", "")
                                .Replace("%", "")
                                .Replace("^", "")
                                .Replace("*", "")
                                .Replace("+", "")
                                .Replace("=", "")
                                .Replace(":", "")
                                .Replace(";", "")
                                .Replace("/", "")
                                .Replace(@"\", "");
0

3 ответа 3

7

Можно воспользоваться Regex.Replace - в указанной входной строке заменяет все строки, соответствующие шаблону регулярного выражения, указанной строкой замены.

  string input = "This @#$[ываыв] %is text with some test data %$ dsffs !@#$%";
  string pattern = "[-.,(){}@#$%^&*!+=:;/\\[\\]]+";
  string replacement = "";
  Regex rgx = new Regex(pattern);
  string result = rgx.Replace(input, replacement);

  // This @#$[ываыв] %is text with some test data %$ dsffs !@#$%
  Console.WriteLine("Original String: {0}", input);
  // This ываыв is text with some test data  dsffs
  Console.WriteLine("Replacement String: {0}", result);  

http://ideone.com/CVYgJG

Я не знаток регулярок, но как пример, думаю, это поможет найти правильный путь.

2
  • Всем спасибо! Но я сделал еще короче string newString = Regex.Replace(oldString, "[-.,(){}@#$%^<>?&*!+=:;/\[\]]+", "");
    – ArtemiyMVC
    Commented 24 авг. 2016 в 12:00
  • @ArtemiyMVC я специально разделил, чтоб видно было что откуда берется. Но вы, конечно же, вправе сделать всё как душе угодно) Главное регулярку составить правильно Commented 24 авг. 2016 в 12:06
2

Как вам и советовали выше, воспользуйтесь регуляркой. Проблема вашего кода не только в его многословности и дублировании, но ещё и в том, что при каждом вызове Replace строка создаётся заново (в силу неизменяемости строк в C#). Поэтому если строки достаточно большие и/или этот код будет вызываться много раз, вы рискуете получить существенный оверхед на многократное пересоздание вашей строки. Не берусь судить, что там происходит в недрах Regex.Replace, но наверняка его алгоритм подходит к этому вопросу существенно экономнее, нежели многократный вызов Replace с пересозданием строки.

1
  • Да, Regex.Replace внутри пользуется StringBuilder'ом.
    – andreycha
    Commented 24 авг. 2016 в 12:40
1

Если количество вызовов и/или размеры строк достаточно велики, то рекомендую следующий метод, который существенно (в несколько раз) выигрывает по скорострельности у Regex:

private static string ClearString(string s)
{
    StringBuilder sb = new StringBuilder(s.Length);
    foreach (char c in s)
    {
        switch (c)
        {
            case ' ':
            case '\t':
            case '-':
            case '+':
            case '=':
            case '.':
            case ',':
            case ':':
            case ';':
            case '(':
            case ')':
            case '{':
            case '}':
            case '[':
            case ']':
            case '!':
            case '@':
            case '#':
            case '$':
            case '%':
            case '^':
            case '&':
            case '*':
            case '\\':
            case '/':
                break;
            default:
                sb.Append(c);
                break;
        }
    }
    return sb.ToString();
}

Компактностью тут, конечно, и не пахнет, но эффективность достойная.

2
  • Привели бы примеры цифр для полноты картины. (Хотя автора, по-моему, волнует исключительно компактность кода.)
    – andreycha
    Commented 24 авг. 2016 в 14:30
  • @andreycha Да я с радостью. Выполняется примерно в 5 раз быстрее, чем заранее скомпилированный Regex.
    – Dmitry D.
    Commented 24 авг. 2016 в 15:00

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.