Question

我想执行与现在正在执行的功能相同的过程，但我想要更紧凑的功能。

任何人都可以帮我减少这段代码......？

private void UpdateView(bool IsUpdateViewRequired)
{
        try
        {
            if (IsUpdateViewRequired == true)
            {
                this.Height = 210;
                labelStatus.Text = string.Empty;
                progressBar.Value = 0;
                buttonCancel.Visible = true;
                buttonStart.Visible = false;
            }
            else
            {
                this.Height = 165;
                buttonCancel.Visible = false;
                buttonStart.Visible = true;
            }
        }
        catch (Exception)
        {
            return;
        }
}

Answer 1

private void UpdateView(bool IsUpdateViewRequired)
{
    if (IsUpdateViewRequired) progressBar.Value = (labelStatus.Text = string.Empty).Length;
    this.Height = (buttonStart.Visible = !(buttonCancel.Visible = IsUpdateViewRequired)) ? 165 : 210;
}

享受简短;）抱歉，这是不可读的。

Answer 2

你可以将代码减少到下面的代码（我不认为这会有任何区别，因为你的语句只是简单的命令）

private void UpdateView(bool IsUpdateViewRequired)
{

            if (IsUpdateViewRequired == true)
            {
                this.Height = 210;
                labelStatus.Text = string.Empty;
                progressBar.Value = 0;

            }
            else
            {
                this.Height = 165;
            }
            buttonCancel.Visible = IsUpdateViewRequired;
            buttonStart.Visible = !IsUpdateViewRequired;

}

Answer 3

首先，您不需要try/catch。您也不需要== true。 this.也是多余的。此外，你可以消除一些if声明：

 private void UpdateView(bool IsUpdateViewRequired)
 {
            Height =IsUpdateViewRequired? 210:165;
            buttonCancel.Visible = IsUpdateViewRequired;
            buttonStart.Visible = !IsUpdateViewRequired;
            if (IsUpdateViewRequired)
            {
               labelStatus.Text = string.Empty;
               progressBar.Value = 0;
            }            
}

C＃：如何减少以下代码

3 个答案: