请教Cstring += 崩溃问题

[女神打Boss]

LONG CDDAAnalyzerDlg::OnSerialPortCommunication(WPARAM ch, LPARAM port)
{
	str_myCommTemp.Format("%X", ch);
	//遍历串口资源
	for (int i = 0; i < COMMNUMBER; i++)
	{
		if (my_stuComm.b_comm[i])//已打开的串口
		{
			if (port == my_stuComm.i_commNum[i])//找对应的串口号
			{
是这一句	       	my_stuComm.str_comm[i] += str_myCommTemp;//收到数据
				my_stuComm.i_timeCount[i] = (int)TIMEOUT;//重置定时器计数
				break;
			}
		}
	}
	return 0;
}
这个结构体
	struct my_comminfo
	{
		int i_commNum[COMMNUMBER];//打开过的串口号
		BOOL b_comm[COMMNUMBER];//与之对应的串口打开标志
		CSerialPort my_SerialPort[COMMNUMBER];//串口资源对象
		CString str_comm[COMMNUMBER];//串口收到数据
		int i_timeCount[COMMNUMBER];//定时器计数
	};
	my_comminfo my_stuComm;
str_myCommTemp;//这是个Cstring类型成员变量
//调试崩溃断点
extern "C" _CRTIMP int __cdecl _CrtIsValidHeapPointer(
        const void * pUserData
        )
{
        if (!pUserData)
            return FALSE;

        if (!_CrtIsValidPointer(pHdr(pUserData), sizeof(_CrtMemBlockHeader), FALSE))
            return FALSE;

        return HeapValidate( _crtheap, 0, pHdr(pUserData) );
}//断点在这

调用堆栈



其他说明，这是一个接收多个串口的消息响应函数，每个字符就触发一次这个函数，所以函数速度很快
崩溃的时间不定，可能是运行5分钟，可能是1小时

请问我该怎么解决，用什么方法替换 += ，或者 += 在使用时应注意什么细节







————————————————分割广告————————————————————————————

[女神打Boss]

//串口消息响应
LONG CDDAAnalyzerDlg::OnSerialPortCommunication(WPARAM ch, LPARAM port)
{
	str_myCommTemp.Format("%X", ch);
	//遍历串口资源
	for (int i = 0; i < COMMNUMBER; i++)
	{
		if (my_stuComm.b_comm[i])//已打开的串口
		{
			if (port == my_stuComm.i_commNum[i])//找对应的串口号
			{
				my_stuComm.b_strcomm[i] = FALSE;
				my_stuComm.str_comm[i] += str_myCommTemp;//收到数据
				my_stuComm.b_strcomm[i] = TRUE;
				my_stuComm.i_timeCount[i] = (int)TIMEOUT;//重置定时器计数
				break;
			}
		}
	}
	return 0;
}
//定时器处理
void CALLBACK CDDAAnalyzerDlg::my_TimeEvent(UINT uTimerID, UINT uMsg, DWORD_PTR dwUser, DWORD_PTR dw1, DWORD_PTR dw2)  
{
	CDDAAnalyzerDlg* mainDlg = (CDDAAnalyzerDlg*)dwUser;
	for (int i = 0; i < COMMNUMBER; i++)
	{
		if (mainDlg->my_stuComm.i_timeCount[i] <= 0 && mainDlg->my_stuComm.b_comm[i])//串口定时器计数小于等于0  //串口是打开的
		{
			//处理串口数据
			
			if (mainDlg->my_stuComm.str_comm[i] != "" && mainDlg->my_stuComm.b_strcomm[i])//串口有数据 //互斥变量判断
			{
				mainDlg->str_myTimerComm.Format("COM%dRecv:%s", mainDlg->my_stuComm.i_commNum[i], mainDlg->my_stuComm.str_comm[i]);
				mainDlg->DataDisplay(mainDlg->str_myTimerComm);///???
				//mainDlg->my_stuComm.str_comm[i].Empty();//清空
				mainDlg->my_stuComm.str_comm[i] = "";
			}
		}
		mainDlg->my_stuComm.i_timeCount[i]--;//定时器计数减1
	}
}

