Можно ли упростить этот код?

Этот фрагмент кода if/else, и я хочу знать, можно ли его упростить до нескольких строк. Код работает отлично, но я предпочитаю более эффективный и чистый способ.Можно ли упростить этот код?

if (textBox_naam.Text.Length < 3) 
{ 
    textBox_naam.BackColor = Color.FromArgb(205, 92, 92); 
} 
else 
{ 
    textBox_naam.BackColor = Color.White; 
} 

if (textBox_email.Text.Length < 5) 
{ 
    textBox_email.BackColor = Color.FromArgb(205, 92, 92); 
} 
else 
{ 
    textBox_email.BackColor = Color.White; 
} 

if (textBox_body.Text.Length < 20) 
{ 
    textBox_body.BackColor = Color.FromArgb(205, 92, 92); 
} 
else 
{ 
    textBox_body.BackColor = Color.White; 
}

источник

2012-04-26 Devator

У нас есть сайт-сестра для обзора кода рабочего кода. –

Чтобы уточнить, похоже, вы имеете в виду разные текстовые поля в вашем коде? Я почти собирался свернуть утверждения ветки в один, если ... иначе, если ... Но я думаю, что ответ Дарена помогает? – Eniola

Ваших простые ставки (! Никаких ухищрений не участвуют) будут:

SetBackColor(textBox_naam, 3, GOOD_COLOR, BAD_COLOR); 
SetBackColor(textBox_email, 5, GOOD_COLOR, BAD_COLOR); 
SetBackColor(textBox_body, 20, GOOD_COLOR, BAD_COLOR);

с методом SetBackColor определен следующим образом:

public void SetBackColor(TextBox tb, int minLength, Color goodColor, Color badColor) 
{ 
    tb.BackColor = tb.Text.Length < minLength ? badColor : goodColor; 
}

источник

2012-04-26 13:26:03

Вы имеете в виду 3, 5, 20 – sinelaw

@sinelaw, справа. обновлено. благодаря! –

Учитывая, что «GOOD_COLOR» и «BAD_COLOR» одинаковы во всех трех случаях, нет никаких причин сделать их параметрами для функции. Просто передайте в поле и длину и пусть они являются константами внутри функции. – mjfgates

Вы можете использовать троичной, если потом еще оператор

textBox_naam.BackColor = textBox_naam.Text.Length < 3 ? Color.FromArgb(205, 92, 92) : Color.White;

Это не какой-либо более эффективным, но будет использовать меньше строк кода.

источник

2012-04-26 13:25:45 Phil

Ну вы могли бы использовать сокращенный если заявления ...

Color other=Color.FromArgb(205,92,92); 
textBox_naam.BackColor=(textBox_naam.Text.Length<3?other:Color.White); 
textBox_email.BackColor=(textBox_email.Text.Length<5?other:Color.White); 
textBox_body.BackColor=(textBox_body.Text.Length<20?other:Color.White);

источник

2012-04-26 13:28:31 Rudu

У вас есть Reshaper установлен? Я думаю, вы найдете отличную (будущую) помощь, используя расширение JetBrain Reshaper для VS. Зависит от этого, это замечательно must-have инструмент для разработчиков .NET.

источник

2012-04-26 14:18:57 olatunjee

Не могли бы вы поделиться тем, как расширение можно использовать в этом сценарии? – Eniola

Можно ли упростить этот код?

ответ

Смежные вопросы