c# - このコードを簡略化できますか？

Question

このコードはたくさん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;
}

score 18 · Accepted Answer

あなたの最も簡単な賭け（トリックは含まれていません！）は次のようになります：

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;
}

score 2 · Accepted Answer

三項ifthenelse演算子を使用できます

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

これはこれ以上効率的ではありませんが、使用するコード行が少なくなります。

score 2 · Accepted Answer

さて、あなたは速記のifステートメントを使うことができます...

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);

score 0 · Accepted Answer

Reshaperをインストールしていますか？JetBrainのVS用Reshaper拡張機能を使用すると、すばらしい（将来の）支援が得られると思います。それに応じて、それは.NET開発者にとって素晴らしい必須ツールです。

c# - このコードを簡略化できますか？

4 に答える 4

Related

Reference