为什么5个工作线程的执行顺序保持不边?达人请进.

czb 2007-08-07 05:03:43
程序首先创建了一个投递线程,5个工作线程(线程0、线程1、线程2、线程3、线程4)。投递线程用于产生数据,并把产生的数据放到一个队列中。5个工作线程从这个数据队列中提取数据,并把提取后的数据给删掉,并随机延迟一段时间。
每个工作线程延迟一个随机的时间后,继续从数据队列中提取数据。
这5个工作线程是处于竞争状态的,如线程0、线程1、线程3、线程4处于运行状态,线程2处于空闲状态,则下一次系统调度,就调用线程2。

现在我的问题是,程序中这5个工作线程,是按顺序处理队列数据的。即先是线程1处理数据,再是线程2,3,4,0;一个循环之后,还是1、2、3、4、0;再循环之后,接着还是1、2、3、4、0...
这5个工作线程根本就没有抢时间去跑。

付关键代码如下
#define workThreadCount 5;
volatile bool isMainThreadWorking;
std::queue<int> lst_data;

std::queue<int> lst_p_thread_data[5];

int count;
CCriticalSection critical_section;
CEvent eventBeginWork;

UINT postThread(LPVOID pParam);
UINT workThread(LPVOID pParam);

UINT postThread(LPVOID pParam)
{
time_t t;
srand((unsigned) time(&t));

while(isMainThreadWorking&&count<10000){
critical_section.Lock();

lst_data.push(count);
count++;

critical_section.Unlock();
if(count%10==0)
{
eventBeginWork.SetEvent();
}

Sleep(rand()%800);
}
return 0;
}

UINT workThread(LPVOID pParam)
{
int ThreadNo;
std::queue<int> lst_work_thread_data;
time_t t;
srand((unsigned) time(&t));

ThreadNo = *((int*)pParam);

//CString sPrintMessage;
//sPrintMessage.Format(_T("线程%d开始"),ThreadNo);
//AfxMessageBox(sPrintMessage);

while(isMainThreadWorking){

WaitForSingleObject(eventBeginWork,INFINITE);

critical_section.Lock();

CString threadNo;
threadNo.Format(_T("线程%d正在运行"),ThreadNo);
//((CDataWriterDlg*)AfxGetApp()->m_pMainWnd)->m_display_static.SetWindowTextW(threadNo);

while(!lst_data.empty())
{
lst_work_thread_data.push(lst_data.front());
lst_data.pop();
}

critical_section.Unlock();

Sleep(rand()%1200);
}

while(!lst_work_thread_data.empty())
{
lst_p_thread_data[ThreadNo].push(lst_work_thread_data.front());
lst_work_thread_data.pop();
}

return 0;
}

void CDataWriterDlg::OnBnClickedStart()
{
// TODO: 在此添加控件通知处理程序代码
isMainThreadWorking = true;
pPostThread = AfxBeginThread((AFX_THREADPROC)postThread,NULL);

for(int i=0;i<5;i++)
{
pWorkThread[i] = AfxBeginThread((AFX_THREADPROC)workThread,(LPVOID)&i);
}
}

void CDataWriterDlg::OnBnClickedStop()
{
// TODO: 在此添加控件通知处理程序代码
isMainThreadWorking = false;
WaitForSingleObject(pPostThread->m_hThread,INFINITE);

eventBeginWork.SetEvent();
eventBeginWork.SetEvent();
eventBeginWork.SetEvent();
eventBeginWork.SetEvent();
eventBeginWork.SetEvent();

WaitForSingleObject(pWorkThread[0]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[1]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[2]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[3]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[4]->m_hThread,INFINITE);

}

