我的应用程序必须连接到API并下载大约1200个项目。因为API如何工作,我不能一次要求所有项目,我必须为每个项目发送一个查询。项目很小,所以下载很快,如果我在很短的时间内发送了太多的请求(每2-3秒100个请求),服务器会阻止我(4-5秒)。我不希望服务器每次阻止我的应用程序大约10-12次,所以我决定实现一个故意减速,另一个线程只是睡眠0.4秒,工人将加入它,所以如果工人完成每个在0.4s标记之前下载它将等待,如果它完成后者,它将直接进入下一个
期望的实施:
private void DownloadLibrary_DoWork(object sender, DoWorkEventArgs e)
{
int max = 1200;
int slowdown = 400; // milliseconds
Thread methronome = new Thread(() => Thread.Sleep(slowdown));
for (int i = 0; i < max; i++)
{
methronome.Start();
DownloadItem(i);
WorkerDownloadLibrary.ReportProgress((int)i / max, string.Format("Dowloaded item {0} out of {1}", i+1, max));
methronome.Join();
}
}
private void UserClickStart(object sender, RoutedEventArgs e)
{
System.Windows.MessageBox.Show("Let's start!");
WorkerDownloadLibrary.RunWorkerAsync();
}
private void DownloadLibrary_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
System.Windows.MessageBox.Show("Completed!");
}
问题在于,如果我这样做,后台工作程序会提前结束(RunWorkerCompleted执行),如果我同时评论methronome.Start()
和methronome.Join()
行,它会按预期工作,但是当然最终会被阻止。 我做错了什么?
我可以直接在后台工作器上实现等待,因为无论如何下载速度都足够快,但设计看起来仍然很糟糕,而且在慢速计算机上它会减慢太多。
工作但执行缓慢:
private void DownloadLibrary_DoWork(object sender, DoWorkEventArgs e)
{
int max = 1200;
int slowdown = 400; // milliseconds
for (int i = 0; i < max; i++)
{
DownloadItem(i);
WorkerDownloadLibrary.ReportProgress((int)i / max, string.Format("Dowloaded item {0} out of {1}", i+1, max));
Thread.Sleep(slowdown);
}
}
还有另一种方法:一个接一个地发送所有查询,并且仅在服务器阻塞时等待,但在设计上看起来非常糟糕,并且当下载每2秒停止时用户将抓住它的头。如果我这样做,服务器可能永久阻止我
工作但实施不好
private void DownloadLibrary_DoWork(object sender, DoWorkEventArgs e)
{
int max = 1200;
int slowdown = 5000; // milliseconds
string serverResponse;
for (int i = 0; i < max; i++)
{
do
{
serverResponse = DownloadItem(i);
if (serverResponse != "OK")
Thread.Sleep(slowdown);
} while (serverResponse != "OK") ;
WorkerDownloadLibrary.ReportProgress((int)i / max, string.Format("Dowloaded item {0} out of {1}", i+1, max));
}
}
如何实施第一个选项?
答案 0 :(得分:2)
您的方法不起作用,因为as the documentation explains:
一旦线程终止,则无法通过另一次调用Start来重新启动。
因此,最简单的解决方法是每次都重新创建线程:
for (int i = 0; i < max; i++)
{
Thread methronome = new Thread(() => Thread.Sleep(slowdown));
methronome.Start();
//..
然而,目前还不清楚你为什么要这样做。代码已在另一个线程中运行,因此使用Thread.Sleep
的第二种方法似乎很好。你可以通过测量你是否真的需要等待整个时间来改进它(也许一些下载持续一段时间,所以不需要总是等待所有400毫秒)。可用于测量时间块(与线程不同,可以“重新启动”)的方便类是Stopwatch类。这个想法(在评论中也由jessehouwing发布)看起来像是:
int max = 1200;
int slowdownMs = 400;
Stopwatch sw = new Stopwatch();
for (int i = 0;i < max; ++i)
{
sw.Restart();
DownloadItem(i);
ReportProgress(i, max);
sw.Stop();
Thread.Sleep(Math.Max(0, (int)(slowdownMs - sw.ElapsedMilliseconds)));
}
//....