Условный оператор неприятности - сделать код меньше

Question

Я хочу, чтобы этот код меньше

if (char.IsLetter(e.KeyChar) || e.KeyChar == ',' || e.KeyChar == '/' || e.KeyChar == ';' || e.KeyChar == '<' || e.KeyChar == '>') 
     { 
      e.Handled = true; 
     } 

Но я не знаю, как ... Пожалуйста, помогите

Благодарности

Answer 1

Как кто-то намекнул в комментарий, вы можете использовать Contains.

char[] allowable = new [] { ',', '/', ';', '<', '>' }; 

if (char.IsLetter(e.KeyChar) || allowable.Contains(e.KeyChar)) 
{ 
    e.Handled = true; 
} 

Возможно, вы захотите сделать этот массив static readonly в родительском классе.

Как Деннис отметил в комментарии ниже, хорошая альтернатива всем это может быть:

char[] allowable = new [] { ',', '/', ';', '<', '>' }; 

e.Handled = char.IsLetter(e.KeyChar) || allowable.Contains(e.KeyChar); 

Это определенно короче, но это хорошо, чтобы отметить, что это не совсем синонимы. Но они будут функционально эквивалентны, если у вас нет предшествующих условий, которые устанавливают e.Handled без звонка return.

Answer 2

2 линии?

char[] allowableChars = new [] { ',', '/', ';', '<', '>' }; 
e.handled = (char.IsLetter(e.KeyChar) || allowableChars.Contains(e.KeyChar));