void CDataWriterDlg::OnBnClickedSave()
{
// TODO: 在此添加控件通知处理程序代码
CStdioFile mFile;
CFileException mExcept;
mFile.Open(_T("Result.txt"), CFile::modeCreate|CFile::modeWrite, &mExcept);
CString string;
string.Format(_T("count=%d\r\n"),count);
//string.Append(_T(","));

for(int i=0;i<5;i++)
{
CString threadNO;
threadNO.Format(_T("Thread No:%d\r\n"),i);
string.Append(threadNO);
while(!lst_p_thread_data[i].empty())
{
CString values;
values.Format(_T("%d,"),lst_p_thread_data[i].front());
lst_p_thread_data[i].pop();
string.Append(values);
}
string.Append(_T("\r\n"));
}

mFile.WriteString(string);
mFile.Close();
}

class CDataWriterDlg : public CDialog
{
// 构造
public:
CDataWriterDlg(CWnd* pParent = NULL); // 标准构造函数

// 对话框数据
enum { IDD = IDD_DATAWRITER_DIALOG };

protected:
virtual void DoDataExchange(CDataExchange* pDX); // DDX/DDV 支持

public:
CWinThread* pPostThread;
CWinThread* pWorkThread[5];

// 实现
protected:
HICON m_hIcon;

// 生成的消息映射函数
virtual BOOL OnInitDialog();
afx_msg void OnSysCommand(UINT nID, LPARAM lParam);
afx_msg void OnPaint();
afx_msg HCURSOR OnQueryDragIcon();
DECLARE_MESSAGE_MAP()
public:
afx_msg void OnDestroy();
public:
afx_msg void OnBnClickedStart();
public:
afx_msg void OnBnClickedStop();
public:
afx_msg void OnBnClickedSave();
public:
CStatic m_display_static;
};
...全文
296 15 打赏 收藏 转发到动态 举报
写回复
用AI写文章
15 条回复
切换为时间正序
请发表友善的回复…
发表回复
czb 2007-08-15
  • 打赏
  • 举报
回复
感谢各位朋友的大力帮助
结贴了
jasonshark 2007-08-11
  • 打赏
  • 举报
回复
你的event是不是自动reset的?这样的话你每次SetEvent实际上只有一个线程能进入Ready状态. 至于每次SetEvent的时候究竟是哪个线程进入Ready态,这个应该是操作系统的implementation detail了, 而且具体的调度策略可能在每个版本的操作系统中可能还有变化,毕竟是undocumented的东西. 我只能分析的到这么多了,呵呵~

PS:我刚开始认为是线程亲缘性的原因,但想了想觉得不对,亲缘性是指单个线程调度到多个CPU上的选择策略
czb 2007-08-11
  • 打赏
  • 举报
回复
如jasonshark(没暑假了...) 所说,把时间的延迟调到两个极端,操作系统确实是会不按照顺序来调度线程了,不过这不是我所需要的.
我需要的是,postThread里根本就不需要时间延迟,然后在每count增加到10的整数倍的时候,设置一下信号,见postThrea中下面的代码
if(count%10==0)
{
eventBeginWork.SetEvent();
}
但这时奇怪的时候,只有5个中的一个workThread执行了,而且这个workThread处理完了所有的数据,其它的4个workThread根本没有执行.难道其它的SetEvent没有反应过来?
czb 2007-08-08
  • 打赏
  • 举报
回复
jasonshark(没暑假了...) :
可以这么说了.
我按照你说的测试一下.
jasonshark 2007-08-08
  • 打赏
  • 举报
回复
我大概明白你的意思...
你是说尽管每个线程Sleep的时间长短不一样. 但最终操作系统调度执行结果,却看不出Sleep时间不同, 是不是这样?

你可以尝试把其实一两个线程Sleep的时间特别长,达到3,4秒,而其他的在1秒之内,再看看结果呢
czb 2007-08-08
  • 打赏
  • 举报
回复
大家观察"running thread no:"下面的行,这些行显示了哪个线程正在取队列中的数据,由上面的数据可知,正在取队列中数据的线程总是按照线程"1,2,3,4,0"的顺序来进行,即这5个工作线程根本就没有抢时间去跑.

这么说,大家应该都明白了吧?
czb 2007-08-08
  • 打赏
  • 举报
