最近在改进考勤系统客户端多线程实现时遇到了线程异步和野指针问题
client:多线程实现ui界面显示(主线程)、人脸检测(检测线程)、socket网络通信(通信线程)三个任务。
主线程:
- TimerEvent实时读取摄像头数据显示,并发送图像数据至detect_thread进行人脸检测
- QTimer定时发送信号至socket_thread建立TCP连接,连接成功则关闭定时器,断开连接则打开定时器(信号槽实现);
- 显示detect_thread检测的人脸框,显示socket_thread接收的来着server端返回的考勤信息。
检测线程:
- 人脸检测;
- 检测成功,送人脸框至主线程显示;(if detectFlag>0, detectFlag++) 若连续检测到5帧人脸(detectFlag>5),触发socket_thread向server端发送人脸图像
- 检测失败,(detectFlag=0)
网络通信线程:
- 基于QTcpSocket与server端创建TCP连接,得到socket通信对象;
- write发送人脸图像至sever端(detectFlag=-1);
- read接收server返回的考勤信息 。
PS:调试中出现了野指针问题,且由于线程异步运行造成的程序崩溃,原因如下
**1.野指针socket:**由于QTcpSocket对象只能在一个线程中创建及使用,而子线程的构造函数是运行在主线程下的,故我们只能在子线程 connect槽函数中给QTcpSocket对象分配内存(new)。于是我就水灵灵得忘记初始化QTcpSocket指针对象了,故子线程 connect槽函数运行之前,socket是个野指针。
**2.线程运行是异步的:**若检测线程先完成连续5次的人脸检测,触发通信线程调用socket的write接口发送数据,而此时socket还未创建连接, 没有分配内存,程序就崩溃了。
解决方法:
step1. 指针定义后一定要初始化!在检测线程的构造函数中初始化socket为nullptr;
cpp
tcpCommunicate::tcpCommunicate(QMutex& mutexFlag, QObject *parent)
:mutexFlag(mutexFlag), QObject(parent)
{
qDebug()<<"tcpCommunicate() ID: "<<QThread::currentThreadId();
tcpSocket = nullptr;
}
void tcpCommunicate::clientConnetToHost()
{
qDebug()<<"tcpCommunicate slot threadID: "<<QThread::currentThreadId();
if(!tcpSocket) tcpSocket = new QTcpSocket;
tcpSocket->connectToHost("xxx.xxx.xxx.x", 22); //连接服务器
qDebug()<<"正在连接服务器";
connect(tcpSocket, &QTcpSocket::disconnected, this, [&](){
tcpSocket->close();
tcpSocket->deleteLater();
emit tcpDisConnected();
});
connect(tcpSocket, &QTcpSocket::connected, this, [&](){
emit tcpConnected();
});
connect(tcpSocket, &QTcpSocket::readyRead, this, &tcpCommunicate::readSocketData);
}
**step2.使用指针前先判断是否为空!**socket->write前,先判断socket是否为空,是否已经创建连接。
cpp
void tcpCommunicate::sendSocketData(cv::Mat image, int* detectFlag)
{
//qDebug()<<"sendSocketData";
// JPEG压缩
vector<uchar> buf;
vector<int> params = {cv::IMWRITE_JPEG_QUALITY, 90};
imencode(".jpg", image, buf, params);
QByteArray byteBuf((const char*)buf.data(), buf.size()); //Mat->jpeg格式的QbyteArray
// QDataStream 对数据串行打包
quint64 bufSize = byteBuf.size();
QByteArray sendData; // QByteArray 作为缓冲区存储字节数据
QDataStream sendStream(&sendData, QIODevice::WriteOnly); // 串行化流
sendStream.setVersion(QDataStream::Qt_5_14); // 设置Qt串行化版本
sendStream<<bufSize<<byteBuf; // 串行发送:数据长度->人脸数据
// write
/* 线程异步产生的问题:detect线程可能快于tcpCom线程 -> 在没有建立连接的情况下tcpSocket进行数据发送*/
/* PS:指针定义时先初始化为空,使用前先判断是否为空*/
if(tcpSocket && tcpSocket->state() == QAbstractSocket::ConnectedState){
//qDebug()<<"Send TCp";
tcpSocket->write(sendData);
mutexFlag.lock();
*detectFlag = -1;
mutexFlag.unlock();
}else{
qDebug()<<"disconn";
}
}