实现一次性布尔开关的标准方法是什么？

Question

假设我有如下代码，processData 每分钟执行数百甚至数千次：

class DataProcessor {
private:
    DataValidator* validator;
    bool atLeastOneDataPoint;
    bool dataIsValid(Data* dataToValidate) {
        return validator->validate(dataToValidate);
    }

public:
    // ...

    void processData(Data* dataToProcess) {
        if (dataIsValid(dataToProcess) || !atLeastOneDataPoint) {
            // process data
            // ...
            atLeastOneDataPoint = true;
        }
    }

    // ...
}

从它的名字可以推断，atLeastOneDataPoint 是一个真正只需要设置一次的变量，但在上面的代码中，每次在第一个数据点之后调用processData 时都会设置它。当然，我可以将赋值行更改为：

if (!atLeastOneDataPoint) atLeastOneDataPoint = true;

但这只会用一堆不必要的布尔检查替换一堆不必要的分配。

我不关心这段代码的性能；真的，我只是被做一些完全不必要的事情的想法所困扰。有没有一种标准的方式来设置这样的一次性开关，在设计上更直观“合适”？

至于关心这个是否会让我成为一个糟糕的程序员：请让我们改天再讨论吧。

Answer 1

我想说你已经得到的是最好的方法：只需在每次迭代时设置布尔标志。

如果您不关心性能，我认为这是最易读的解决方案：这很好而且很明显 - 我已经处理了一个有效的数据点，所以我将标志设置为 true。

即使性能是一个问题，我认为您也很难找到任何比将单个布尔值设置为 true 更有效的操作。

Answer 2

回调

先注册一个名为“call_once_at_init()”的函数，该函数会注册“call_always_after_init()”以供以后使用。

Answer 3

如果您不使用 atLeastOneDataPoint 作为流程数据中的第一个标志

void processData(Data* dataToProcess) {
   if ( (atLeastOneDataPoint = (dataIsValid(dataToProcess) || !atLeastOneDataPoint) ) ) {
      // process data
      //...
   }
}

但是每分钟 10000 次不算什么 - 当它每秒 150 万次时担心

Answer 4

这不是你想要的吗？

void processData(Data* dataToProcess) {
    if (!atLeastOneDataPoint) atLeastOnedataPoint = true;
    if (dataIsValid(dataToProcess)) {
        // process data
        // ...
    }
}

这实际上只执行atLeastOnedataPoint = true 一次。

即使每次都这样做，if (!atLeastOneDataPoint) atLeastOnedataPoint = true; 也应该花费不到 100 ns（可能要少得多）。即使在 100 ns 时，如果您在一分钟内执行 1000 次，那一分钟也将消耗 100us = 0.1ms。

我打赌你有“更大的鱼要炸”。