标题:QNetworkReply 崩溃，非法访问
作者:介农酥
日期:2019-12-25 14:51
内容:
我的程序一到中午就崩溃，我发现内存在不断的加，后来知道是因为QNetworkAccessManager管理的QNetworkReply内存需要释放，我做了试验，在finished的信号deleterLater，然后置nullptr，确实可以防止内存泄漏，但是我不知道为什么，我在finished信号槽中访问reply会崩溃，finished槽中reply难道还会NULL。我只能在finished信号对应的槽中加一个判断
if (!reply) {
return;
}
但是我网上和官网的看的几个例子，好像没有先判断这个reply是不是非法。如果非法，我之前没有deleterLater和置nullptr的时候，怎不见得崩溃。
程序说明如下，这个QNetworkAccessManager在异步请求的时候，还会有其他请求，所以，本次finished响应不一定是本次请求发出的，可能是它后一个请求。关键代码如下：
reply = manager->get(request);
connect(reply, SIGNAL(finished()), this, SLOT(httpRead()));
void ZT::httpRead()
{
//非要加个reply是否空的判断吗？
if (!reply) {
return;
}
if (reply->error()) {
//错误直接返回，因为连条码都没有
qDebug() deleteLater();
reply = nullptr;
return;
}
......
reply->deleteLater();
reply = nullptr;
}
目前崩溃是不是因为内存泄漏，还不清楚，只能等这个好了再说，还有这个finished信号和readyReady哪个信号作为读取的比较好，我之前一个程序返回的数据比较多，在readyReady的信号槽中读取，会读取不完成，但是finished能够全部读取，所以我后面的程序都改成finished的了。
2019/12/26追加
我发现有这么一个现象，关键代码如下:
request.setUrl(QUrl(postUrl));reply = manager->get(request); ..
#1 [20091001753 12-25 17:24]
auto reply = manager.get(QNetworkRequest(QUrl("http://www.qtcn.org/")));
connect(reply,&QNetworkReply::finished,this,[&,reply]{
if(!reply->error()){
qDebug()
#2 回 20091001753 的帖子 [介农酥 12-25 21:09]
20091001753:auto reply = manager.get(QNetworkRequest(QUrl("http://www.qtcn.org/")));
connect(reply,&QNetworkReply::finished,this,[&,reply]{
if(!reply->error()){
qDebug()
#3 回 介农酥 的帖子 [angelus 12-26 09:31]
介农酥:谢谢回复，恕我直言，没感觉有什么不同啊，你这里唯一就是没有reply=NULL，reply是成员变量，匿名函数方括号我按照你的写法，提示错误，但是这和发出信号的reply应该是一样的吧。其实我看网上确实很多没有置NULL的，我是看到官网有一个例子置null，所以我也置null了，但置null为 .. (2019-12-25 21:09)
reply = manager.get(QNetworkRequest(QUrl("http://www.qtcn.org/")));
如果你是复用这行代码，有可能有多个地点调用这个函数，不要把reply定义成成员变量
尽量在使用的时候创建
QNetworkReply *reply =manager.get(QNetworkRequest(QUrl("http://www.qtcn.org/")));
然后在对应的槽里边获取QNetworkReply *reply =qobject_cast(sender());
取出数据，然后删除该指针，释放内存。如果你想加上双保险，防止内存泄漏，就创建一个列表，
创建一个指针就把指针加入到列表中，在槽中如果释放就删除列表中对应的，然后创建一个界面，监控这个列表就行
如果你使用的是同步调用，那么用成员变量是没问题的，用完释放指针然后置空
本身多次异步调用，还用的同一个指针，会有未知的风险的
#4 回 angelus 的帖子 [介农酥 12-26 12:06]
angelus:
reply = manager.get(QNetworkRequest(QUrl("http://www.qtcn.org/")));
如果你是复用这行代码，有可能有多个地点调用这个函数，不要把reply定义成成员变量
尽量在使用的时候创建
QNetworkReply *reply =manager.get(QNetworkRequest(QUrl("http://www.qtcn.org/")));
.......
谢谢回复！我试试！
我发现有这么一个现象，关键代码如下:
request.setUrl(QUrl(postUrl));
reply = manager->get(request);
//直接deleteLater
reply->deleteLater();reply = nullptr;
上面的代码不会崩溃，但好像没有这么做的。
---------------------------------------------------------------------------------
request.setUrl(QUrl(postUrl));
reply = manager->get(request);
connect(reply, &QNetworkReply::finished, this, [=]() //在QNetWorkReply的finished信号中deleteLater
{
if (reply->error()) {
//错误直接返回，因为连条码都没有
qDebug() deleteLater();
reply = nullptr;
return;
}
reply->deleteLater();
reply = nullptr;
});
上面的代码会很快崩溃。这点其实不是很理解，因为我程序的另外一个类中的http请求在finished中槽deleteLater，虽然也会崩溃，但不会这么快。
-------------------------------------------------- ..
#5 [angelus 12-26 13:10]
未知错误很可能是比较诡异的，你先修改成非成员变量形式，如果还是报错，记得把代码发上来，我试试看