Как сделать, если короче?

Question

Я понял, что при использовании Regex я могу ввести значение | для поиска многих значений. Например:

Regex sabrina = new Regex("Rihanna|rihanna|Sabrina|sabrina"); 

У меня есть строка, которую я хочу, чтобы сравнить с другими значениями, поэтому я использую if так:

if (rihanna == "Rihanna" || rihanna == "sabrina") 

Мой вопрос, если это возможно сделать if короче? Я знаю, что этот код не работает, но то, что я искал что-то вроде этого:

if (rihanna == "Rihanna|sabrina") 

Answer 1

Если вы ищете более легкую поддержание потенциально длинного списка кандидатов в строку, почему бы не использовать оператор switch? Технически это будет «длиннее» по характеру, но если вам нужно модифицировать вещи по дороге или обеспечить некоторую тонкую вторичную обработку, это может сделать многое намного проще.

Answer 2

Вы можете использовать List<string> наряду с методом Enumerable.Contains.

Пример:

var values = new List<string> { "Rihanna", "rihanna", "Sabrina", "sabrina" }; 

if (values.Contains(rihanna)) 
{ 
    // etc. 
} 

if заявление теперь намного более кратким, и вы можете добавить больше элементов в список быстро и легко. Если вы часто выполняете эту операцию, вам следует переместить объявление списка в переменную-член в содержащем классе, чтобы каждый раз, когда вызывается этот метод, вы не наносите накладные расходы на инициализацию списка.

Даже лучшим решением было бы использовать Enumerable.Any и String.Equals выполнить сравнение без учета регистра:

if (values.Any(x => String.Equals(rihanna, x, StringComparison.OrdinalIgnoreCase)) 
{ 
    // your code here 
} 

Хотя if заявление теперь уже решение является гораздо более надежным.

Answer 3

решение одной линии:

if("Rihanna|rihanna|Sabrina|sabrina".Split(new[] {'|'}).Contains(rihanna)) 

или без использования String.Split метода:

if ((new[] { "Rihanna", "rihanna", "Sabrina", "sabrina" }).Contains(rihanna)) 

Если такая проверка выполняется более чем один раз, вы должны сохранить string[] массив возможных значений в переменную:

var names = new[] { "Rihanna", "rihanna", "Sabrina", "sabrina" }; 

if(names.Contains(rihanna)) 

Answer 4

Вы можете использовать свой собственный Regex и использовать его в качестве одной линии ..

if (Regex.Match(rihanna, "Rihanna|rihanna|Sabrina|sabrina").Success) 

Или сделать это случай-insensative как это ..

if (Regex.Match(rihanna, "rihanna|sabrina", RegexOptions.IgnoreCase).Success){ 

Answer 5

Игнорирование корпус, который выглядит как-то вы собираетесь за, и использовать String.Compare:

if(String.Compare(rihanna, "sabrina", StringComparison.OrdinalIgnoreCase) == 0 || 
    String.Compare(rihanna, "rihana", StringComparison.OrdinalIgnoreCase) == 0) { 
    // do stuff 
} 

Answer 6

В качестве альтернативы выше ответов, возможно, вы могли бы рассмотреть возможность использования метода расширения:

public static bool EqualsAny(this string input, params string[] values) 
{ 
    return values.Any(v => String.Equals(input, v, StringComparison.OrdinalIgnoreCase)); 
} 

с использованием как:

if (rihanna.EqualsAny("Rihanna", "Sabrina")) 

Это должно дать довольно читаемые/компактные/короткие if сравнения для вас, особенно в сравнении с некоторыми другими альтернативами здесь.

Проверьте, есть ли у вас OrdinalIgnoreCase или нет или вы можете добавить любые перегрузки. Мог бы даже реализовать весь трубопровод/расщепление, где вы проходите в «Rihanna Sabrina».

EDIT: Возможно, более критично, он централизует , как вы сравниваете, а не дублируете код везде.

Answer 7

Dictionary или HashSet (возможно, с регистрацией нечувствительных к регистру имен) - это другой подход, который даст вам лучшую производительность, если у вас есть несколько элементов и часто повторяются проверки.

var listToCheck = new Dictionary<string, string>( 
    StringComparer.CurrentCultureIgnoreCase) { { "rihanna", null}}; 

if (listToCheck.ContainsKey("Rihanna")) ....