回复
点击start按钮(OnBnClickedStart),运行一段时间后再点击stop按钮(OnBnClickedStop),然后点保存save按钮(OnBnClickedSave),结果如下
count=1595

Thread No:0

40,41,42,43,44,45,46,47,48,49,90,91,92,93,94,95,96,97,98,99,140,141,142,143,144,145,146,147,148,149,190,191,192,193,194,195,196,197,198,199,240,241,242,243,244,245,246,247,248,249,290,291,292,293,294,295,296,297,298,299,340,341,342,343,344,345,346,347,348,349,390,391,392,393,394,395,396,397,398,399,440,441,442,443,444,445,446,447,448,449,490,491,492,493,494,495,496,497,498,499,540,541,542,543,544,545,546,547,548,549,590,591,592,593,594,595,596,597,598,599,640,641,642,643,644,645,646,647,648,649,690,691,692,693,694,695,696,697,698,699,740,741,742,743,744,745,746,747,748,749,790,791,792,793,794,795,796,797,798,799,840,841,842,843,844,845,846,847,848,849,890,891,892,893,894,895,896,897,898,899,940,941,942,943,944,945,946,947,948,949,990,991,992,993,994,995,996,997,998,999,1040,1041,1042,1043,1044,1045,1046,1047,1048,1049,1090,1091,1092,1093,1094,1095,1096,1097,1098,1099,1140,1141,1142,1143,1144,1145,1146,1147,1148,1149,1190,1191,1192,1193,1194,1195,1196,1197,1198,1199,1240,1241,1242,1243,1244,1245,1246,1247,1248,1249,1290,1291,1292,1293,1294,1295,1296,1297,1298,1299,1340,1341,1342,1343,1344,1345,1346,1347,1348,1349,1390,1391,1392,1393,1394,1395,1396,1397,1398,1399,1440,1441,1442,1443,1444,1445,1446,1447,1448,1449,1490,1491,1492,1493,1494,1495,1496,1497,1498,1499,1540,1541,1542,1543,1544,1545,1546,1547,1548,1549,1590,1591,1592,1593,1594,

Thread No:1

0,1,2,3,4,5,6,7,8,9,50,51,52,53,54,55,56,57,58,59,100,101,102,103,104,105,106,107,108,109,150,151,152,153,154,155,156,157,158,159,200,201,202,203,204,205,206,207,208,209,250,251,252,253,254,255,256,257,258,259,300,301,302,303,304,305,306,307,308,309,350,351,352,353,354,355,356,357,358,359,400,401,402,403,404,405,406,407,408,409,450,451,452,453,454,455,456,457,458,459,500,501,502,503,504,505,506,507,508,509,550,551,552,553,554,555,556,557,558,559,600,601,602,603,604,605,606,607,608,609,650,651,652,653,654,655,656,657,658,659,700,701,702,703,704,705,706,707,708,709,750,751,752,753,754,755,756,757,758,759,800,801,802,803,804,805,806,807,808,809,850,851,852,853,854,855,856,857,858,859,900,901,902,903,904,905,906,907,908,909,950,951,952,953,954,955,956,957,958,959,1000,1001,1002,1003,1004,1005,1006,1007,1008,1009,1050,1051,1052,1053,1054,1055,1056,1057,1058,1059,1100,1101,1102,1103,1104,1105,1106,1107,1108,1109,1150,1151,1152,1153,1154,1155,1156,1157,1158,1159,1200,1201,1202,1203,1204,1205,1206,1207,1208,1209,1250,1251,1252,1253,1254,1255,1256,1257,1258,1259,1300,1301,1302,1303,1304,1305,1306,1307,1308,1309,1350,1351,1352,1353,1354,1355,1356,1357,1358,1359,1400,1401,1402,1403,1404,1405,1406,1407,1408,1409,1450,1451,1452,1453,1454,1455,1456,1457,1458,1459,1500,1501,1502,1503,1504,1505,1506,1507,1508,1509,1550,1551,1552,1553,1554,1555,1556,1557,1558,1559,