不知道我这样加的锁对不对

[zgl7903]

建议使用 CRITICAL_SECTION InitializeCriticalSection EnterCriticalSection LeaveCriticalSection DeleteCriticalSection

[赵4老师]

不用CString，只用BYTE []或BYTE *不就完了。

[Eleven]

是不是有内存泄漏？？

[oyljerry]

是不是数据同步等的问题。被同时修改读取，改坏地址了

[女神打Boss]

引用 3 楼 oyljerry 的回复:

是不是数据同步等的问题。被同时修改读取，改坏地址了

这个好像有可能，我在一个定时器中使用了这个 ‘

//定时器处理
void CALLBACK CDDAAnalyzerDlg::my_TimeEvent(UINT uTimerID, UINT uMsg, DWORD_PTR dwUser, DWORD_PTR dw1, DWORD_PTR dw2)  
{
	CDDAAnalyzerDlg* mainDlg = (CDDAAnalyzerDlg*)dwUser;
	for (int i = 0; i < COMMNUMBER; i++)
	{
		if (mainDlg->my_stuComm.i_timeCount[i] <= 0 && mainDlg->my_stuComm.b_comm[i])//串口定时器计数小于等于0  //串口是打开的
		{
			//处理串口数据
			if (mainDlg->my_stuComm.str_comm[i] != "")//串口有数据
			{
				mainDlg->str_myTimerComm.Format("COM%dRecv:%s", mainDlg->my_stuComm.i_commNum[i], mainDlg->my_stuComm.str_comm[i]);
				mainDlg->DataDisplay(mainDlg->str_myTimerComm);///???
				mainDlg->my_stuComm.str_comm[i].Empty();//清空
			}
		}
		mainDlg->my_stuComm.i_timeCount[i]--;//定时器计数减1
	}
}

我该怎么处理呢，我已经有定时器计数来控制了，再加个互斥变量？

[zgl7903]

可能 + 和 Empty 冲突了， 加锁试试看

[女神打Boss]

引用 9 楼 zgl7903 的回复:

DataDisplay 是不是有累计历史显示？ 限制一个最大长度，超出这个长度 丢弃最前面的 运行一段时间后 设置断点停下来 查看字符串长度是否异常大， GDI句柄数、线程数等，如果无太大变化基本可认为正常， 运行1天1周再对比

你说的这些在哪看

[赵4老师]

检查是否资源泄漏的办法之一： 在任务管理器 进程 查看 选择列 里面选择：内存使用、虚拟内存大小、句柄数、线程数、USER对象、GDI对象 让你的程序(进程)不退出,循环执行主流程很多遍，越多越好，比如1000000次甚至无限循环，记录以上各数值，再隔至少一小时，越长越好，比如一个月，再记录以上各数值。如果以上两组数值的差较大或随时间流逝不断增加，则铁定有对应资源的资源泄漏！ 搜“GDI泄露检测”

[女神打Boss]

引用 7 楼 zgl7903 的回复:

建议使用 CRITICAL_SECTION InitializeCriticalSection EnterCriticalSection LeaveCriticalSection DeleteCriticalSection

这样可以，但是有内存增加的现象，不用这个有时候可以运行1小时左右不会有内存增加，不知道为什么

[zgl7903]

DataDisplay 是不是有累计历史显示？ 限制一个最大长度，超出这个长度 丢弃最前面的 运行一段时间后 设置断点停下来 查看字符串长度是否异常大， GDI句柄数、线程数等，如果无太大变化基本可认为正常， 运行1天1周再对比

[女神打Boss]

7楼可以解决这个问题 但是我内存增加的问题好像并没有解决，留这个坑以后再说吧