我有一个Timer对象的向量。每个Timer对象都会启动一个模拟生长期的std :: thread。我正在使用Command模式。
正在发生的事情是每个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()
,一切都会在没有崩溃的情况下完成,但它会阻止我的应用程序执行,直到这些计时器完成。
答案 0 :(得分:3)
你为什么在这里使用线程? Timers::execute()
在计时器上调用execute
,然后等待它完成,然后在下一个调用execute
,依此类推。你为什么不直接在Timers::execute()
中调用timer函数而不是生成一个线程然后等待呢?
线程允许您编写并发执行的代码。你想要的是串行执行,因此线程是错误的工具。
更新:在更新的代码中,您在后台线程上运行start_timers
,这很好。但是,通过分离该线程,您可以使线程继续运行到作用域的末尾。这意味着在线程完成之前,计时器对象_g
和_g1
甚至Timers
对象_timers
可能会被销毁。考虑到计时器线程的耗时性质,以及您使用detach
而不是join
以避免代码阻塞的事实,这肯定是导致问题的原因。
如果在线程上运行代码,则需要确保该线程访问的所有对象都具有足够长的生命周期,以便在线程访问它们时它们仍然有效。对于分离的线程,这很难实现,因此不建议使用分离的线程。
一种选择是在线程_timers
旁边创建一个包含_g
,_g1
和t1
的对象,并让其析构函数与线程连接。您需要做的就是确保对象存在,直到等待定时器完成是安全的。
答案 1 :(得分:1)
您可以在unique_ptr
中的thread
添加GrowingTimer
,而不是在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的答案是正确的,因为它似乎不需要多个后台线程,它只是按顺序执行任务而不是在开始一个新线程之前启动一个线程并等待它完成。
答案 2 :(得分:1)
如果你不想干扰程序的执行,你可以像@Joel那样做,但也可以在Timers类中添加一个线程来执行向量中的线程。