Thread No:2

10,11,12,13,14,15,16,17,18,19,60,61,62,63,64,65,66,67,68,69,110,111,112,113,114,115,116,117,118,119,160,161,162,163,164,165,166,167,168,169,210,211,212,213,214,215,216,217,218,219,260,261,262,263,264,265,266,267,268,269,310,311,312,313,314,315,316,317,318,319,360,361,362,363,364,365,366,367,368,369,410,411,412,413,414,415,416,417,418,419,460,461,462,463,464,465,466,467,468,469,510,511,512,513,514,515,516,517,518,519,560,561,562,563,564,565,566,567,568,569,610,611,612,613,614,615,616,617,618,619,660,661,662,663,664,665,666,667,668,669,710,711,712,713,714,715,716,717,718,719,760,761,762,763,764,765,766,767,768,769,810,811,812,813,814,815,816,817,818,819,860,861,862,863,864,865,866,867,868,869,910,911,912,913,914,915,916,917,918,919,960,961,962,963,964,965,966,967,968,969,1010,1011,1012,1013,1014,1015,1016,1017,1018,1019,1060,1061,1062,1063,1064,1065,1066,1067,1068,1069,1110,1111,1112,1113,1114,1115,1116,1117,1118,1119,1160,1161,1162,1163,1164,1165,1166,1167,1168,1169,1210,1211,1212,1213,1214,1215,1216,1217,1218,1219,1260,1261,1262,1263,1264,1265,1266,1267,1268,1269,1310,1311,1312,1313,1314,1315,1316,1317,1318,1319,1360,1361,1362,1363,1364,1365,1366,1367,1368,1369,1410,1411,1412,1413,1414,1415,1416,1417,1418,1419,1460,1461,1462,1463,1464,1465,1466,1467,1468,1469,1510,1511,1512,1513,1514,1515,1516,1517,1518,1519,1560,1561,1562,1563,1564,1565,1566,1567,1568,1569,

Thread No:3

20,21,22,23,24,25,26,27,28,29,70,71,72,73,74,75,76,77,78,79,120,121,122,123,124,125,126,127,128,129,170,171,172,173,174,175,176,177,178,179,220,221,222,223,224,225,226,227,228,229,270,271,272,273,274,275,276,277,278,279,320,321,322,323,324,325,326,327,328,329,370,371,372,373,374,375,376,377,378,379,420,421,422,423,424,425,426,427,428,429,470,471,472,473,474,475,476,477,478,479,520,521,522,523,524,525,526,527,528,529,570,571,572,573,574,575,576,577,578,579,620,621,622,623,624,625,626,627,628,629,670,671,672,673,674,675,676,677,678,679,720,721,722,723,724,725,726,727,728,729,770,771,772,773,774,775,776,777,778,779,820,821,822,823,824,825,826,827,828,829,870,871,872,873,874,875,876,877,878,879,920,921,922,923,924,925,926,927,928,929,970,971,972,973,974,975,976,977,978,979,1020,1021,1022,1023,1024,1025,1026,1027,1028,1029,1070,1071,1072,1073,1074,1075,1076,1077,1078,1079,1120,1121,1122,1123,1124,1125,1126,1127,1128,1129,1170,1171,1172,1173,1174,1175,1176,1177,1178,1179,1220,1221,1222,1223,1224,1225,1226,1227,1228,1229,1270,1271,1272,1273,1274,1275,1276,1277,1278,1279,1320,1321,1322,1323,1324,1325,1326,1327,1328,1329,1370,1371,1372,1373,1374,1375,1376,1377,1378,1379,1420,1421,1422,1423,1424,1425,1426,1427,1428,1429,1470,1471,1472,1473,1474,1475,1476,1477,1478,1479,1520,1521,1522,1523,1524,1525,1526,1527,1528,1529,1570,1571,1572,1573,1574,1575,1576,1577,1578,1579,

Thread No:4

