QNetworkAccessManager несколько загрузок не удаются

Question

QNetworkAccessManager несколько загрузок не удаются

В моем приложении у меня есть метод для загрузки файлов на сервер, это отлично работает.

Но когда я вызываю этот метод несколько раз одновременно (например, итерируя по результату chooseFilesDialog), первые 7 (более или менее) файлов загружаются правильно, остальные никогда не загружаются.

Я думаю, что это должно быть связано с тем фактом, что сервер не допускает больше X подключений из одного источника, может быть?

Как я могу убедиться, что загрузка ожидает свободного установленного соединения?

это мой метод:

QString Api::FTPUpload(QString origin, QString destination)
{
qint64 timestamp = QDateTime::currentMSecsSinceEpoch();
QUrl url("ftp://ftp."+getLSPro("domain")+destination);
url.setUserName(getLSPro("user"));
url.setPassword(getLSPro("pwd"));

QFile *data = new QFile(origin, this);
if (data->open(QIODevice::ReadOnly))
{
QNetworkAccessManager *nam = new QNetworkAccessManager();
QNetworkReply *reply = nam->put(QNetworkRequest(url), data);
reply->setObjectName(QString::number(timestamp));
connect(reply, SIGNAL(uploadProgress(qint64, qint64)), SLOT(uploadProgress(qint64, qint64)));

return QString::number(timestamp);
}
else
{
qDebug() << "Could not open file to FTP";
return 0;
}
}

void Api::uploadProgress(qint64 done, qint64 total) {
QNetworkReply *reply = (QNetworkReply*)sender();
emit broadCast("uploadProgress","{\"ref\":\""+reply->objectName()+"\" , \"done\":\""+QString::number(done)+"\", \"total\":\""+QString::number(total)+"\"}");
}

1

c++qnetworkaccessmanager qnetworkrequest qt qt5

Решение

Другие решения

Других решений пока нет …

Источник

Accepted Answer

Во-первых, не создавайте QNetworkManager при каждом запуске загрузки.
Во-вторых, вам обязательно нужно удалить все, что вы new()иначе у вас останутся утечки памяти. Это включает в себя QFile, QNetworkManager И QNetworkReply (!).
В-третьих, вы должны ждать finished() сигнал.

Api::Api() {  //in the constructor create the network access manager
nam = new QNetworkAccessManager()
QObject::connect(nam, SIGNAL(finished(QNetworkReply*)), this, SLOT(finished(QNetworkReply*)));
}

Api::~Api() {  //in the destructor delete the allocated object
delete nam;
}

bool Api::ftpUpload(QString origin, QString destination) {
qint64 timestamp = QDateTime::currentMSecsSinceEpoch();
QUrl url("ftp://ftp."+getLSPro("domain")+destination);
url.setUserName(getLSPro("user"));
url.setPassword(getLSPro("pwd"));

//no allocation of the file object;
//will automatically be destroyed when going out of scope
//I use readAll() (see further) to fetch the data
//this is OK, as long as the files are not too big
//If they are, you should allocate the QFile object
//and destroy it when the request is finished
//So, you would need to implement some bookkeeping,
//which I left out here for simplicity
QFile file(origin);
if (file.open(QIODevice::ReadOnly)) {
QByteArray data = file.readAll();   //Okay, if your files are not too big
nam->put(QNetworkRequest(url), data);
return true;

//the finished() signal will be emitted when this request is finished
//now you can go on, and start another request
}
else {
return false;
}
}

void Api::finished(QNetworkReply *reply) {
reply->deleteLater();  //important!!!!
}

0