64,637
社区成员
发帖
与我相关
我的任务
分享100分。多线程滴。。搞两天了还是有问题。。
有点长，但是条理说的还算清晰。。麻烦各位耐心看下。。
现有。。ABCD 4种线程类（继承自封装的线程类）。
队列如下。
主线程 开始任务时 分别 创建ABCD各5个，也就开了20个线程了，他们共用一个queue_。
ABCD关系如下。
主线程 -> A ，主线程拿到数据给A线程处理。
A -> B A处理完把数据给B。
B -> C B处理完把数据给C。
C -> D C处理完把数据给D。
该条数据处理完毕。
开始任务就是一口气开20个线程。一股脑跑。。等数据处理完线程自己退出没啥问题，也没有泄漏。
但是想控制线程的停止（各种崩），
暂停（SuspendThread），恢复（ResumeThread），（偶尔会卡死界面，很奇怪）
就问题多多了啊。
停止我试了：
1.停止任务时直接把queue_清空。。不说了程序直接崩。
2.每个线程对象的任务循环内 设置 退出变量。设置为false时，break。。怎么也崩！！！
3.直接TerminateThread每个线程，不安全啊。有内存泄漏。。
4.？？？？
想不到怎么搞了啊。。多线程用的不太熟。。大大们指导下把。。
能安全退出每个线程，没内存泄漏和崩溃。
现有。。ABCD 4种线程类（继承自封装的线程类）。
队列如下。
struct QueueInfo
{
DataA //A类要处理的数据，处理完存入DataB
DataB //B类要处理的数据，处理完存入DataC
DataC //C类要处理的数据，处理完存入DataD
DataD //D类要处理的数据，处理完任务结束
}
CQueue<QueuueInfo> queue_; //CQueue是简单封装的带自动锁的进队出队操作的模版。
主线程 开始任务时 分别 创建ABCD各5个，也就开了20个线程了，他们共用一个queue_。
ABCD关系如下。
主线程 -> A ，主线程拿到数据给A线程处理。
A -> B A处理完把数据给B。
B -> C B处理完把数据给C。
C -> D C处理完把数据给D。
该条数据处理完毕。
开始任务就是一口气开20个线程。一股脑跑。。等数据处理完线程自己退出没啥问题，也没有泄漏。
但是想控制线程的停止（各种崩），
暂停（SuspendThread），恢复（ResumeThread），（偶尔会卡死界面，很奇怪）
就问题多多了啊。
停止我试了：
1.停止任务时直接把queue_清空。。不说了程序直接崩。
2.每个线程对象的任务循环内 设置 退出变量。设置为false时，break。。怎么也崩！！！
3.直接TerminateThread每个线程，不安全啊。有内存泄漏。。
4.？？？？
想不到怎么搞了啊。。多线程用的不太熟。。大大们指导下把。。
能安全退出每个线程，没内存泄漏和崩溃。
...全文
请发表友善的回复…
发表回复
szhyu 2013-11-27
- 打赏
- 举报
是不是常用new delete了?
反正就是要想尽一切办法不要在线程外操作线程..
wahaha_1986 2013-11-26
- 打赏
- 举报
是滴。
现在是所有线程开始前都会等待一个全局Event，停止的时候会触发它，然后执行清空队列操作。
执行完了再让线程继续跑。自然结束任务。不挂了。但是界面线程居然有时会卡死。。我郁闷啊。[/quote]
1.你是做了什么操作后卡死的还是直接运行一段时间就自然卡死了，卡死的时候查看下CPU是否正常，有没有调用同步发送等待函数。
无证临时程序员 2013-11-26
- 打赏
- 举报
嗯。我查查。。[/quote]
信号量会不会更好点
狂风暴雨 2013-11-26
- 打赏
- 举报
为啥把程序中大量没必要使用指针的的地方换为普通成员变量居然可以了..
无证临时程序员 2013-11-25
- 打赏
- 举报
主线程 开始任务时 分别 创建ABCD各5个，也就开了20个线程了，他们共用一个queue_。Todd_Pointer 2013-11-25
- 打赏
- 举报
代码实在赖得看。 我瞎猜，你是不是在主线程或什么其它线程里设置了个退出标志，完了后就把队列清了？ 先join，等人家真退了再清，不要喊了一声fire-in-the-hole队友还没回过神来你就炸了。
赵4老师 2013-11-25
- 打赏
- 举报
仅供参考
//循环向a函数每次发送200个字节长度（这个是固定的）的buffer,
//a函数中需要将循环传进来的buffer，组成240字节（也是固定的）的新buffer进行处理，
//在处理的时候每次从新buffer中取两个字节打印
#ifdef WIN32
#pragma warning(disable:4996)
#endif
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef WIN32
#include <windows.h>
#include <process.h>
#include <io.h>
#define MYVOID void
#define vsnprintf _vsnprintf
#else
#include <unistd.h>
#include <sys/time.h>
#include <pthread.h>
#define CRITICAL_SECTION pthread_mutex_t
#define MYVOID void *
#endif
//Log{
#define MAXLOGSIZE 20000000
#define MAXLINSIZE 16000
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
static char logstr[MAXLINSIZE+1];
char datestr[16];
char timestr[16];
char mss[4];
CRITICAL_SECTION cs_log;
FILE *flog;
#ifdef WIN32
void Lock(CRITICAL_SECTION *l) {
EnterCriticalSection(l);
}
void Unlock(CRITICAL_SECTION *l) {
LeaveCriticalSection(l);
}
void sleep_ms(int ms) {
Sleep(ms);
}
#else
void Lock(CRITICAL_SECTION *l) {
pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
pthread_mutex_unlock(l);
}
void sleep_ms(int ms) {
usleep(ms*1000);
}
#endif
void LogV(const char *pszFmt,va_list argp) {
struct tm *now;
struct timeb tb;
if (NULL==pszFmt||0==pszFmt[0]) return;
vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);
ftime(&tb);
now=localtime(&tb.time);
sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);
sprintf(timestr,"%02d:%02d:%02d",now->tm_hour ,now->tm_min ,now->tm_sec );
sprintf(mss,"%03d",tb.millitm);
printf("%s %s.%s %s",datestr,timestr,mss,logstr);
flog=fopen(logfilename1,"a");
if (NULL!=flog) {
fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);
if (ftell(flog)>MAXLOGSIZE) {
fclose(flog);
if (rename(logfilename1,logfilename2)) {
remove(logfilename2);
rename(logfilename1,logfilename2);
}
} else {
fclose(flog);
}
}
}
void Log(const char *pszFmt,...) {
va_list argp;
Lock(&cs_log);
va_start(argp,pszFmt);
LogV(pszFmt,argp);
va_end(argp);
Unlock(&cs_log);
}
//Log}
#define ASIZE 200
#define BSIZE 240
#define CSIZE 2
char Abuf[ASIZE];
char Cbuf[CSIZE];
CRITICAL_SECTION cs_HEX ;
CRITICAL_SECTION cs_BBB ;
struct FIFO_BUFFER {
int head;
int tail;
int size;
char data[BSIZE];
} BBB;
int No_Loop=0;
void HexDump(int cn,char *buf,int len) {
int i,j,k;
char binstr[80];
Lock(&cs_HEX);
for (i=0;i<len;i++) {
if (0==(i%16)) {
sprintf(binstr,"%03d %04x -",cn,i);
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
} else if (15==(i%16)) {
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
sprintf(binstr,"%s ",binstr);
for (j=i-15;j<=i;j++) {
sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
}
Log("%s\n",binstr);
} else {
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
}
}
if (0!=(i%16)) {
k=16-(i%16);
for (j=0;j<k;j++) {
sprintf(binstr,"%s ",binstr);
}
sprintf(binstr,"%s ",binstr);
k=16-k;
for (j=i-k;j<i;j++) {
sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
}
Log("%s\n",binstr);
}
Unlock(&cs_HEX);
}
int GetFromRBuf(int cn,CRITICAL_SECTION *cs,FIFO_BUFFER *fbuf,char *buf,int len) {
int lent,len1,len2;
lent=0;
Lock(cs);
if (fbuf->size>=len) {
lent=len;
if (fbuf->head+lent>BSIZE) {
len1=BSIZE-fbuf->head;
memcpy(buf ,fbuf->data+fbuf->head,len1);
len2=lent-len1;
memcpy(buf+len1,fbuf->data ,len2);
fbuf->head=len2;
} else {
memcpy(buf ,fbuf->data+fbuf->head,lent);
fbuf->head+=lent;
}
fbuf->size-=lent;
}
Unlock(cs);
return lent;
}
MYVOID thdB(void *pcn) {
char *recv_buf;
int recv_nbytes;
int cn;
int wc;
int pb;
cn=(int)pcn;
Log("%03d thdB thread begin...\n",cn);
while (1) {
sleep_ms(10);
recv_buf=(char *)Cbuf;
recv_nbytes=CSIZE;
wc=0;
while (1) {
pb=GetFromRBuf(cn,&cs_BBB,&BBB,recv_buf,recv_nbytes);
if (pb) {
Log("%03d recv %d bytes\n",cn,pb);
HexDump(cn,recv_buf,pb);
sleep_ms(1);
} else {
sleep_ms(1000);
}
if (No_Loop) break;//
wc++;
if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
}
if (No_Loop) break;//
}
#ifndef WIN32
pthread_exit(NULL);
#endif
}
int PutToRBuf(int cn,CRITICAL_SECTION *cs,FIFO_BUFFER *fbuf,char *buf,int len) {
int lent,len1,len2;
Lock(cs);
lent=len;
if (fbuf->size+lent>BSIZE) {
lent=BSIZE-fbuf->size;
}
if (fbuf->tail+lent>BSIZE) {
len1=BSIZE-fbuf->tail;
memcpy(fbuf->data+fbuf->tail,buf ,len1);
len2=lent-len1;
memcpy(fbuf->data ,buf+len1,len2);
fbuf->tail=len2;
} else {
memcpy(fbuf->data+fbuf->tail,buf ,lent);
fbuf->tail+=lent;
}
fbuf->size+=lent;
Unlock(cs);
return lent;
}
MYVOID thdA(void *pcn) {
char *send_buf;
int send_nbytes;
int cn;
int wc;
int a;
int pa;
cn=(int)pcn;
Log("%03d thdA thread begin...\n",cn);
a=0;
while (1) {
sleep_ms(100);
memset(Abuf,a,ASIZE);
a=(a+1)%256;
if (16==a) {No_Loop=1;break;}//去掉这句可以让程序一直循环直到按Ctrl+C或Ctrl+Break或当前目录下存在文件No_Loop
send_buf=(char *)Abuf;
send_nbytes=ASIZE;
Log("%03d sending %d bytes\n",cn,send_nbytes);
HexDump(cn,send_buf,send_nbytes);
wc=0;
while (1) {
pa=PutToRBuf(cn,&cs_BBB,&BBB,send_buf,send_nbytes);
Log("%03d sent %d bytes\n",cn,pa);
HexDump(cn,send_buf,pa);
send_buf+=pa;
send_nbytes-=pa;
if (send_nbytes<=0) break;//
sleep_ms(1000);
if (No_Loop) break;//
wc++;
if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
}
if (No_Loop) break;//
}
#ifndef WIN32
pthread_exit(NULL);
#endif
}
int main() {
#ifdef WIN32
InitializeCriticalSection(&cs_log);
InitializeCriticalSection(&cs_HEX );
InitializeCriticalSection(&cs_BBB );
#else
pthread_t threads[2];
int threadsN;
int rc;
pthread_mutex_init(&cs_log,NULL);
pthread_mutex_init(&cs_HEX,NULL);
pthread_mutex_init(&cs_BBB,NULL);
#endif
Log("Start===========================================================\n");
BBB.head=0;
BBB.tail=0;
BBB.size=0;
#ifdef WIN32
_beginthread((void(__cdecl *)(void *))thdA,0,(void *)1);
_beginthread((void(__cdecl *)(void *))thdB,0,(void *)2);
#else
threadsN=0;
rc=pthread_create(&(threads[threadsN++]),NULL,thdA,(void *)1);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
rc=pthread_create(&(threads[threadsN++]),NULL,thdB,(void *)2);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
#endif
if (!access("No_Loop",0)) {
remove("No_Loop");
if (!access("No_Loop",0)) {
No_Loop=1;
}
}
while (1) {
sleep_ms(1000);
if (No_Loop) break;//
if (!access("No_Loop",0)) {
No_Loop=1;
}
}
sleep_ms(3000);
Log("End=============================================================\n");
#ifdef WIN32
DeleteCriticalSection(&cs_BBB );
DeleteCriticalSection(&cs_HEX );
DeleteCriticalSection(&cs_log);
#else
pthread_mutex_destroy(&cs_BBB);
pthread_mutex_destroy(&cs_HEX);
pthread_mutex_destroy(&cs_log);
#endif
return 0;
}
狂风暴雨 2013-11-25
- 打赏
- 举报
嗯。我查查。。
狂风暴雨 2013-11-25
- 打赏
- 举报
是滴。
现在是所有线程开始前都会等待一个全局Event，停止的时候会触发它，然后执行清空队列操作。
执行完了再让线程继续跑。自然结束任务。不挂了。但是界面线程居然有时会卡死。。我郁闷啊。
狂风暴雨 2013-11-25
- 打赏
- 举报
这个我也觉得前人这样不好。。前人滴。。现在放一起的但是没有互相干涉，看起来不爽。
nightkids_008 2013-11-25
- 打赏
- 举报
ABCD之间有依赖关系的话，还是用通知吧，那边线程等待这个通知。
你用条件变量去做吧，再不会去google下条件变量
wahaha_1986 2013-11-25
- 打赏
- 举报
1.我调试了。。清空队列后。有的线程恰好在出队。。拿到了空。。后面肯定崩。
你队列不是有自动锁的吗？如果是互斥锁，有的队列正在出队操作，你的主线程就不会执行清空队列操作，必须阻塞等到那个正在操作队列的线程执行完后，你才可以获取到队列资源，然后执行清空操作，而且你在做清空操作时，其他线程也必须阻塞同步等待主线程释放队列。
CoolEgos 2013-11-24
- 打赏
- 举报
学习了，呜呜呜
狂风暴雨 2013-11-24
- 打赏
- 举报
前人写的。。离职了。。客户又要东西。。
让我这13界毕业的。。界面木有，底层没完成，插件木有，全自己搞。。当锻炼了。。[/quote]
........没文档，没注释什么的么？前面的那个人什么态度啊，太不敬业了。[/quote]
一个月之前离职的时候给我讲解一下午。做其他项目呢，一个月没摸过。客户突然喊着要。注释有代码看了两天也还熟。。主要我对多线程同步，控制操作等不熟。添加功能改代码就容易出问题。
瓶盒 2013-11-24
- 打赏
- 举报
线程停止的代码怎么写的？
一般应该这样来
DWORD dwExit;
GetExitCodeThread(m_pThread->m_hThread,&dwExit);
if(TerminateThread(m_pThread->m_hThread,dwExit)){
m_bThreadRuning=false;
EndJob();
}
Adol1111 2013-11-24
- 打赏
- 举报
前人写的。。离职了。。客户又要东西。。
让我这13界毕业的。。界面木有，底层没完成，插件木有，全自己搞。。当锻炼了。。[/quote]
........没文档，没注释什么的么？前面的那个人什么态度啊，太不敬业了。
max_min_ 2013-11-24
- 打赏
- 举报
1.我调试了。。清空队列后。有的线程恰好在出队。。拿到了空。。后面肯定崩。
循环
run()
{
do
{
p_par_date_ = queue_->DataA(); //从DataA中取出数据处理。。
if(p_par_date_)
{
SetStatus(kRunning);
DataB data_b = DoSome(); //处理DataA数据。。转为DataB
xdelete(p_par_date_);
p_par_date_ = NULL;
queue_->EnDataB(data_b); //DataB进入队列。给后面线程处理。
}
else
Sleep(100);
SetStatus(kFree);
} while (true);
SetStatus(kFinish);
}
手动停止。
for(auto pThread = curls_.begin(); pThread != curls_.end(); ++pThread )
{
(*pThread )->Stop(); //该如何stop是好。。
delete *pThread;
}
其他几个线程也是一样的。
狂风暴雨 2013-11-24
- 打赏
- 举报
沉的好快啊..
狂风暴雨 2013-11-24
- 打赏
- 举报
有道理..我去试试
lm_whales 2013-11-24
- 打赏
- 举报
Dev C++ Windows xp下 ,缺省设置，控制台程序，所有线程都有输出的情况下
使用暂停（SuspendThread），恢复（ResumeThread）
调度,就很不好.
加载更多回复（6）
