具有启用/禁用按钮的 C# 性能和可读性

Question

我经常发现自己面对这样的代码：

if(Something > 0)
{
    btnOne.Enabled = true;
    btnTwo.Enabled = true;
    btnThree.Enabled = false:
}
else
{
    btnOne.Enabled = false;
    btnTwo.Enabled = false;
    btnThree.Enabled = true:
}

而且我一直想知道是不是这样更好，或者这样说：

bool ButtonEnabled = (Something > 0);

btnOne.Enabled = ButtonEnabled;
btnTwo.Enabled = ButtonEnabled;
btnThree.Enabled = !ButtonEnabled;

意识到这个问题有点争论，让我们把“可读性”放在一边“因素并专注于性能因素......什么是最好的？又一个任务还是一个条件？

预先感谢您的建议（或者更好的编写方式）！

编辑：更正了我的第二个片段中的错误。 编辑：两个最初的例子并不等同......

Answer 1

您无法比较这两段代码，无论是在可读性还是性能上，因为它们给出了不同的结果。

第一个代码与第二个代码等效的版本为：

if(Something > 0)
{
    btnOne.Enabled = true;
    btnTwo.Enabled = true;
    btnThree.Enabled = false;
    btnFour.Enabled = false;
}
else
{
    btnOne.Enabled = false;
    btnTwo.Enabled = false;
    btnThree.Enabled = true;
    btnFour.Enabled = true;
}

第二个代码与第一个代码等效的版本为：

bool ButtonEnabled = (Something > 0);

btnOne.Enabled = ButtonEnabled ? true : btnOne.Enabled;
btnTwo.Enabled = ButtonEnabled ? true : btnTwo.Enabled;
btnThree.Enabled = !ButtonEnabled ? false : btnThree.Enabled;
btnFour.Enabled = !ButtonEnabled ? false : btnFour.Enabled;

因此，第一段代码显然比其等效替代方案更高效、更易读，并且第二段代码比它的等效替代方案更短并且更容易维护。

Answer 2

这取决于被调用的属性。如您所知，财产可以做任何事情。在 Windows 窗体或 WPF 中，我不会担心这个问题。为了正确性和可读性，我主张后一种风格。如果您每次都设置所有必要的变量，那么遗漏某些内容并使某个按钮处于无效状态的可能性就会较小。

我会做类似的事情

bool ButtonEnabled = (Something > 0);
btnOne.Enabled = ButtonEnabled;
btnTwo.Enabled = ButtonEnabled;
btnThree.Enabled = !ButtonEnabled;
btnFour.Enabled = !ButtonEnabled;

Answer 3

在这种情况下，无论您在两者之间看到什么性能差异，很可能都是微不足道的，因此我会选择最具可读性的一个。

Answer 4

是的，与您的应用程序同时显示十万个按钮不同，高度关注可读性，而不是微观优化！无论如何，UI 层更新控件视觉效果所需的时间将比“启用”分配长 10.000 倍！

解决方案 2 实际上几乎就是您在使用数据绑定时想要做的事情（您已经非常接近了：p）。实际上，您可以编写类似于以下内容的代码：

public class MyClass {
    public bool IsSomethingTrue { get; set; } // with notification on property changed
    public bool IsSomethingFalse { get { return !IsSomethingTrue; } }

    private AMethod() {
        ...
        IsSomethingTrue = Something > 0;
        ...
    }

您的 UI 将类似于（WPF 风格）：

<Button IsEnabled={Binding IsSomethingTrue} /> <!-- btn 1 -->
<Button IsEnabled={Binding IsSomethingTrue} /> <!-- btn 2 -->
<Button IsEnabled={Binding IsSomethingFalse} /> <!-- btn 3 -->
<Button IsEnabled={Binding IsSomethingFalse} /> <!-- btn 4 -->
<!-- Want a 5th button ? just add it without changing your code-behind ! -->

这种模式允许您根据需要添加任意数量的按钮，而无需每次都更改方法。当方法往往非常复杂时，这尤其有用，并且提高了可读性。

它适用于 WPF、Qt、Java，我认为 Winforms 应该提供一些数据绑定功能。