c++ 11:定期调用c++函数

C++ 11: Calling a C++ function periodically

本文关键字：c++ 调用 函数      更新时间：2023-10-16

我已经把一个简单的c++定时器类放在一起，应该从各种例子中定期调用给定的函数，如下所示:

#include <functional>
#include <chrono>
#include <future>
#include <cstdio>
class CallBackTimer
{
public:
    CallBackTimer()
    :_execute(false)
    {}
    void start(int interval, std::function<void(void)> func)
    {
        _execute = true;
        std::thread([&]()
        {
            while (_execute) {
                func();                   
                std::this_thread::sleep_for(
                std::chrono::milliseconds(interval));
            }
        }).detach();
    }
    void stop()
    {
        _execute = false;
    }
private:
    bool            _execute;
};

现在我想从c++类中调用它，如下所示:

class Processor()
{
    void init()
    {
         timer.start(25, std::bind(&Processor::process, this));
    }
    void process()
    {
        std::cout << "Called" << std::endl;
    }
};

然而，这个调用带有错误

terminate called after throwing an instance of 'std::bad_function_call'
what():  bad_function_call

代码中的问题是，"start"函数中的lambda表达式使用[&]语法通过引用捕获局部变量。这意味着lambda通过引用捕获interval和func变量，它们都是start()函数的局部变量，因此，它们在从该函数返回后消失。但是，从该函数返回后，lambda在分离的线程中仍然是活动的。这时你就会得到"bad-function-call"异常，因为它试图通过引用一个不再存在的对象来调用func。

您需要做的是使用lambda上的[=]语法按值捕获局部变量，如下所示:

void start(int interval, std::function<void(void)> func)
{
    _execute = true;
    std::thread([=]()
    {
        while (_execute) {
            func();                   
            std::this_thread::sleep_for(
            std::chrono::milliseconds(interval));
        }
    }).detach();
}

当我尝试它时，这个工作。

或者，您也可以列出您想要更显式地捕获的值(我通常建议使用lambdas):

void start(int interval, std::function<void(void)> func)
{
    _execute = true;
    std::thread([this, interval, func]()
    {
        while (_execute) {
            func();                   
            std::this_thread::sleep_for(
            std::chrono::milliseconds(interval));
        }
    }).detach();
}

编辑

正如其他人指出的那样，使用分离线程并不是一个很好的解决方案，因为您很容易忘记停止线程，并且无法检查它是否已经在运行。此外，您可能应该使_execute标志原子化，只是为了确保它不会被优化，并且读/写是线程安全的。你可以这样做:

class CallBackTimer
{
public:
    CallBackTimer()
    :_execute(false)
    {}
    ~CallBackTimer() {
        if( _execute.load(std::memory_order_acquire) ) {
            stop();
        };
    }
    void stop()
    {
        _execute.store(false, std::memory_order_release);
        if( _thd.joinable() )
            _thd.join();
    }
    void start(int interval, std::function<void(void)> func)
    {
        if( _execute.load(std::memory_order_acquire) ) {
            stop();
        };
        _execute.store(true, std::memory_order_release);
        _thd = std::thread([this, interval, func]()
        {
            while (_execute.load(std::memory_order_acquire)) {
                func();                   
                std::this_thread::sleep_for(
                std::chrono::milliseconds(interval));
            }
        });
    }
    bool is_running() const noexcept {
        return ( _execute.load(std::memory_order_acquire) && 
                 _thd.joinable() );
    }
private:
    std::atomic<bool> _execute;
    std::thread _thd;
};
相关文章：