将 bool 设置为 false VS 在设置之前检查它是否已经为 false

Question

我知道这可能是过度优化，但我仍然想知道什么是更好的。

所以让我们添加一些背景信息。 我正在 Unity 上开发一款游戏，它使用 C# 编写游戏脚本，并且 Unity 制定了在游戏的每一帧运行的方法。 对于我正在开发的游戏，我在代码中使用触发器。它们是简单的布尔值，以 false 开头。当游戏中发生特定事件时，特定触发器将设置为 true，将立即激活事件并再次设置为 false。这一切都发生在一个帧中。

所以这意味着在每一帧结束时我必须将触发器重置为 false。 这就是我的疑问所在。我应该将所有触发器设置为 false，与它们的当前值无关，还是应该添加 IF 条件来检查它们是否尚未为 false？

Answer 1

一般来说，我建议您在设置之前检查布尔值。我没有任何关于哪个性能更高的数据，但它具有功能优势。如果您正在使用字段，检查和设置布尔值没有副作用。但是，如果您正在使用属性，则 getter 和 setter 可能会产生一些副作用，如果您每帧都设置它们，可能会导致不良行为。如果您只是使用该布尔值，则并不总是立即清楚您正在使用字段还是属性。

我建议您不要在 Update 方法中使用标志布尔值。 Update 方法应该只执行对象处于活动状态时必须执行的操作。我假设您有一些代码在发生某些情况时设置标志，并且在您的 Update 方法中执行某些操作，直到满足另一个条件并重置该标志。

// This is my flag variable.
public bool isCounting;
public int maximum;
private int _currentCount;

private void Update()
{
    if (IsConditionMet()) {
        // Should I check if isCounting is already false before setting?
        isCounting = false;
    }

    if (isCounting) {
        DoWork();
    }
    else
    {
        return;
    }
}

private bool IsConditionMet()
{
    return _currentCount >= maximum;
}

private void DoWork()
{
    _currentCount++;
}

当某些其他代码将标志 isCounting 设置为 true 时，_currentCount 将增加，直到达到最大值并停止计数。计数开始时恰好有一帧，计数停止时正好一帧。在所有其他帧期间，Update 方法检查标志变量并且不执行任何其他操作。

相反，您可以使用协程并调用方法：

public void StartCounting()
{
    StartCoroutine(Count());
}

private IEnumerable Count()
{
    while (!IsConditionMet())
    {
        DoWork();
        yield new WaitForSeconds(1);
    }
}

此代码公开了一个名为 StartCounting 的公共方法。要开始计数，您可以调用 StartCounting，而不是设置公共字段 isCounting。然后，该方法启动一个在 Update 代码外部执行的协程。协程 Count 检查条件是否满足，如果不满足则继续工作。完成工作后，协程会等待一秒钟并再次尝试，直到 _currentCount 达到最大值。这样，您就不必担心检查和设置标志值，而是调用 StartCounting 方法一次并让它工作直到完成。