30,31,32,33,34,35,36,37,38,39,80,81,82,83,84,85,86,87,88,89,130,131,132,133,134,135,136,137,138,139,180,181,182,183,184,185,186,187,188,189,230,231,232,233,234,235,236,237,238,239,280,281,282,283,284,285,286,287,288,289,330,331,332,333,334,335,336,337,338,339,380,381,382,383,384,385,386,387,388,389,430,431,432,433,434,435,436,437,438,439,480,481,482,483,484,485,486,487,488,489,530,531,532,533,534,535,536,537,538,539,580,581,582,583,584,585,586,587,588,589,630,631,632,633,634,635,636,637,638,639,680,681,682,683,684,685,686,687,688,689,730,731,732,733,734,735,736,737,738,739,780,781,782,783,784,785,786,787,788,789,830,831,832,833,834,835,836,837,838,839,880,881,882,883,884,885,886,887,888,889,930,931,932,933,934,935,936,937,938,939,980,981,982,983,984,985,986,987,988,989,1030,1031,1032,1033,1034,1035,1036,1037,1038,1039,1080,1081,1082,1083,1084,1085,1086,1087,1088,1089,1130,1131,1132,1133,1134,1135,1136,1137,1138,1139,1180,1181,1182,1183,1184,1185,1186,1187,1188,1189,1230,1231,1232,1233,1234,1235,1236,1237,1238,1239,1280,1281,1282,1283,1284,1285,1286,1287,1288,1289,1330,1331,1332,1333,1334,1335,1336,1337,1338,1339,1380,1381,1382,1383,1384,1385,1386,1387,1388,1389,1430,1431,1432,1433,1434,1435,1436,1437,1438,1439,1480,1481,1482,1483,1484,1485,1486,1487,1488,1489,1530,1531,1532,1533,1534,1535,1536,1537,1538,1539,1580,1581,1582,1583,1584,1585,1586,1587,1588,1589,



running thread no:

1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,0,1,2,3,4,
czb 2007-08-08
  • 打赏
  • 举报
回复
不好意思,我上面的说明说得不太明白,这里重新再说明一下,并且根据jasonshark(没暑假了...) 和fronz()的建议,每个线程根据线程号采用固定时间的延迟,并把正在运行的线程号也记录下来.修改后的DataWriterDlg.cpp程序如下

#include "stdafx.h"

#include <list>
#include <string>
#include <queue>
#include <afxmt.h>
#include <time.h>


#include "DataWriter.h"
#include "DataWriterDlg.h"

#ifdef _DEBUG
#define new DEBUG_NEW
#endif

#define workThreadCount 5;
volatile bool isMainThreadWorking;
std::queue<int> lst_data;
std::queue<int> running_thread_no;
std::queue<int> lst_p_thread_data[5];

int count;
CCriticalSection critical_section;
CEvent eventBeginWork;

UINT postThread(LPVOID pParam);
UINT workThread(LPVOID pParam);

UINT postThread(LPVOID pParam)
{
time_t t;
srand((unsigned) time(&t));

while(isMainThreadWorking&&count<10000){
critical_section.Lock();

lst_data.push(count);
count++;

critical_section.Unlock();
if(count%10==0)
{
eventBeginWork.SetEvent();
}

//Sleep(rand()%800);
Sleep(rand()%300);
}
return 0;
}

UINT workThread(LPVOID pParam)
{
int ThreadNo;
std::queue<int> lst_work_thread_data;
time_t t;
srand((unsigned) time(&t));

ThreadNo = *((int*)pParam);

//CString sPrintMessage;
//sPrintMessage.Format(_T("线程%d开始"),ThreadNo);
//AfxMessageBox(sPrintMessage);

while(isMainThreadWorking){

WaitForSingleObject(eventBeginWork,INFINITE);

critical_section.Lock();

//CString threadNo;
//threadNo.Format(_T("线程%d正在运行"),ThreadNo);
//((CDataWriterDlg*)AfxGetApp()->m_pMainWnd)->m_display_static.SetWindowTextW(threadNo);
running_thread_no.push(ThreadNo);

while(!lst_data.empty())
{
lst_work_thread_data.push(lst_data.front());
lst_data.pop();
}

critical_section.Unlock();

//Sleep(rand()%1200);
Sleep(200+ThreadNo*100);
}

while(!lst_work_thread_data.empty())
{
lst_p_thread_data[ThreadNo].push(lst_work_thread_data.front());
lst_work_thread_data.pop();
}

return 0;
}

