4

У меня есть множественные проверки в if-е, они содержат в себе повторяющийся код, можно ли этого избежать? В дальнейшем символов для проверки будет больше.

2
  • 1
    Собери в хэшсет и проверяй на contains
    – Adrug
    16 фев 2019 в 11:37
  • Смотря, что Вы хотите добиться. Если Вы хотите исключить, что в Вашем коде есть спец символы - то Regex Вам в помощь
    – WinnerIT
    17 фев 2019 в 15:30

2 ответа 2

5

Делается это так. Добавляем нужные символы в HashSet<char> (кстати, в него нельзя добавить одинаковые символы ни случайно, ни специально). Далее везде в коде проверяешь есть ли символ в HashSet<char>. Почитай про HashSet<T>, работает быстрее, чем обход массива, к примеру, на каждой итерации.

using System;
using System.Collections.Generic;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            var charSet = new HashSet<char>(new[]
            {
                ' ', '-', ',', '.', '_', ':', ';', '?', '(', ')', '!'
            });

            string text = "Hi! My name is John.";
            Console.WriteLine(text);

            for (int i = 0; i < text.Length; i++)
            {
                if (charSet.Contains(text[i]))
                {
                    Console.WriteLine(i);
                }
            }

            Console.ReadKey(true);
        }
    }
}
3

Я бы сделал это при помощи String.Contains() из-за наглядности

var map = " ,._:;&()!";
for (int i = 0; i < text.Length; i++) {
    if (map.Contains(text[i])) {
        ...
    }
}

Если же Вам важна производительность, все это дело можно сложить в HashSet и искать в нем

var map = " ,._:;&()!";
var set = new HashSet<char>(map);
for (int i = 0; i < text.Length; i++) {
    if (set.Contains(text[i])) {
        ...
    }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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