64,648
社区成员
发帖
与我相关
我的任务
分享C++11 标准库多线程问题
各位大神，下面是我在vs中写多线程的时候遇到的错误，这个错误很奇怪，出现在<memory>中，貌似是释放内存出错？是这样吗？
****************************************************************************
这个错误呢是一会儿正确，程序能够正确运行，一会程序就会出现这个错误，而且都不是在debug的时候出错，ctrl+F5 也就是运行的时候出错，没有办法，定位，请各位大神给点建议呗，都几天了，时间都耗在这个问题上了！
****************************************************************************
这个错误呢是一会儿正确，程序能够正确运行，一会程序就会出现这个错误，而且都不是在debug的时候出错，ctrl+F5 也就是运行的时候出错，没有办法，定位，请各位大神给点建议呗，都几天了，时间都耗在这个问题上了！
...全文
请发表友善的回复…
发表回复
jack_leiwin 2013-12-01
- 打赏
- 举报
前辈，你看一下我下面这段代码，就几行，不是很麻烦
while(taskList.size()>0||stdThreadNumA>0)
{
if(taskList.size()>0)
{
stMutexN.lock();
stdThreadNumA++;
stMutexN.unlock();
thread newThread(standardThreadOfCreateInternalOrLeafNode,&pba);
newThread.detach();
}
else
{
sleep(1);
}
}
taodm 2013-11-19
- 打赏
- 举报
楼主这个问题，基本属于滥用智能指针导致的多线程并发问题。
google 当析构函数遇到多线程── C++ 中线程安全的对象回调
如果这个没有帮助的话，楼主这代码基本就得扔了。大海捞针式排错是太痛苦了。
另外，汇编对楼主的问题完全没有帮助，是楼主没学好C++基础
jack_leiwin 2013-11-19
- 打赏
- 举报
大神，您的意思是说，这个不一定是智能指针出现的问题是吗？
[/quote][/quote]
谢谢！
[/quote][/quote]
谢谢！jack_leiwin 2013-11-19
- 打赏
- 举报
可惜我痛恨我最，没能够学好汇编！
现在有太忙！要想当年本科时候的时光，都不知道该说啥了！
现在有太忙！要想当年本科时候的时光，都不知道该说啥了！
todd_leftcode 2013-11-14
- 打赏
- 举报
我啥也没说，瞎装装牛B而已。
但重构代码可能真的有必要，要持续进行重构，拆解各个功能，分解各种耦合，最后达到拿出一个类可以直接贴到另一个空项目中进行测试（当然这只是个比方，意思是说要可以单元测试）。不要写了10天代码没报过错，第11天崩溃了，然后还得倒回去一行行分析前10天的代码。
大神，您的意思是说，这个不一定是智能指针出现的问题是吗？[/quote]
但重构代码可能真的有必要，要持续进行重构，拆解各个功能，分解各种耦合，最后达到拿出一个类可以直接贴到另一个空项目中进行测试（当然这只是个比方，意思是说要可以单元测试）。不要写了10天代码没报过错，第11天崩溃了，然后还得倒回去一行行分析前10天的代码。
大神，您的意思是说，这个不一定是智能指针出现的问题是吗？[/quote]
赵4老师 2013-11-13
- 打赏
- 举报
崩溃的时候在弹出的对话框按相应按钮进入调试，按Alt+7键查看Call Stack里面从上到下列出的对应从里层到外层的函数调用历史。双击某一行可将光标定位到此次调用的源代码或汇编指令处。
dyw 2013-11-12
- 打赏
- 举报
贴出库代码没有多少意义。看错误信息，Access Violation通常是指针未初始化问题。可以把业务代码临时屏蔽，渐次放开部分代码 逐步找出错误所在。或者添加Log来追踪。
jack_leiwin 2013-11-12
- 打赏
- 举报
能举一些出现此问题的例子吗？
jack_leiwin 2013-11-12
- 打赏
- 举报
大神，您的意思是说，这个不一定是智能指针出现的问题是吗？
todd_leftcode 2013-11-11
- 打赏
- 举报
narrow down your problem first!
你贴了图上这段代码是一个C++运行时的内部工具类，能此起这个类出现故障的原因、模式太多了。
如果你发现你根本没有任何思路narrow down your problem，那你基本应该重写/重构你的代码了。
用C++写程序，必须步步为营稳扎稳打
lm_whales 2013-11-07
- 打赏
- 举报
在一个双8核cpu,32线程的年代说这话是不是差了点意思[/quote]
更加随机了。
jack_leiwin 2013-11-07
- 打赏
- 举报
jack_leiwin 2013-11-07
- 打赏
- 举报
今天发现，出现的错误不是唯一的，下面显示在另一个位置出错了
menzi11 2013-11-07
- 打赏
- 举报
在一个双8核cpu,32线程的年代说这话是不是差了点意思
SKATE11 2013-11-06
- 打赏
- 举报
多线程调度的先后顺序是随机的 能不用就别用
赵4老师 2013-11-06
- 打赏
- 举报
仅供参考
//循环向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;
}
menzi11 2013-11-06
- 打赏
- 举报
从第二张图的代码片段来看,_Uses应该是个原子计数器,
每次和_Uses相关的线程退出时_Uses减一,并检测如果_Uses==0
的话(也就是说明这是最后一个和资源相关的线程在退出),便摧毁
和这几条线程相关的资源,其实就是维护了一个带原子引用计数的单例,只不过资源没有
封装成一个类罢了.
_Uses没看着定义,看上去应该是static std::atomic<int>,楼主你可以在这个位置
下个断点:
if(_MT_DECR(_Ignored,_Uses)==0) {
__debugbreak(); //这里
_Destroy();
.................
}
下断点后运行,考察if成功时_Uses的值,如果_Uses的值==0,但命中断点的当前运行的线程
并不是最后一个被退出的线程,或者uses==0但其他线程仍在占用资源,说明计数器_Uses写的
有问题.又或者你在某处出现了意外修改了_Uses多占内存的操作.
又或者这个资源的全局计数器要求每增加一条线程都要你手动inc,你却没有inc.
根据你说的"debug没错release出错,一会儿不出错一会儿出错",我非常怀疑是资源
计数器没有跟随线程数增长所以资源计数器一直是1,那么第一个线程退出时
资源就被释放了.肯定就错了
stereoMatching 2013-11-05
- 打赏
- 举报
如果用了smart pointer,有遵守RAII的话
可能的问题（个人不多的经验）是死锁，race condition
这些没有代码真的是不知道问题出在那里
还有一个初学者常见的错误
void do_something()
{
std::unique_ptr<object> obj(new obj);
std::async(func, obj);
}
icosagon 2013-11-05
- 打赏
- 举报
看样子是用了智能指针，
一会正确一会错误，可能是多线程问题，没有源码也看不出来，只能猜一下。
看你贴的图，智能指针的实现是线程安全的，看你怎么用的智能指针，一般直接传值的情况，智能指针是不会有多线程问题的，传引用会有多线程问题。
stereoMatching 2013-11-05
- 打赏
- 举报
不知道你的代码有没有遵守RAII的原则？
没有的话先把代码都用stl的containers或smart pointer
保护起来，std::unique_ptr无论空间或dereference都可以
做到zero cost,不用担心性能问题，一切应以稳定以及可读性
为准，微调是后期的事情
加载更多回复（5）