void CDataWriterDlg::OnBnClickedStart()
{
// TODO: 在此添加控件通知处理程序代码
isMainThreadWorking = true;
pPostThread = AfxBeginThread((AFX_THREADPROC)postThread,NULL);

for(int i=0;i<5;i++)
{
pWorkThread[i] = AfxBeginThread((AFX_THREADPROC)workThread,(LPVOID)&i);
}
}

void CDataWriterDlg::OnBnClickedStop()
{
// TODO: 在此添加控件通知处理程序代码
isMainThreadWorking = false;
WaitForSingleObject(pPostThread->m_hThread,INFINITE);

eventBeginWork.SetEvent();
eventBeginWork.SetEvent();
eventBeginWork.SetEvent();
eventBeginWork.SetEvent();
eventBeginWork.SetEvent();

WaitForSingleObject(pWorkThread[0]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[1]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[2]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[3]->m_hThread,INFINITE);
WaitForSingleObject(pWorkThread[4]->m_hThread,INFINITE);

}

void CDataWriterDlg::OnBnClickedSave()
{
// TODO: 在此添加控件通知处理程序代码
CStdioFile mFile;
CFileException mExcept;
mFile.Open(_T("Result.txt"), CFile::modeCreate|CFile::modeWrite, &mExcept);
CString string;
string.Format(_T("count=%d\r\n"),count);
//string.Append(_T(","));

for(int i=0;i<5;i++)
{
CString threadNO;
threadNO.Format(_T("Thread No:%d\r\n"),i);
string.Append(threadNO);
while(!lst_p_thread_data[i].empty())
{
CString values;
values.Format(_T("%d,"),lst_p_thread_data[i].front());
lst_p_thread_data[i].pop();
string.Append(values);
}
string.Append(_T("\r\n"));
}

string.Append(_T("\r\n"));
string.Append(_T("running thread no:\r\n"));
while(!running_thread_no.empty())
{
CString values;
values.Format(_T("%d,"),running_thread_no.front());
running_thread_no.pop();
string.Append(values);
}

mFile.WriteString(string);
mFile.Close();
}
fronz 2007-08-07
  • 打赏
  • 举报
回复
啊,原来写在一起的,看错了。
fronz 2007-08-07
  • 打赏
  • 举报
回复
MARK,
还不如直接设置固定时间延迟来看看到底结果如何。2000,2500,。。
应该有变化吧。
jasonshark 2007-08-07
  • 打赏
  • 举报
回复
Shit... I misunderstood the time function...
I thought it was a constructor....

Sorry... :(
jasonshark 2007-08-07
  • 打赏
  • 举报
回复
oops, I meant rand(0xCCCCCCCC) % 1200
jasonshark 2007-08-07
  • 打赏
  • 举报
回复
I guess these two lines caused the problem:

time_t t;
srand((unsigned) time(&t));

this makes srand to set the same seed every time under DEBUG build(0xCCCCCCCC), so all your threads are actually Sleeping for the same amount of time: 0xCCCCCCCC % 1200
yjgx007 2007-08-07
  • 打赏
  • 举报
回复
好复杂
DentistryDoctor 2007-08-07
  • 打赏
  • 举报
回复
因为有同步,所以,总是只有一个线程能够抢到。。。

15,471

社区成员

发帖
与我相关
我的任务
社区描述
VC/MFC 进程/线程/DLL
社区管理员
  • 进程/线程/DLL社区
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
暂无公告

试试用AI创作助手写篇文章吧