2

我有一个定时器对象的向量。每个 Timer Object 都会启动一个模拟生长期的 std::thread。我正在使用命令模式。

发生的事情是每个 Timer 一个接一个地被执行，但我真正想要的是一个被执行......然后一旦完成，下一个......一旦完成下一个......同时不干扰主程序的执行

class Timer 
{
    public:

        bool _bTimerStarted;
        bool _bTimerCompleted;

        int _timerDuration;

        virtual ~Timer() { }
        virtual void execute()=0;
        virtual void runTimer()=0;

        inline void setDuration(int _s) { _timerDuration = _s; };
        inline int getDuration() { return _timerDuration; };

        inline bool isTimerComplete() { return _bTimerCompleted; };
};

class GrowingTimer : public Timer
{
    public:
        void execute()
        {
            //std::cout << "Timer execute..." << std::endl;

            _bTimerStarted = false;
            _bTimerCompleted = false;

            //std::thread t1(&GrowingTimer::runTimer, this); //Launch a thread
            //t1.detach();

            runTimer();
        }

        void runTimer()
        {
            //std::cout << "Timer runTimer..." << std::endl;

            _bTimerStarted = true;

            auto start = std::chrono::high_resolution_clock::now();
            std::this_thread::sleep_until(start + std::chrono::seconds(20));

            _bTimerCompleted = true;

            std::cout << "Growing Timer Finished..." << std::endl; 
        }
};

class Timers
{
    std::vector<Timer*> _timers;

    struct ExecuteTimer
    {
        void operator()(Timer* _timer) { _timer->execute(); }
    };

    public:
        void add_timer(Timer& _timer) { _timers.push_back(&_timer); }

        void execute()
        {
            //std::for_each(_timers.begin(), _timers.end(), ExecuteTimer());

            for (int i=0; i < _timers.size(); i++)
            {
                 Timer* _t = _timers.at(i);
                _t->execute();

                //while ( ! _t->isTimerComplete())
                //{

                //}
            }
        }
};

执行上述类似：

Timers _timer;
GrowingTimer _g, g1;

_g.setDuration(BROCCOLI::growTimeSeconds);
_g1.setDuration(BROCCOLI::growTimeSeconds);

_timer.add_timer(_g);
_timer.add_timer(_g1);

start_timers();

}

void start_timers() 
{
    _timer.execute();
}

在 Timers::execute 中，我尝试了几种不同的方法来执行第一个而不是执行下一个，直到我以某种方式发出信号完成。

更新：

我现在这样做是为了执行一切：

Timers _timer;
GrowingTimer _g, g1;

_g.setDuration(BROCCOLI::growTimeSeconds);
_g1.setDuration(BROCCOLI::growTimeSeconds);

_timer.add_timer(_g);
_timer.add_timer(_g1);

//start_timers();

std::thread t1(&Broccoli::start_timers, this); //Launch a thread
t1.detach();

}

void start_timers() 
{
    _timer.execute();
}

第一次完成（我看到“已完成”cout），但在_t->execute();内部崩溃并for loop带有 EXEC_BAD_ACCESS。我添加了一个 cout 来检查向量的大小，它是 2，所以两个计时器都在里面。我确实在控制台中看到了这一点：

this    Timers *    0xbfffd998
_timers std::__1::vector<Timer *, std::__1::allocator<Timer *> >

如果我将 更改detach()为join()一切都完成而没有崩溃，但它会阻止我的应用程序的执行，直到这些计时器完成。

4

3 回答 3

3

你为什么在这里使用线程？Timers::execute()调用execute一个计时器，然后等待它完成，然后调用execute下一个，依此类推。你为什么不直接调用定时器函数Timers::execute()而不是产生一个线程然后等待它呢？

线程允许您编写并发执行的代码。你想要的是串行执行，所以线程是错误的工具。

更新：在您start_timers在后台线程上运行的更新代码中，这很好。但是，通过分离该线程，您会使线程运行超出范围的末尾。这意味着在线程完成之前，计时器对象甚至_g对象都可能被销毁。鉴于计时器线程的耗时性质，以及您使用而不是为了避免代码阻塞的事实，这肯定是您的问题的原因。_g1Timers_timersdetachjoin

如果您在线程上运行代码，那么您需要确保该线程访问的所有对象都具有足够长的生命周期，以便在线程访问它们时它们仍然有效。对于分离的线程，这尤其难以实现，因此不建议使用分离的线程。

一种选择是创建一个包含_timers,_g并_g1沿着线程的对象t1，并使其析构函数与线程连接。然后，您需要做的就是确保对象一直存在，直到可以安全地等待计时器完成为止。

于 2013-08-07T07:56:02.310 回答
1

您可以将 aunique_ptr添加到threadinGrowingTimer而不是将其创建为本地对象execute并调用detach. 您仍然可以在 中创建线程execute，但您可以通过unique_ptr::reset调用来完成。

然后使用join代替（向基类isTimerComplete添加一个join函数）。Timer轮询机制将isTimerComplete非常低效，因为它基本上会用完该线程的整个时间片不断轮询，而join将阻塞直到另一个线程完成。

一个例子join：

#include <iostream>
#include <chrono>
#include <thread>

using namespace std;

void threadMain()
{
    this_thread::sleep_for(chrono::seconds(5));

    cout << "Done sleeping\n";
}

int main()
{
    thread t(threadMain);

    for (int i = 0; i < 10; ++i)
    {
        cout << i << "\n";
    }

    t.join();

    cout << "Press Enter to exit\n";

    cin.get();
    return 0;
}

请注意主线程如何在其他线程执行其操作时继续运行。请注意，Anthony 的回答是正确的，因为您似乎不需要多个后台线程来按顺序执行任务，而不是启动一个线程并等待它完成，然后再启动一个新线程。

于 2013-08-07T04:35:08.413 回答
1

如果您不想干扰程序的执行，您可以执行@Joel 所说的操作，但也可以在 Timers 类中添加一个线程，该线程将执行向量中的线程。

于 2013-08-07T04:40:28.117 回答