Есть ли (более чистый) способ консолидировать несколько условий в C#?

Question

Я новичок в C# и не нахожу ответы на простые вопросы. MSDN может быть очень загадочным для кого-то на моем уровне, и у меня нет специалиста в моем распоряжении для регулярных вопросов, таких как эти.

Это пример кода, я работаю с:

while (ans != "a" || ans != "b" || ans != "c" || ans != "d"); 

Есть ли способ, чтобы очистить это?

Answer 1

Вы можете поместить эти строки в списке и проверьте, если ответ будет в списке

var list = new List<string>{"a", "b", "c", "d"}; 
while (!list.Contains(ans)) { ... } 

Комментарий воспитывается хороший момент. Технически набор лучше для этой задачи, потому что он может сделать просмотр для членства очень быстро

var set = new HashSet<string>{"a", "b", "c", "d"}; 
while (!set.Contains(ans)) { ... } 

Answer 2

Используйте HashSet/словарь быстрый способ делать то, что вы хотите, особенно если список или строки длиной:

var set = new HashSet<string>() {"a", "b", "c", "d"}; 
while (!set.ContainsKey(ans)) { ... } 

Answer 3

if(new List<string>{"a", "b", "c", "d"}.Any(x=> x == ans)) 

Answer 4

В качестве предупреждения все эти ответы приносят в жертву производительность для «чистого кода». Все эти ответы должны распределять память, а затем сравнивать каждое значение в массиве с вашей переменной, которая также имеет накладные расходы (хотя использование набора/словаря на большем наборе значительно снижает накладные расходы для поиска). То, что вы опубликовали, будет самым эффективным способом сделать это. Однако, если вы решите сохранить то, что у вас есть, порядок, в котором у вас есть эти условия, может ускорить процесс. Поскольку только одно из этих условий должно быть ложным, чтобы вы не смогли, вы, как правило, хотите поставить их в порядке «с большей вероятностью, чтобы потерпеть неудачу первым».

Answer 5

«Мне нужно что-то, что постоянно оценивалось, если был дан правильный ответ (a, b, c и d - действительные ответы)».

Мне кажется, что если ответ «f» или что-то, что не указано «a», «b», «c» или «d», вам нужно, чтобы пользователь ввел другой ответ. Если это правильно, вам нужно & & вместо ||.

Я бы просто проверил введенный ответ на символы 'a' - 'd', а не как строку. Это позволяет пропустить использование списков, HashSets или Словари.

using System; 

public class Program 
{ 
    public static void Main() 
    { 
     Console.WriteLine("1 + 2 = ?"); 
     Console.WriteLine("a: 1"); 
     Console.WriteLine("b: 2"); 
     Console.WriteLine("c: 3"); 
     Console.WriteLine("d: 4"); 

     Console.Write("Select answer: "); 

     char ans; 
     do 
     { 
      // Only accept the first character, using .ToLower to ignore casing 
      ans = Console.ReadLine().ToLower()[0]; 
      if (ans < 'a' || 'd' < ans) 
      { 
       Console.WriteLine("Please select a, b, c, or d!"); 
      } 
     } while (ans < 'a' || 'd' < ans); // This is your clean up 


     Console.WriteLine(ans == 'c' ? "You are correct!" : "You are incorrect!"); 
    } 
} 

Результаты:

1 + 2 = ? 
a: 1 
b: 2 
c: 3 
d: 4 
Select answer: l 
Please select a, b, c, or d! 
m 
Please select a, b, c, or d! 
o 
Please select a, b, c, or d! 
c 
You are correct! 

См рабочий пример здесь ... https://dotnetfiddle.net/us7D3K